8000 fix(2fa): TOTP Verification by occorune · Pull Request #2620 · better-auth/better-auth · GitHub
[go: up one dir, main page]

Skip to content

fix(2fa): TOTP Verification #2620

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

occorune
Copy link
Contributor

The twoFactorEnabled boolean isn't enabled when enabling 2FA and you have to verify a code before confirming but you are unable to do this if twoFactorEnabled isn't set to true.

Copy link
vercel bot commented May 11, 2025

@occorune is attempting to deploy a commit to the better-auth Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
@mrge-io mrge-io bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrge reviewed 1 file and found no issues. Review PR in mrge.io.

Copy link
pkg-pr-new bot commented May 11, 2025

Open in StackBlitz

npm i https://pkg.pr.new/better-auth/better-auth@2620
npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/cli@2620
npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/expo@2620
npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/stripe@2620

commit: 7dd1542

@occorune occorune changed the title Fix TOTP get-totp-uri fix(2fa): TOTP Verification May 12, 2025
@occorune
Copy link
Contributor Author

Hoping this can get merged soon due to "severity" lol

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

Successfully merging this pull request may close these issues.

2 participants
0