-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[PM-4108] Add "twoFa" to the TotpFieldNames array in autofill-constants.ts #6413
Conversation
add "twoFa" to the TotpFieldNames constant
Thank you for your contribution! We've added this to our internal Community PR board for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and looks good to me; thanks for the contribution!
Next step is having our QA team take a look.
Thanks again for your contribution @pejeio - the changes have passed QA and have been merged. |
"approvals_code", -- facebook.com "code", -- cash.app, docker.com, dropbox.com, evernote.com, lincolnfinancial.com, "otc", -- live.com "otc-code", -- evernote.com "otp-code", -- getpostman.com "otpcode", -- amazon.com "pin", -- linkedin.com "security_code", -- docusign.com "twofa", -- #6413 (but lowercased to match convention) "verificationCode", -- bestbuy.com Co-authored-by: Jonathan Prusik <jprusik@classynemesis.com>
…ts.ts (bitwarden#6413) * Update autofill-constants.ts add "twoFa" to the TotpFieldNames constant * add "twoFa" to the TotpFieldNames array in autofill-constants.ts
"approvals_code", -- facebook.com "code", -- cash.app, docker.com, dropbox.com, evernote.com, lincolnfinancial.com, "otc", -- live.com "otc-code", -- evernote.com "otp-code", -- getpostman.com "otpcode", -- amazon.com "pin", -- linkedin.com "security_code", -- docusign.com "twofa", -- bitwarden#6413 (but lowercased to match convention) "verificationCode", -- bestbuy.com Co-authored-by: Jonathan Prusik <jprusik@classynemesis.com>
…ts.ts (bitwarden#6413) * Update autofill-constants.ts add "twoFa" to the TotpFieldNames constant * add "twoFa" to the TotpFieldNames array in autofill-constants.ts
"approvals_code", -- facebook.com "code", -- cash.app, docker.com, dropbox.com, evernote.com, lincolnfinancial.com, "otc", -- live.com "otc-code", -- evernote.com "otp-code", -- getpostman.com "otpcode", -- amazon.com "pin", -- linkedin.com "security_code", -- docusign.com "twofa", -- bitwarden#6413 (but lowercased to match convention) "verificationCode", -- bestbuy.com Co-authored-by: Jonathan Prusik <jprusik@classynemesis.com>
Type of change
Objective
Improvement of TOTP field recognition. This change adds
"twoFa"
to theTotpFieldNames
array in apps/browser/src/autofill/services/autofill-constants.ts.Many websites now use
"twoFa"
as an ID for their TOTP text input fields. By including it in theTotpFieldNames
array, Bitwarden will be better equipped to identify and autofill these fields, enhancing the user experience for a wide range of websites.