Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disabled backup allowed #537

Conversation

prateek-kommunicate
Copy link
Contributor

Removed allowed backup. As suggested in Android VPAT report. We are not using this feature.

@prateek-kommunicate prateek-kommunicate changed the base branch from master to development August 20, 2024 13:06
@@ -37,7 +37,6 @@

<application
android:name="kommunicate.io.sample.KommunicateApplication"
android:allowBackup="true"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prateek-kommunicate Default value looks true, removing this should not help. Please make it false

https://developer.android.com/guide/topics/manifest/application-element#allowbackup

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh I see. I just checked that out more. Above Android 12 allowBackup false wont stop the device to device data transfers. So for that we have to add the data_extraction_rules which disables all the device to device transfers and cloud backups.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, as discussed please test it with the Sample SDK with reverse configurations.

@prateek-kommunicate prateek-kommunicate merged commit 5926f44 into development Sep 9, 2024
prateek-kommunicate added a commit that referenced this pull request Sep 24, 2024
* Disabled backup allowed

* Marked allow backup to false

---------

Co-authored-by: Prateek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants