-
Notifications
You must be signed in to change notification settings - Fork 57
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
Change to EncryptedSharedPreferences for Android? #13
Comments
There is no any specific reason. As is stated in README, code for android is based of https://www.apriorit.com/dev-blog/432-using-androidkeystore |
True, the migration piece probably doesn't make this worth the effort. I appreciate the response. I might take a stab and creating a similar plugin as I need to learn this stuff anyway. |
Hey @martinkasa , I'd like to reopen this discussion and take a stab at delivering this feature. I see two main advantages. The first being it would greatly simplify the encryption process for Android and use the standard encryption methods/practices provided by Android. The second and primary driver is that it would also allow for much easier access to the storage if you need to access values from a background process. For example, 90% of the app might be an Ionic/Capacitor app, but perhaps there are some background processes that need to get an acessToken/refreshToken for to sync background data with an external API. Implementation seems pretty straightforward but curious your thoughts on the best way to run a migration? Or perhaps not run a migration and have a flag to determine what time of encryption process is used. This way the app could decide and control if/when it wants to migrate the data and be in full control of that process. |
I would say on data access perform the migration, if you |
Disclaimer, I'm fairly new to Android and Capacitor development.
For Android, is there a reason EncryptedSharedPreferences aren't used? It seems like this would greatly simplify the plugin and essentially remove the need for the entire
PasswordStorageHelper.java
class.The text was updated successfully, but these errors were encountered: