-
-
Notifications
You must be signed in to change notification settings - Fork 276
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
Autotype - Paypal #1920
Comments
Not all two-screen forms work because the Autofill algorithm requires a password field to function. This will be solved when the algorithm takes into account the context of the page according to its URL, probably in a library. |
This totally works for me in Firefox based browsers (Mull, Fennec, Firefox Beta). There's occasionally breakage with login forms forcing you to type in each TOTP digit separatedly but that's independent from KeePassDX and more depending on the browser and its settings (desktop mode, fingerprint protection, extensions etc.). |
Actually, since updating to 4.1.1 it doesn't work anymore. I didn't have this problem with 4.1.0 or 4.0.8. |
Autofill at general does not work quite often. Is this a known issue? |
The recognition algorithm hasn't been changed in 4.1.1, I've just tested it and it works well for forms with native browsers but not for browsers with compatibility, like before for paypal. |
And I repeat, for forms that require two screens, one for the username/email and the other for the passords, autofill will only work for the password screen because the algorithm doesn't know in advance that the second screen will contain a password. |
URL matching works well for me, what URL do you have for Paypal in your entry? |
I am using "https://www.paypal.com/signin" Above the numbers on the keyboard will be the button for keepass. (Though other keyboards have the same issue with different methods like the popup) I am using fennec 132.0.0 currently |
The same URL that is mentioned in the issue description: https://www.paypal.com/signin As I wrote in #1820 the algorithm now only seems to work if the path is absent from the URL. With just https://www.paypal.com it should work. |
Though that issue seems to be about post-keyboard-popup detection. The problem i have is specificly that the popup doesn't show in the first place, so that subdomain matching is not relevant (or available, since the db is still locked) I am honestly not even sure if this is a bug in keepassdx at all. It might be an issue somewhere else, because after @J-Jamet overhauled the autotype a couple of months ago, nearly all login types work for me again. It is just specifically paypal (and my own unraid server) that dont work. Though sadly i don't really know enough about the autotype-service to start debugging this on .y own. |
@newhinton What keyboard do you use? I've been using Heliboard. At least in the past, the keyboard or the settings of it seemed to make a different because the suggestion strip seemed to overwrite the KeePassDX button and make it disappear. |
At the moment i am using https://github.com/florisboard/florisboard. However, i only recently started using it. Before i used the grapheneos-default keyboard (which is probably very close to the aosp-default as you can get) and it had the same issue. (that one does not have a "key" for the passwords, but a popup-overlay directly in the content) |
Describe the bug
Sorry to bother you again. It seems that specifically paypal does not work with keepass dx. I think it never worked since the main issue with autotype was fixed.
To Reproduce
Steps to reproduce the behavior:
Alternatively, the checkout process also does not work.
Expected behavior
The popup shows up.
KeePass Database
content://
URI):KeePassDX:
Android:
Additional context
I am not sure if this is even a bug. Recently i can't even paste the totp-code into its field from aegis via clipboard, i wouldn't be surprised if they actively tried to break autofill. Though i can't imagine why. Oh, and the used browser is firefox.
Thank you for looking into it if you so choose!
The text was updated successfully, but these errors were encountered: