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

Lock databases on removable storage when it is removed #11361

Open
DanEble opened this issue Oct 14, 2024 · 1 comment
Open

Lock databases on removable storage when it is removed #11361

DanEble opened this issue Oct 14, 2024 · 1 comment

Comments

@DanEble
Copy link

DanEble commented Oct 14, 2024

Summary

For an open database that was loaded from a removable storage device, removing the storage device signals intent to make the secret information unavailable.

Examples

If the application can accurately determine when to apply this, it should be an application setting.

☑︎ Enable database quick unlock (Touch ID / Windows Hello)
☑︎ Lock databases when session is locked or lid is closed
☑︎ Lock databases after minimizing the window
☑︎ Lock databases on removable storage when it is removed   <---
☑︎ ...

If there are look-alike scenarios in which this might be inconvenient, making it a database setting would probably be better.

Context

I keep a password database on a thumb drive.

@droidmonkey
Copy link
Member

droidmonkey commented Oct 14, 2024

I'm conflicted on this request. I don't think there is a good "cross-platform" way to know a file is on a removable storage device (as opposed to the network share for example). We would also need some form of a trigger for the lock event, which would likely mean you would have to periodically observe read/write access to the file. I could definitely see this as a potential cause for bugs (if enabled) due to unforeseen locks happening, we already have a hard enough time with lid close / session lock.

This could be a decent per-database setting that locks the database if the underlying file is no longer accessible. It could also be implemented as a prompt/warning to the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants