[5.5.0] getInstallReferrer reqiure READ_PHONE_STATE and WRITE_EXTERNAL_STORAGE

Question

Hello my app was to require READ_PHONE_STATE and WRITE_EXTERNAL_STORAGE permissions on Android after update to 5.5.0.

I checked in this commit extra permissions not required. Then can’t build this commit and after build next commit in my android/app/build/intermediates/bundle_manifest/debug/processDebugManifest/bundle-manifest/AndroidManifest.xml added:

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

I don’t use getInstallReferrer and don’t want request extra permissions form user. Can it be excluded?

[mikehardy]

So you want something like this in your AndroidManifest:

<manifest xmlns:android="http://schemas.android.com/apk/res/android" 
  xmlns:tools="http://schemas.android.com/tools" package="com.kullki.kscore">

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

[Bardiamist]

@mikehardy

Note, that commit (I checked) also added:

<uses-permission android:name="com.google.android.finsky.permission.BIND_GET_INSTALL_REFERRER_SERVICE" />
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />

I have crash on device if try add:

<uses-permission android:name="com.google.android.finsky.permission.BIND_GET_INSTALL_REFERRER_SERVICE" tools:node="remove" />

So, my full removing permissions code:

<!-- https://github.com/react-native-community/react-native-device-info/issues/955 -->
<uses-permission android:name="android.permission.READ_PHONE_STATE" tools:node="remove" />
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" tools:node="remove" />
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" tools:node="remove" />

READ_EXTERNAL_STORAGE looks very dangerous:) Would good to make these permissions optional.