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

TOTP support #6

Open
kresbeatz opened this issue Nov 16, 2020 · 3 comments
Open

TOTP support #6

kresbeatz opened this issue Nov 16, 2020 · 3 comments

Comments

@kresbeatz
Copy link

First of all, thank you for the script, I just started to use it today and it works great. However I'd like to ask, is it possible to add keybinding for paste TOTP key additionally to username and password? Thanks in advance!

@ususdei
Copy link
Owner

ususdei commented Jan 5, 2021

Thanks :-)

I first had to read up on the relatively new API in keepassxc for TOTP, but I think it should be doable.

I am just not sure about how to identify the TOTP input field.

  • keepassxc-browser has some heuristics for that, I don't how reliable that one is already.
  • We could pick the currently focused input, this might not be the best UX but we are not to blame if things end up in the wrong field then :-D
  • I do not want to store some additional persistent data or configuration for that...

I'd like try out the new API soonish and see what works for me.

@Quarky9
Copy link

Quarky9 commented Mar 17, 2021

First of all thx for the amazing script !
Apart from TOTP there are other custom fields see keepassxreboot/keepassxc-browser#122
Might be the same mechanism starting with KPH: or TOTP ?

@ususdei
Copy link
Owner

ususdei commented Nov 27, 2021

I finally push my working draft for this feature.
I still think the UX is rather primitive but it is a very generic solution which should work on most pages.
I'm looking forward to gather some feedback and more experience on this.

Regarding the KPH fields:
I have not had any need for them but upon first glance it might be doable as long as you setup the fields manually somehow.
But since the approach would be rather different than TOTP which has its own API in the browser integration, I would like to move this into a separate Item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants