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

Protect settings with password #2646

Closed
eike-fokken opened this issue Jan 24, 2019 · 5 comments
Closed

Protect settings with password #2646

eike-fokken opened this issue Jan 24, 2019 · 5 comments

Comments

@eike-fokken
Copy link

eike-fokken commented Jan 24, 2019

Summary

An attacker that has brief access to my computer with unlocked screen but locked database could go to settings and disable the automatic locking of the database, hoping that I later open the database, thinking it will lock itself afterwards, and briefly leave my computer.

Desired Behavior

It is not easily possible to change program settings.

Possible Solution

Add a password before the settings.

Context

Its just paranoia.

@droidmonkey
Copy link
Member

The only way this would work is if we encrypted the settings file itself and you had to enter a password just to even open KeePassXC. Given the fact that you gave access to your computer (willingly or not) to someone else is the problem here.

@eike-fokken
Copy link
Author

I agree that it is a bad idea to do that. On the other hand I have the impression that if I have security settings I should be able to rely on them.

What do you think about providing a (possibly red) warning stating that settings have changed when opening a safe the first time after changing settings?
This is probably also hard to implement rigorously, as the information when a safe was opened the last time, would be equally easily forged.

@droidmonkey
Copy link
Member

We have tossed around the idea of database specific security settings (ie they are stored in the encrypted database). Also we have an issue to implement enterprise settings via registry key or specific ini file.

@eike-fokken
Copy link
Author

I like this idea of settings stored in the database very much.

@droidmonkey
Copy link
Member

See #891 and #2189

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