Follow gplay directions regarding privacy policy and location disclosure (#1607)

This commit is contained in:
Simon Frei 2021-02-11 20:19:37 +01:00 committed by GitHub
parent e4219eae78
commit 087a6080a2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
9 changed files with 79 additions and 38 deletions

View File

@ -27,6 +27,7 @@ android {
// Changes to these values need to be reflected in `.travis.yml`
compileSdkVersion 29
buildToolsVersion '29.0.3'
ndkVersion = '21.4.7075529'
buildTypes.debug.applicationIdSuffix ".debug"
dataBinding.enabled = true

View File

@ -6,4 +6,6 @@ Source code: https://github.com/syncthing/syncthing-android
Forum: https://forum.syncthing.net/
Issues: https://github.com/syncthing/syncthing-android/issues
Issues: https://github.com/syncthing/syncthing-android/issues
Privacy-policy: https://github.com/syncthing/syncthing-android/blob/main/privacy-policy.md

View File

@ -36,17 +36,24 @@
android:textSize="@dimen/slide_title"
android:textStyle="bold" />
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="20dp"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:text="@string/welcome_text"
android:textAlignment="center"
android:layout_gravity="center_horizontal"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="@dimen/desc_marginTop"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:paddingBottom="@dimen/desc_paddingBottom"
android:text="@string/welcome_text"
android:textAlignment="center"
android:layout_gravity="center_horizontal"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
</ScrollView>
</LinearLayout>
</RelativeLayout>

View File

@ -51,17 +51,24 @@
android:textSize="@dimen/slide_title"
android:textStyle="bold" />
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="20dp"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:text="@string/storage_permission_desc"
android:textAlignment="center"
android:layout_gravity="center_horizontal"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="@dimen/desc_marginTop"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:paddingBottom="@dimen/desc_paddingBottom"
android:text="@string/storage_permission_desc"
android:textAlignment="center"
android:layout_gravity="center_horizontal"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
</ScrollView>
</LinearLayout>

View File

@ -46,22 +46,32 @@
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="center_horizontal"
android:text="@string/location_permission_title"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_title"
android:textStyle="bold" />
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginTop="20dp"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:text="@string/location_permission_desc"
android:textStyle="bold"
android:textAlignment="center"
android:layout_gravity="center_horizontal"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
/>
<ScrollView
android:layout_width="match_parent"
android:layout_height="match_parent">
<TextView
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_gravity="center_horizontal"
android:layout_marginTop="@dimen/desc_marginTop"
android:paddingLeft="@dimen/desc_padding"
android:paddingRight="@dimen/desc_padding"
android:paddingBottom="@dimen/desc_paddingBottom"
android:text="@string/location_permission_desc"
android:textAlignment="center"
android:textColor="@android:color/white"
android:textSize="@dimen/slide_desc" />
</ScrollView>
</LinearLayout>

View File

@ -10,5 +10,7 @@
<dimen name="slide_title">30sp</dimen>
<dimen name="slide_desc">16sp</dimen>
<dimen name="desc_padding">40dp</dimen>
<dimen name="desc_marginTop">20dp</dimen>
<dimen name="desc_paddingBottom">80dp</dimen>
<dimen name="button_height">44dp</dimen>
</resources>

View File

@ -23,8 +23,8 @@ Please report any problems you encounter via Github.</string>
<string name="storage_permission_desc">Syncthing needs to access your storage to do file synchronization.</string>
<!-- Slide 3 -->
<string name="location_permission_title">Location Permission</string>
<string name="location_permission_desc">Syncthing can be configured to run on selected Wi-Fi networks. Android requires applications to have location permissions to be able look up active Wi-Fi network name, as you can sometimes infer users location from the name of the network they are connected to. If you want to use this feature, press the button above to give the required location permissions to Syncthing. Otherwise you can skip this step.</string>
<string name="location_permission_title">Location Permission (background)</string>
<string name="location_permission_desc">Syncthing can be configured to synchronize on selected Wi-Fi networks only. To do that it needs to look up the SSID of the currently connected Wi-Fi, also when the app is closed. Therefore it requires permission to access the location in the background. This information is not used to look up your location. And the only stored information is the Wi-Fi you manually choose to add to the list - no other user data is stored.\n\n If you want to use this feature, press the button above to grant the required permissions to Syncthing. Otherwise you can skip this step.</string>
<!-- Generic texts used everywhere -->
<string name="back">Back</string>
@ -554,6 +554,10 @@ Please report any problems you encounter via Github.</string>
<string name="donate_url" translatable="false">https://syncthing.net/donations/</string>
<!-- Menu item linking privacy policy -->
<string name="privacy_title">Privacy Policy</string>
<string name="privacy_summary">Open the Syncthing-Android privacy policy</string>
<!-- Title of the preference showing upstream version name -->
<string name="syncthing_version_title">Syncthing Version</string>

View File

@ -303,6 +303,14 @@
android:data="@string/donate_url" />
</Preference>
<Preference
android:title="@string/privacy_title"
android:summary="@string/privacy_summary">
<intent
android:action="android.intent.action.VIEW"
android:data="https://github.com/syncthing/syncthing-android/blob/main/privacy-policy.md" />
</Preference>
<Preference
android:key="syncthing_version"
android:title="@string/syncthing_version_title" />

View File

@ -1,6 +1,6 @@
# Privacy Policy
The app syncthing-android does not collect any user information. It does run Syncthing, which does expose some networking information to work and may collect usage-data if you agree to it or run betas/release candidates. This documentation article has specific information on this: https://docs.syncthing.net/users/security.html
The app syncthing-android does not collect any user information. It does run Syncthing, which does expose some networking information to work and may collect usage-data only if you agree to it or run betas/release candidates. This documentation article has specific information on this: https://docs.syncthing.net/users/security.html
The app requires the permission to access the location in the background, but it never does look up your location. The location permission is required to look up wifi SSIDs, which is used to enable/disable syncing on user configurable wifis.