search

floatinghotpot / cordova-plugin-mmedia

Cordova/PhoneGap for Millennial Media Ad
7 stars 6 forks source link

Plugin requires record permissions on Android #8

Closed pantonis closed 8 years ago

pantonis commented 8 years ago

Hi,

I dont understand why a plugin that shows ads needs to record audio. Is this a bug?

floatinghotpot commented 8 years ago

It's required by mMedia SDK, as the plugin is implemented based on the SDK. You can visit mMedia SDK for more details.

pantonis commented 8 years ago

I have spoken with MMedia. They said that the audio is optional. Below is their reply "Glad to hear it's working! That's a good question-- The RECORD_AUDIO permission is optional. If your app doesn't need it, simply don't include it. We used it for interactive voice ads."

Can we have a branch that does not include this?

jmelvin commented 8 years ago

Indeed, I'm asking MMedia the same questions about several permissions...

<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
<uses-permission android:name="android.permission.WRITE_CALENDAR"/>
<uses-permission android:name="android.permission.NFC"/>
<uses-permission android:name="android.permission.BLUETOOTH"/>
<uses-permission android:name="android.permission.VIBRATE"/>
<uses-permission android:name="android.permission.RECORD_AUDIO"/>

Without knowing more, I was going to try and post-process the AndroidManifest.xml file to remove these permissions before cordova build. It may or may not be possible.