You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by GitBox <gi...@apache.org> on 2021/02/17 14:05:08 UTC

[GitHub] [cordova-plugin-media-capture] breautek opened a new issue #209: (Android): Plugin declares unnecessary permissions

breautek opened a new issue #209:
URL: https://github.com/apache/cordova-plugin-media-capture/issues/209


   # Bug Report
   
   ## Problem
   
   ### What is expected to happen?
   
   Plugin unnecessary requires `RECORD_AUDIO`, `WRITE_EXTERNAL_STORAGE`, and `READ_EXTERNAL_STORAGE` permissions.
   
   `RECORD_AUDIO` only is required if the app uses the microphone APIs directly, but this plugin uses an intent to record audio.
   
   The external permissions are already handled by the cordova-plugin-file, which this plugin depends on.
   
   ### What does actually happen?
   
   Not to declare unnecessary permissions.
   
   ## Information
   <!-- Include all relevant information that might help understand and reproduce the problem -->
   
   This is a GH extension of an existing jira ticket: https://issues.apache.org/jira/browse/CB-10814
   
   Special testing may be necessary around what happens if the app (from an external source) declares `RECORD_AUDIO`, will it causes `SecurityErrorException` like with the `CAMERA` permission (See https://issues.apache.org/jira/browse/CB-10120)
   
   ### Command or Code
   <!-- What command or code is needed to reproduce the problem? -->
   
   N/A
   
   ### Environment, Platform, Device
   <!-- In what environment, on what platform or on which device are you experiencing the issue? -->
   
   N/A
   
   ### Version information
   <!-- 
   What are relevant versions you are using?
   For example:
   Cordova: Cordova CLI, Cordova Platforms, Cordova Plugins 
   Other Frameworks: Ionic Framework and CLI version
   Operating System, Android Studio, Xcode etc.
   -->
   cordova-plugin-media-capture: 3.0.4-dev
   
   
   ## Checklist
   <!-- Please check the boxes by putting an x in the [ ] like so: [x] -->
   
   - [x] I searched for existing GitHub issues
   - [x] I updated all Cordova tooling to most recent version
   - [x] I included all the necessary information above
   


----------------------------------------------------------------
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.

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



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