-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Permissions not requested #1100
Comments
I think I've got a similar problem, problems with google maps, after opening the app, (I'm running google maps version 10.19.2 because of the message about google play services that needed to be updated..) please @mar-v-in check and fix |
@D3SOX I noticed you have permission flag APPLY_RESTRICTION set to the background location permission. My phone does not have that: |
|
* on Friday, 2020-10-16 09:41 +0000, Marvin W <notifications@github.com> wrote:
`ACCESS_BACKGROUND_LOCATION` is a hard restricted permission, which means that the decision if you can enable it is done by the app installing it (or if the app is part of the system image, the system decides this). Thus, this maybe is an issue of NanoDroid? @Nanolx
Going through the list of requested permissions I see that there are two permissions not currently granted on my system:
ACCESS_BACKGROUND_LOCATION
ACCESS_COARSE_UPDATES
I guess these are both hard restricted permissions?
|
The ROM is Paranoid Android, self compiled with signature spoofing patch Deleting data didn't change anything Nevertheless, I don't think this is as big of an issue since I haven't noticed anything that doesn't work. Other Apps with background location granted work as expected. |
@mar-v-in likely. I've observed that installing NanoDroid directly into /system seems to always work, but depending on the ROM installing as Magisk Module may fail in this regard. A temporary fix is to install uG as user app
(or I'll investigate further, but may take a few days. |
Yes, it's just like that. Installing NanoDroid into /system before booting the ROM the first time (just as one does with GApps; install ROM > install NanoDroid to /system > boot into ROM) solves that issue. It also properly auto-grants all permissions to GmsCore (except Most A/B or S-A-R devices require /data to be clean when starting the first time (observed this on OnePlus 5T and OnePlus 6), thus installing NanoDroid as Magisk Module is always "dirty flashed" (flash ROM > boot into ROM > boot into TWRP > Flash Magisk > Flash NanoDroid); "clean flash" (install ROM > install Magisk > install NanoDroid > boot into ROM) like mentioned above results in a boot loop on those devices, thus is not possible. For one reason or another Android does deny On older devices where you can flash ROM and Magisk in one go and sucessfully boot up, this does not happen. As mentioned before I observed this on both Android 10 (AICP 15) and Android 11 (Syberia 4.1). Tomorrow I'll update the Magisk Modules's Since no Magisk GApps package exists (until now I never thought about why that's the case) and given that privapps should be available right from the very first start, I suspect that behaviour is actually on purpose, so that proposed change to Unless someone can point me into a different direction. |
…permissions to be granted, see microg/GmsCore#1100 (comment)
One thing left to try is requesting the permission on behalf of the gmscore userId, similar to #1102 On my phone the permission is granted, but when I try to revoke the permission as root, I get:
But using the app's system user id works:
So if you grant it with the "--user" option it might work too? The permission on my phone:
I suppose different roms have different permission flags, the "RESTRICTION_SYSTEM_EXEMPT" makes it work with the "--user" option and "RESTRICTION_UPGRADE_EXEMPT" allows granting the permission on upgrading the apk. @D3SOX None of the exempt flags are set in your case, so I doubt any of these two methods work for you? Considering you rom is Paranoid Android, this may be by design. |
@ploink I've modified the For reference: https://gitlab.com/Nanolx/NanoDroid/-/commit/0437a12adf71a28b370fe0d3a92b92609ef1c038 Time to roll-out NanoDroid 22.90 (23.0-beta1), because now after all this rework and fixing is almost finished, it's time to give brave users the opportunity to test. :] |
Hi, I can't grant some permissions either (They also have
I'm running ProtonAOSP (with Android 11) on a Pixel 5 using Magisk and the microG installer revived module. |
The fix is to install microG not just as a priv-app, but also as a user-app, i. e. an update. a) go to /data/adb/modules/microG_blabla/system/priv-app/GmsCore/GmsCore.apk and tap on the APK to install it as update profit. |
@Nanolx That's a very important step to take. I had GmsCore installed in priv-app but not as an user app and was facing the same issue. Now, being a priv-app and a user app it works. I'll be looking into how to get location to work properly as apps don't seem to be able to get any location even though I have 3 network-based geolocation modules installed (do I need to also install address lookup modules?). I'm using Android 11 (halogenOS) |
@Nanolx - thanks for details. Unfortunately even if did the steps, after reboot and re-opening MicroG - it shows all permissions in place and checking UI "Self-check" shows all checked (originally did check the last two remaining - Location in background and SMS access earlier, straight after installing User-level app following other steps). Waze still shows "Connecting to network" and nothing shows on map. What to do now? Need to re-install MicroG or something?
|
Reinstalled Waze, on launch got question about location access with only two options:
Nothing to select in background. After accepting the only positive option of when running ap, got message "microG DroidGuard Helper keeps stopping". Am on Android 10 (Unihertz Jelly 2). |
Coming back to the location issue, I have been told (and observed) that simply waiting for a while (like 1-2 days) will help get location working. For me, now it works perfectly fine. Network-based geolocation modules:
Address lookup modules:
All of these are obtainable through F-Droid. |
Many thanks for swift answer! Found a workaround by logging in to old Waze account with email via notifications and then all worked all of the sudden. Miserably it is gmail account :-O :-( Second observation is that at the time when problem persists, no search for places works well, first page of results shows up, but not extended ("Around you", after typing name and then "Search") where "Google, FourSquare" and others are. Also additional observation is that there was no Terms & Conditions shown as normally is. Some something around some Google Services is going on or other sort of registration. |
Some of those may be outdated and I am not sure if they interfere with each other. I suggest you start with only one that is known to work before you try others, for example the GSM Location Service fork from devee. He continued the original work from n76 and included some of the code from my fork to make it work with Android 10 and 4G/5G. Remember to go into the backend config and download the appropriate database for your region. Nominatim is good, keep that. |
@ploink thanks for your suggestion!
From my understanding, they do different things in different ways allowing combined location (WiFi, radio, etc.) I will start disabling one after the other, seeing if they worsen location accuracy or speed. EDIT: I'm already using the GSM Location Service fork you suggested. |
Everything should already be fixed on the latest version so I close it for now. |
First, thank you very much for latest release! (v0.2.11.202414)
Even though I collected all the checkmarks in self-check, gmscore is still missing permissions ACCESS_FINE_LOCATION and READ_EXTERNAL_STORAGE. It does not request them.
One can check for ungranted permissions with:
$ dumpsys package com.google.android.gms | grep granted=false
For now I have granted the permissions manually:
$ pm grant com.google.android.gms android.permission.ACCESS_FINE_LOCATION
$ pm grant com.google.android.gms android.permission.READ_EXTERNAL_STORAGE
The text was updated successfully, but these errors were encountered: