You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cordova.apache.org by GitBox <gi...@apache.org> on 2019/05/31 08:33:30 UTC

[GitHub] [cordova-plugin-media] adriano-di-giovanni opened a new pull request #232: android: remove READ_PHONE_STATE permission

adriano-di-giovanni opened a new pull request #232: android: remove READ_PHONE_STATE permission
URL: https://github.com/apache/cordova-plugin-media/pull/232
 
 
   ### Platforms affected
   
   * Android
   
   
   ### Motivation and Context
   
   The plugin uses the `OnAudioFocusChangeListener` to listen to audio focus changes such as the ones that happen on incoming calls. There is no need to request the `android.permission.READ_PHONE_STATE` because it is not needed.
   I think it's good to remove such permission because it has a _dangerous_ protection level.
   
   ### Description
   
   Removed `<uses-permission android:name="android.permission.READ_PHONE_STATE" />` frrom `plugin.xml`
   
   

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


With regards,
Apache Git Services

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