# Kee browser extension bug - using form.name rather than form.getAttribute('name') **URL:** \ **Category:** Kee browser extension\ **Tags:** debug\ **Created:** [February 3, 2021, 8:58pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503 "2021-02-03T20:58:41Z")\ **Posts on this page:** 7\ **Page:** 1
**Author:** ![burtek](https://forum.kee.pm/user_avatar/forum.kee.pm/burtek/32/1249_2.png) [@burtek](https://forum.kee.pm/u/burtek)\ **Post date:** [February 3, 2021, 8:58pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/1 "2021-02-03T20:58:41Z")
I have a entry in KeePass for website [https://login.wyborcza.pl/](https://login.wyborcza.pl/). Recently I tried logging in to that website, but the login and password were not auto-filled. I have never had such an issue with any other website. I made sure that the entry is in subgroup of Kee Home Group, the auto-type option is enabled, “Hide this entry from Kee” is disabled, that the URL matches exactly the login form URL. I even tried setting Form Fields in KeePass’ entry Kee settings as well as removing and creating the entry back from the ground - nothing works, apart from searching for the website manually in extension popup, and manually copying and pasting username and password ---
**Author:** ![burtek](https://forum.kee.pm/user_avatar/forum.kee.pm/burtek/32/1249_2.png) [@burtek](https://forum.kee.pm/u/burtek)\ **Post date:** [February 3, 2021, 9:09pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/2 "2021-02-03T21:09:42Z")
The error from console: ![image](https://forum.kee.pm/uploads/default/original/2X/7/7bd919162b98d193b4345a38eee312b1614c0fc4.png) ---
**Author:** ![burtek](https://forum.kee.pm/user_avatar/forum.kee.pm/burtek/32/1249_2.png) [@burtek](https://forum.kee.pm/u/burtek)\ **Post date:** [February 3, 2021, 9:10pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/3 "2021-02-03T21:10:47Z")
Just found out the issue comes from this particular website having input field’s name set to “name”, which in turn makes `form.name` not be a string name attribute of `
` element, but rather the `` with `name="name"` in this form: ![image](https://forum.kee.pm/uploads/default/original/2X/1/1d571e5b889f28754f82b1eee0092a218a28429d.png) Finally it results in `.toLowerCase()` being called on the `` element rather than string value. Solution: use `form.getAttribute('name')` rather than `form.name` (same for `form.id` and similar code parts) ---
**Author:** ![luckyrat](https://forum.kee.pm/user_avatar/forum.kee.pm/luckyrat/32/10_2.png) [@luckyrat](https://forum.kee.pm/u/luckyrat)\ **Post date:** [February 4, 2021, 4:15pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/4 "2021-02-04T16:15:47Z")
Thanks for the bug report. It looks like the root cause has existed for many years at least but the change I made to that `isFormInteresting` function to enable case-insensitive matching a few months ago is now highlighting the flaw much more clearly in the console and breaking things. I didn’t know form elements were permitted to be named in a way that breaks the properties on the form object and it’s a tough thing to search internet history for but it looks like it has been the case for at least a few years (or at least, that is when MDN first highlighted the problem). When you say “similar code parts” did you have something specific in mind? I can’t find anything extra to be concerned about beyond the items you included in your PR (thanks!) ---
**Author:** ![burtek](https://forum.kee.pm/user_avatar/forum.kee.pm/burtek/32/1249_2.png) [@burtek](https://forum.kee.pm/u/burtek)\ **Post date:** [February 4, 2021, 5:20pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/5 "2021-02-04T17:20:01Z")
Nothing specific, I’m just not familiar with this codebase and don’t know if there are any other references to `form.id`, `form.name` and other properties of `form`. Fixed the PR btw 😉 ---
**Author:** ![luckyrat](https://forum.kee.pm/user_avatar/forum.kee.pm/luckyrat/32/10_2.png) [@luckyrat](https://forum.kee.pm/u/luckyrat)\ **Post date:** [February 4, 2021, 5:31pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/6 "2021-02-04T17:31:29Z")
The only other reference to `form.` that I can find is `form.elements` - we use that to find all the elements in the form. I’m not sure if there is any alternative approach so if someone wants to define a field with `name=elements` in their web page that is going to break everything. On reflection, I can at least limit the damage from that eventuality by checking that the property is actually a list of elements and ignore the form if it is anything unexpected like a form field. Perhaps I can even use the `form` property on each form field in the DOM to build up the list of elements “backwards”. That might be tough to achieve without hitting performance problems on pages with many elements though. ---
**Author:** ![burtek](https://forum.kee.pm/user_avatar/forum.kee.pm/burtek/32/1249_2.png) [@burtek](https://forum.kee.pm/u/burtek)\ **Post date:** [February 4, 2021, 5:53pm UTC](https://forum.kee.pm/t/kee-browser-extension-bug-using-form-name-rather-than-form-getattribute-name/3503/7 "2021-02-04T17:53:50Z")
I mean, “id” and “name” make sense for username login field, I’d imagine “elements” to be much less frequent element name. That being said, it would probably be a good idea to implement some kind of fail-safe logic here, too.