keepmenu icon indicating copy to clipboard operation
keepmenu copied to clipboard

Hide search-disabled groups

Open avidseeker opened this issue 1 year ago • 7 comments

config.ini includes hide_groups to exclude passwords from showing up the menu. However, KeePassXC already provides this functionality in group properties: right click > group > search: Enabled/Disabled/Inherited from parent group.

avidseeker avatar Mar 16 '24 20:03 avidseeker

Ok, I see that KeepassXC has the ability to hide a group from search. I don't understand how that's relevant to Keepmenu?

firecat53 avatar Mar 18 '24 18:03 firecat53

Maybe the ask here is to move this to entry properties and control within keepmenu instead of using custom ini kv?

vaygr avatar Mar 18 '24 19:03 vaygr

Ok, I see that KeepassXC has the ability to hide a group from search. I don't understand how that's relevant to Keepmenu?

It is relevant because keepmenu already implements it.

I'm saying that the hide_groups feature in keepmenu is re-implementing something KeePassXC already has which is "exclude from search". Why not read this property directly from kdbx? especially that it is better with hierarchical groups (see the "inherited from parent group" option).

avidseeker avatar Mar 18 '24 19:03 avidseeker

Pykeepass doesn't implement that feature. Is it actually a kbdx feature or just a KeepassXC feature?

Edit: Here is the XML spec for kbdx. I didn't see anything relevant to group visibility during search, but it's not exactly easy to read 😁

firecat53 avatar Mar 18 '24 19:03 firecat53

I'm going to close this, as it doesn't seem the requested feature is part of the kdbx spec. If I'm wrong, please re-open and I'll take a look. Thanks!

firecat53 avatar Mar 28 '24 00:03 firecat53

I think it's

<xs:element name="EnableSearching" type="TNullableBoolEx" minOccurs="0" />

If I wasn't mistaken.

avidseeker avatar Mar 28 '24 02:03 avidseeker

Ah, I see now. Thanks for that! Re-opening.

Here's the actual spec definition for EnableSearching.

firecat53 avatar Mar 28 '24 16:03 firecat53