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

Exception trying to manually fill password to dynamic field #342

Open
Terrance opened this issue Nov 10, 2024 · 0 comments
Open

Exception trying to manually fill password to dynamic field #342

Terrance opened this issue Nov 10, 2024 · 0 comments

Comments

@Terrance
Copy link

Trying to fill a password field with an entry via the right-click menu or the extension popup UI throws:

TypeError: can't access property "map", entries is undefined index.global.js:17690:7

This appears to be happening at the filter chain inside sortMatchedEntries:

const orderedEntries = this.sortMatchedEntries(
matchResult.entries[matchResult.mostRelevantFormIndex]
);

sortMatchedEntries(entries: Entry[]): Entry[] {
return entries
.map(e => new Entry({ ...e }))
.sort((a, b) => b.relevanceScore - a.relevanceScore);
}

At the call site, matchResult.mostRelevantFormIndex is null and matchResult.entries is [[]], an array of an empty array.

For context, this is a site with separate email and password pages, a JavaScript history transition between the two (as opposed to the email form doing a HTTP submission), and a password field with a random ID each time (which prevents me from teaching Kee to fill it automatically).

As it stands, I have to manually copy the password from the popup UI or KeePass itself and paste it, which is a bit cumbersome.

Versions: KeePass 2.57.1, KeePassRPC 2.0.2.0, Firefox 133.0b6, Kee 4.0.7

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

1 participant