breautek commented on PR #1609:
URL: https://github.com/apache/cordova-android/pull/1609#issuecomment-1906327025

   > But I'm not sure why @breautek thinks intent doesn't need to grant 
permission? Maybe there's a different approach?
   
   I know there was behaviour which was handled in the camera plugin where if 
the `CAMERA` permission is declared in the manifest, then the `CAMERA` 
permission needs to be granted at the app-level. (Which somewhat defeats the 
purpose of using Intents...) Perhaps you have the `CAMERA` permission declared 
in your android manifest somehow, via another plugin or something. The camera 
plugin handles this by checking if the camera permission is present in the 
manifest and only request the camera permission if it is.
   
   Google has announced that starting in August 2024, that they are going to 
start cracking down on apps declaring or requesting permissions that the app 
doesn't actually use or need, so we need to pay close attention when it comes 
to adding new permission requirements.
   
   @jacobg If you could, open up the app that you're testing with inside 
Android Studio and view your `AndroidManifest.xml` file. The Android Studio 
editor for the manifest file will have a tab for `Merged Manifest` view, which 
should represent your manifest as natively compiled. If you can see that the 
`CAMERA` permission is declared, it should tell what library/module it is 
coming from. Likely it will be the "app" module since a plugin that you have 
installed is probably adding it. If you could temporarily remove the CAMERA 
declaration in the manifest and re-test, I think you should find that it will 
work as expected without the camera permission. You would also of course have 
to temporarily disable any code that attempts to request the camera permission 
as well.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org

Reply via email to