Skip to content

Commit e2758cf

Browse files
committed
mobile: drop ACCESS_BACKGROUND_LOCATION from the manifest
The permission was declared but never requested at runtime, so no user was ever prompted for it and nothing depended on it. get_location reads the network provider's last known position from inside the foreground service, which is declared with foregroundServiceType="location"; Android treats that as foreground access while the node runs off-screen, and it needs only ACCESS_COARSE_LOCATION. Play Console flagged the declared-but-unused permission and asked for a background-location declaration, which an optional sensor toggle would not pass. Removing it makes the form moot.
1 parent b4313a8 commit e2758cf

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

mobile/sam-node-app/android/app/src/main/AndroidManifest.xml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
<manifest xmlns:android="http://schemas.android.com/apk/res/android">
22
<uses-permission android:name="android.permission.INTERNET"/>
3+
<!-- Coarse only, and no ACCESS_BACKGROUND_LOCATION: get_location is served
4+
from the foreground service (type "location"), which Android counts as
5+
foreground access while the node runs off-screen. -->
36
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION"/>
4-
<uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION"/>
57
<uses-permission android:name="android.permission.FOREGROUND_SERVICE"/>
68
<uses-permission android:name="android.permission.FOREGROUND_SERVICE_CONNECTED_DEVICE"/>
79
<uses-permission android:name="android.permission.FOREGROUND_SERVICE_LOCATION"/>

0 commit comments

Comments
 (0)