search

keepassxreboot / keepassxc-browser

KeePassXC Browser Extension
GNU General Public License v3.0
1.77k stars 188 forks source link

Username / password field not detected in chrome: https://b2b.dab-bank.de/smartbroker/ #1116

Closed foben closed 3 years ago

foben commented 3 years ago

Expected Behavior

That username/password fields are auto-detected, or can be selected manually.

Current Behavior

Can't even select as custom username/password field. It works in Firefox but not in Chrome.

Steps to Reproduce (for bugs)

  1. Go to https://b2b.dab-bank.de/smartbroker/
  2. In the extension, click "custom username/password field"
  3. None of the input fields will be selectable

Debug info

KeePassXC - 2.6.2 KeePassXC-Browser - {VERSION} Operating system: Win & Linux Browser: Chrome Version 86.0.4240.198

varjolintu commented 3 years ago

For some reason getBoundingClientRect() for the form element returns its height as 0.

KuenzelIT commented 3 years ago

I have the exact same problem with Firefox. Can I somehow define the input fields in the config?

varjolintu commented 3 years ago

@KuenzelIT Seems that's not working either. The plugin thinks those input fields have no size, so they are ignored.

KuenzelIT commented 3 years ago

Well, smartbroker does not even call it's password field a password field, but an "identifier". So I assume it's not surprising if they do some crazy JavaScript magic. Still pretty annoying that every other website works perfectly, but not smartbroker.

varjolintu commented 3 years ago

@KuenzelIT The only way to get this work for now is to force fill through the context menu, but then you'll have to fill username and password separately. Another way is to use KeePassXC's Global Auto-Type.

KuenzelIT commented 3 years ago

@KuenzelIT The only way to get this work for now is to force fill through the context menu, but then you'll have to fill username and password separately. Another way is to use KeePassXC's Global Auto-Type.

Thanks, I'm using the force fill for now. Global Auto Type sounds a little scary to me.

varjolintu commented 3 years ago

@KuenzelIT The only way to get this work for now is to force fill through the context menu, but then you'll have to fill username and password separately. Another way is to use KeePassXC's Global Auto-Type.

Thanks, I'm using the force fill for now. Global Auto Type sounds a little scary to me.

It just means launching the Auto-Type using a global keyboard shortcut. Makes sometimes things much easier.

KuenzelIT commented 3 years ago

@KuenzelIT The only way to get this work for now is to force fill through the context menu, but then you'll have to fill username and password separately. Another way is to use KeePassXC's Global Auto-Type.

Thanks, I'm using the force fill for now. Global Auto Type sounds a little scary to me.

It just means launching the Auto-Type using a global keyboard shortcut. Makes sometimes things much easier.

Thanks a lot, this is very useful. Now I unterstand what you meant, it works only for one field, but is a lot quicker than using the context menu.

varjolintu commented 3 years ago

This is now tracked in #1358.