digidem / comapeo-mobile

The next version of Mapeo mobile
GNU General Public License v3.0
5 stars 1 forks source link

fix: disable android:allowBackup manifest option #393

Closed achou11 closed 4 months ago

achou11 commented 4 months ago

Addresses the SecureStore error I keep seeing in #370 . This PR disables Android's Auto Backup feature which is basically a mechanism that allows you to store your app data via a Google Drive account so that you can restore apps and their data later or on another device. Expo enables this by default: https://docs.expo.dev/versions/v50.0.0/config/app/#allowbackup

I don't have the feature enabled on my devices, but seems like disabling the manifest value fixes the issue I was seeing with SecureStore being unable to decrypt the vault everytime I loaded a new dev client, so maybe there's some underlying thing that's not playing well with the module.

In general, I don't think it's desirable for us to have the app enabled for this feature. I've run into really confusing complications with Manyverse in the past because of it and in general, seems to not play well with apps that use local databases.