ususdei / qute-keepassxc

Qutebrowser userscript to fetch credentials from KeepassXC password database
MIT License
54 stars 10 forks source link

Add support to handle multiple accounts #11

Closed nivit closed 2 years ago

nivit commented 2 years ago

The script now allows a user to choose a username from a rofi menu, when a site as multiple accounts available.

ususdei commented 2 years ago

Nice approach! I like rofi a lot. I hope you don't mind me tweaking the code just a little :-)

nivit commented 2 years ago

I hope you don't mind me tweaking the code just a little :-)

No, I don't. You're welcome! :-)

nivit commented 2 years ago

Only a little note: if you press ESC to exit the rofi menu without selecting an account, you got:

name 'e' is not defined

because the output from rofi is void and a ValueError exception is raised. Maybe we should handle it directly with

except subprocess.CalledProcessError as e:

?