Skip to content

Commit 7921e5c

Browse files
committed
Strip READ_PHONE_STATE from merged manifest
A transitive dependency (likely ML Kit face detection / Play Services Basement in the full flavor) was injecting READ_PHONE_STATE into the merged manifest. The app doesn't need it, so remove it via the same tools:node="remove" pattern already used for INTERNET and ACCESS_NETWORK_STATE.
1 parent 7542d56 commit 7921e5c

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

app/src/main/AndroidManifest.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
<!-- Prevent dependencies from adding these permissions -->
1919
<uses-permission android:name="android.permission.INTERNET" tools:node="remove"/>
2020
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" tools:node="remove"/>
21+
<uses-permission android:name="android.permission.READ_PHONE_STATE" tools:node="remove"/>
2122

2223
<application
2324
android:name=".SnapSafeApplication"

0 commit comments

Comments
 (0)