Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Version 1.1.0 requires Permission ACCESS_BACKGROUND_LOCATION #86

Open
louis123562 opened this issue Apr 26, 2024 · 4 comments
Open

Version 1.1.0 requires Permission ACCESS_BACKGROUND_LOCATION #86

louis123562 opened this issue Apr 26, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@louis123562
Copy link

If we try to publish an internal release in the google play console, we get two errors, smth like that:

Error
Your declaration of permissions for location determination in the background needs to be updated.

Error
This release contains permissions that have not yet been explained in the Play Console. You can update the permissions declarations via the sensitive app permissions. Further information

I guess this comes from the Manifest in the background-runner sources itself.

image

In our own Manifest, we do not add ACCESS_BACKGROUND_LOCATION:

image

We are not in the need of accessing the GPS-Location with the background runner.

Is there a way to take the permission out?

@louis123562
Copy link
Author

In the manifest merger report, it is visible, that it comes from the background runner:

24-->[:capacitor-background-runner] /Users/l.spieckerhoff/workspace/repositories/planzeit-ionic-v5-neu/node_modules/@capacitor/background-runner/android/build/intermediates/merged_manifest/debug/AndroidManifest.xml:11:5-77
24-->[:capacitor-background-runner] /Users/l.spieckerhoff/workspace/repositories/planzeit-ionic-v5-neu/node_modules/@capacitor/background-runner/android/build/intermediates/merged_manifest/debug/AndroidManifest.xml:11:22-74
25    <uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION" />

@theproducer theproducer self-assigned this Apr 26, 2024
@theproducer theproducer added the enhancement New feature or request label Apr 26, 2024
@theproducer
Copy link
Collaborator

Yeah, this has been a known potential issue on iOS as well.

Im working on a potential solution to this that may come in the form of a CLI that will help add / remove APIs and their relevant permissions (in addition to removing unused API code, which could be an issue as well).

@louis123562
Copy link
Author

Yeah, to solve this issue temporarily, i deleted the lines in the AndroidManifest.xml in node_modules.

@louis123562
Copy link
Author

Workaround for iOS: When not checking "Location updates" in Background Modes, the iOS app crashes on the first dispatchEvent. If you do not need to track the gps location with the runner, disable the functionality in node_modules:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants