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

Show a warning (graphical) if someone changes "Settings" when all the vaults are locked #574

Open
2 tasks done
serrq opened this issue Feb 15, 2025 · 5 comments
Open
2 tasks done
Labels
type:feature-request New feature or request

Comments

@serrq
Copy link

serrq commented Feb 15, 2025

Please agree to the following

Summary

Security (potential risk)

Motivation

I noticed that it is possible changing some app settings without a password.

So when a user, temporarily lose the device control, anyone can changes app behaviors without user awareness.

Put a graphical warning in the home directory (when settings change) resettable only after provided a master password or a local pin.

Considered Alternatives

No response

Anything else?

No response

@serrq serrq added the type:feature-request New feature or request label Feb 15, 2025
@serrq serrq changed the title Show a warning (graphical) if someone changes "Settings" without a database unlock Show a warning (graphical) if someone changes "Settings" when all the databases are locked Feb 15, 2025
@serrq serrq changed the title Show a warning (graphical) if someone changes "Settings" when all the databases are locked Show a warning (graphical) if someone changes "Settings" when all the vaults are locked Feb 15, 2025
@serrq
Copy link
Author

serrq commented Feb 16, 2025

For the same reason: "add a new vault" (creating) should be possible only when at least a pre-existant vault is opened.

An app (local) pin (six digits) can solve these issues effectively.

@serrq
Copy link
Author

serrq commented Feb 16, 2025

An app (local) pin also prevent that unauthorized actors can deleting a vault.

But it remains always possible to do it via file manager, so this poses a serious reasoning on security.

Are administrator privileges necessary to prevent unauthorized actors from erasing a vault?

@SailReal
Copy link
Member

#13 is not exactly the same but I think this could be an option that the complete app is locked with a pin and only when entered, it can be accessed.

Are administrator privileges necessary to prevent unauthorized actors from erasing a vault?

Someone with access e.g. to your (cloud) files can delete the vault at anytime. On the device itself you need to go through the UI of the app or need root access to make changes in the database but also have a look at our security-target.

@serrq
Copy link
Author

serrq commented Feb 17, 2025

Device side: With "you" you mean any entity with capability (fingers, human) to touch, physically, the screen UI?

@SailReal
Copy link
Member

Device side: With "you" you mean any entity with capability (fingers, human) to touch, physically, the screen UI?

Yes, the screen UI of the Cryptomator Android app.

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

No branches or pull requests

2 participants