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

[FEATURE REQUEST] Local data encryption #262

Open
domportera opened this issue Jul 17, 2024 · 5 comments
Open

[FEATURE REQUEST] Local data encryption #262

domportera opened this issue Jul 17, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@domportera
Copy link

Is your feature request related to a problem? Please describe.
Quite often these applications are used to scan highly sensitive documents (esp Card Wallet) - existing on-device unencrypted can be a security risk

Describe the solution you'd like
Local database encryption by default for scanned-but-not-exported documents/cards
option to export PDFs with password
the easiest development-wise might be to have the user be able to set a global password, stored locally (encrypted via fingerprint/android unlock) and have this global password apply to everything exported

Describe alternatives you've considered
An application to encrypt once exported, though no alternative solution can exist for the apps' internal scans outside

Additional context
I'm a big fan of OSS card wallet and my suggestion largely stems from that, as I am keeping these files for as long as my phone lives. I'd like it to be as secure as possible

I have not checked the source to see if you using file-based encryption already for application files, so ignore that part if you are!

Thanks!! Love your work

@domportera domportera added the enhancement New feature or request label Jul 17, 2024
@farfromrefug
Copy link
Collaborator

@domportera thanks for the feature request.
Right now the data is not encrypted and could be easily accessed. Files (images) are written on disk for optimized access and memory usage (no need to load full image in memory like i would have to do if the image was in the sqlite database).
Not sure how to solve this right now but it is a great feature idea.
I think if you choose in the settings to store data on the internal storage it would already be more secure as you would need to be root to access the data.

PS: the level of "security" right now is about the same as the photos in your gallery.

@domportera
Copy link
Author

yeah that makes sense. really I'd say the level of security is at least more secure than the gallery 😅 DCIM folder permissions etc etc. So this isn't urgent.

It doesnt have to be a database implementation, I just assumed that's how you organized the cards. if the android file encryption system I linked isn't quite straightforward to seamlessly implement I get that - trading off usability for this security is likely not worth it.

however exporting/importing with a PDF password should be more straightforward I think, and would cover a fair amount of security concerns by itself for most users

@farfromrefug
Copy link
Collaborator

@domportera Yes i Will look at pdf with a password. Will also sync about encryption

@fbetil
Copy link

fbetil commented Nov 12, 2024

Hi, i'm interested to with local data encryption on wallet because even if i activate fingerprint access, images are directly accessible on SD cards.
Thanks a lot

@farfromrefug
Copy link
Collaborator

@fbetil i really like the idea. However it will require quite a bit of work, and right now without more sponsoring i can't work on big features like this.
Will leave this open. Will see what i can do later on

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants