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

Webauthn - prevent email leak #290

Open
jay-dee7 opened this issue Mar 12, 2023 · 0 comments
Open

Webauthn - prevent email leak #290

jay-dee7 opened this issue Mar 12, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@jay-dee7
Copy link
Member

Is your feature request related to a problem? Please describe.
Reference - https://www.w3.org/TR/webauthn-2/#sctn-username-enumeration

The Above link mentions about a possible email leak, if we use email as a field for user identification. Even though, in case of Webauthn, the identifier is username but we also accept an email as the the required parameter and fail when that email already exists.
An attacker can attempt to signup with an email that doesn't exist and successfully signup but if the email already exists, we'll prompt the user with an error saying email already exists. This indicates that the user already exists and the particular email has an account with the relaying party (OpenRegistry in our case).

Describe the solution you'd like
The solution is to interrupt the signup flow, at BeginRegistration step, we send an email with an OTP to the provided email and ask the user to enter the OTP. If the OTP is valid, then we proceed with the signup flow.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@jay-dee7 jay-dee7 added the enhancement New feature or request label Mar 12, 2023
@jay-dee7 jay-dee7 self-assigned this Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant