-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
Since 0.24 when using Kee for autofill the {KPOTP} is not replaced #46
Comments
I'm not that familiar with Kee. Did you do anything else beside updating KeePassOTP? I'm mainly confused because I understand that {KPOTP} is printed "as-is", it should either be replaced by the current OTP or by an error message in case {KPOTP} is an unknown placeholder |
I only updated KeePassOTP Kee is here: https://github.com/kee-org/browser-addon (for firefox it is on mozilla store - https://addons.mozilla.org/en-US/firefox/addon/keefox/ ) Where would I be able to see this error message you mentioned? I can check and ev. send you a patch if I see what exactly failed. |
(how Kee is used is explained here: https://www.ghacks.net/2020/01/08/kee-is-a-firefox-and-chrome-extension-that-can-auto-fill-passwords-from-keepass/ ) and thnx for looking into this @Rookiestyle |
Version 0.25.2 does contain the required fix |
<3 thanks, confirmed & up and running now! :-) |
Hello
thanks for great plugin!
I use it with latest Keepass, Firefox and Kee plugin for autofill
However last releases borked Kee a bit ...
Last working version was 0.23 (so I stick to it for now)
From 0.24 what used to work with Kee, now just fills {KPOTP} instead of time OTP
if I replace it with {TimeOTP} it gets filled, but with wrong entry (last time I tried with 0.25, didn't test 0.25.1 with this approach, I saw you did some changes in counting, but not sure related to which fields)
From my POV this is a regression, however if KPOTP shouldn't be used anymore as field for fill in with Kee, I'll gladly fix my entries
Can you let me know what is supposed to work and what doesn't now?
tia
L
The text was updated successfully, but these errors were encountered: