-
-
Notifications
You must be signed in to change notification settings - Fork 278
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
Filter Autofill search results #519
Comments
Normally, the autofill search parser looks at all the fields, so it should work. |
Ooops, yes 2.5RC1, sorry for not stating that. Lineage 16, OnePlus 5. Ah, I didn't know about the entry limit of 6. Then my issue is something different: KeePassDX matches parts of domains in the "user" field. This leads to entries being suggested where they needn't be. Example database: keepassdx-test.zip Screenshots: This is with KeePassDX 2.5RC1 on an OnePlus 7, OxygenOS 10.0.4 in Firefox Preview 4.2.1 To reproduce:
I see that this behaviour can be useful if the user hasn't set the URL field. |
Dunno if I should rename this issue and continue discussion here, or open a new one. You decide :) |
Thank you for your detailed feedback. We can continue here it is not annoying.
It is also possible to add both options. I think it is best to implement the 1.1 solution and add the second solution in a second step. |
1.1)) I agree that ignoring the Leaving the URL field empty and relying on the Given that the Whether by setting or not: I can't think of any argument for or against specific to this case. Generally I tend to value sensible defaults over a plethora of settings, but I know that many people in the FOSS world like to be able to customise everything. (tldr: no setting) 1.2)) Manually whitelisting fields to search does indeed sound like quite a rabbit hole, I wouldn't go there. 2)) Would be good a catch-all solution for other future cases where the parsing doesn't work as expected. |
No, the So I'm going to develop the solution 1.1 |
Is your feature request related to a problem? Please describe.
KeePassXC allows having more than one web domain associated with a single entry. They do that by storing those additional URLs as custom attributes that have keys of this form:
KP2A_URL, KP2A_URL_1, KP2A_URL_2, ..., KP2A_URL_n
Describe the solution you'd like
Please consider trying to parse those additional attributes as well when resolving entries for a given domain.
Describe alternatives you've considered
Ignore them, as those attribute keys are not standardised (as far as I know).
Additional context
In my case: my university login can be used in tons of different places, so I have over eight URLs for that entry.
KeePassXC added this in v2.5.0 with this PR.
The text was updated successfully, but these errors were encountered: