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

Update Android permissions and features #10

Merged

Conversation

ath0mas
Copy link

@ath0mas ath0mas commented Aug 15, 2023

  1. remove FLASHLIGHT permission
    Since using the camera, with CAMERA permission, already allows to use the flashlight.
    see https://stackoverflow.com/a/48553060

2. remove explicit hardware.camera feature
The CAMERA permission automatically requires some camera features.
see https://developer.android.com/guide/topics/manifest/uses-feature-element#permissions
Let app developers decide whether these are required or not, set <uses-feature> by themselves if needed, preventing conflict with other plugins here.

  1. declare hardware.camera features but to disable filtering based on the CAMERA permission
    The CAMERA permission automatically requires some camera features.
    see https://developer.android.com/guide/topics/manifest/uses-feature-element#permissions

Related to #7 and therefore also to phonegap#882.

Since using the camera, with CAMERA permission, already allows to use the flashlight.
see https://stackoverflow.com/a/48553060
The CAMERA permission automatically requires all camera features.
see https://developer.android.com/guide/topics/manifest/uses-feature-element#permissions
Let app developers decide whether these are required or not, set <uses-feature> by themselves if needed, preventing conflict with other plugins here.
@ath0mas ath0mas changed the title Cleanup Android permissions and features Update Android permissions and features Aug 18, 2023
@ath0mas ath0mas force-pushed the android-permissions-and-features branch from a4c5f78 to 5564ff2 Compare August 19, 2023 00:04
@okhiroyuki
Copy link
Owner

@ath0mas
Your PR was very helpful.
Thank you very much.

@okhiroyuki okhiroyuki merged commit 2e72d4c into okhiroyuki:master Aug 20, 2023
@ath0mas ath0mas deleted the android-permissions-and-features branch August 20, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants