Cordova-plugin-googlemaps: Why do You need READ_PHONE_STATE

[piernik]

I'm submitting a ... (check one with "x")

• [x] question
• [ ] any problem or bug report
• [ ] feature request

If you choose 'problem or bug report', please select OS: (check one with "x")

• [x] Android
• [ ] iOS

After installing this plugin app needs READ_PHONE_STATE permission.
I get feedback - people doesn't like it :/
My app is getting low score only because of that permission.
Is it necessary? What is it for?

[wf9a5m75]

Does this plugin request the READ_PHONE_STATE permission?
https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/4bc39caafe52815148892956197500bb44238ce0/plugin.xml#L222-L234

[wf9a5m75]

ping

[battika]

Actually I was also surprised to see that the latest APK I submitted to Play Store also required READ_PHONE_STATE. I could not figure it why, but it was definitely there.

[piernik]

I've made clean install of ionic and added google maps plugin. Then in platforms/android/build/intermediates/manifest/full/release/AndroidManifest.xml was added:

    <android:uses-permission android:name="android.permission.READ_PHONE_STATE" />
    <android:uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />

[wf9a5m75]

This plugin does not request the READ_EXTERNAL_STORAGE and the READ_PHONE_STATE permissions.
Did you see the plugin.xml?

[wf9a5m75]

Check out the plugin.xml first.

https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/4bc39caafe52815148892956197500bb44238ce0/plugin.xml

[makers-mark]

@piernik & @battika

It looks to be a play services issue according to this stack overflow thread. You should read all of this, even the replies, and a potential workaround for now:

<manifest ...
    xmlns:tools="http://schemas.android.com/tools"
    ... >

<uses-permission
    android:name="android.permission.READ_PHONE_STATE"
    tools:node="remove" />

EDIT: In particular google play services 12.0.0

[battika]

Thanks, so the new Play Services version is causing READ_PHONE_STATE to be a required feature.

[makers-mark]

@wf9a5m75
@battika
@piernik
It is fixed maybe with 12.0.1?

[piernik]

@wf9a5m75
@makers-mark
I've installed newest version of this plugin 2.3.0-rc1, deleted platform folder and when I build app
android.permission.READ_PHONE_STATE is still there.
Can You confirm that READ_PHONE_STATE is not required?

Edit: I think it works. I've removed plugin and added rc. BTW there is error in https://github.com/mapsplugin/cordova-plugin-googlemaps/commit/8b5bb66a4c05e403f9e4360fcd4e1c937d518b51

cordova plugin add cordova-plugin-googlemaps#2.3.0-rc1 should be cordova plugin add [email protected]

[wf9a5m75]

Did I write cordova plugin add cordova-plugin-googlemaps#2.3.0-rc1 in somewhere?

[piernik]

In readme file: https://github.com/mapsplugin/cordova-plugin-googlemaps/commit/8b5bb66a4c05e403f9e4360fcd4e1c937d518b51 line 13

[piernik]

@wf9a5m75 You released 2.2.9 This version is dependent on google play services 15?

[wf9a5m75]

I released v2.3.1

[piernik]

Oh - great!
Thanks