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

Incomplete tags specification #6

Open
palant opened this issue Apr 8, 2023 · 1 comment
Open

Incomplete tags specification #6

palant opened this issue Apr 8, 2023 · 1 comment

Comments

@palant
Copy link

palant commented Apr 8, 2023

Section 3.4.2.1 currently has to say about tags:

A list of used-defined tags for this entry separated by semicolons ";".

From what I can see in the code, this isn’t entirely correct. While KeePass and keepass-rs in fact seem to store tags separated by semicolons, both will accept semicolons and commas as tag separators. And KeePassXC stores tags separated by commas, while accepting semicolons, colons and commas as tag separators when reading the database.

What should it say in the spec? Probably something like:

A list of user-defined tags. While an implementation SHOULD use semicolons ";" to separate tags, it is RECOMMENDED that it also accepts commas "," and colons ":" as tag separator.

Btw, KDBX 4.1 added group tags.

@phoerious
Copy link
Member

I think at this point any of those options MUST be accepted.

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

No branches or pull requests

2 participants