mvdan / bitw

Minimalist BitWarden client
BSD 3-Clause "New" or "Revised" License
169 stars 15 forks source link

Fix broken login due to webauthn #42

Closed quexten closed 1 year ago

quexten commented 1 year ago

Having webauthn (method 7) as one of the 2fa methods on your account prevents login, since while parsing, this line:

if err != nil || i < 0 || i >= _TwoFactorProviderMax { ensures that the methods are all below 7. Even if you have other 2fa methods configured, you do not get to choose them.

This commit does not implement webauthn login, but does fix broken login when webauthn is one of the methods on the account, by letting you choose the other methods.

This partially fixes #38