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.

Sign up for GitHub

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

Jump to bottom

Root group protection #126

Open
vaygr opened this issue Sep 4, 2022 · 1 comment
Open

Root group protection #126

vaygr opened this issue Sep 4, 2022 · 1 comment

Comments

@vaygr
Copy link
Contributor

vaygr commented Sep 4, 2022

Thanks for this nice little piece of software!

I've been playing around with keepmenu lately trying to break it, and noticed an issue. Not sure if it's by design.

It seems it's easy to wipe out the root group in keepmenu. The database then becomes effectively useless: I can't add the root group back because adding any group requires specifying parent group, which doesn't exist in the first place. When I then try to open the database in KeePassXC I get an error Error while reading the database: No root group.

In KeePassXC the root group is protected, so maybe it makes sense to do the same in keepmenu?
@vaygr vaygr changed the title Protect root group Root group protection Sep 4, 2022
@firecat53
Copy link
Owner

Yep, I can reproduce. Makes sense to not allow the root group to be deleted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@firecat53 @vaygr