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

[SECURITY] Email enumeration #21

Open
topsecret-chat-admin opened this issue May 18, 2023 · 1 comment
Open

[SECURITY] Email enumeration #21

topsecret-chat-admin opened this issue May 18, 2023 · 1 comment
Assignees
Labels
bug Something isn't working fixed The issue has been resolved and ready for testing released The issue has been released tested The issue has passed testing and ready to realese
Milestone

Comments

@topsecret-chat-admin
Copy link
Contributor

Notice

The identification and proposed resolution of this issue has been kindly provided by Kunal Mhaske and this ticket has been logged on his behalf.

Vulnerability Name

Email enumeration at the SignUp page

Vulnerable URL

https://topsecret.chat/app/signup/signup.html

Description

At the signup page a msg box says The email xxx@yyy.zzz is already in use.
The problem is that any used email gets the same error message. while when you enter any other e-mail regardless of whether it is fake or not & valid or not it gets accepted. which means any Email (could be fake) is valid except registered emails in the database. So an attacker can compare both responses (success & failure) and enumerate users' emails on a large scale.

Impact

Brute force attack at large scale may disclose users' email.

Mitigation

A better security practice is simply saying that you sent a link to the e-mail no matter if they have an account already or not. If they have already registered and another process is done, the Email message must say that someone tried to signup with that Email address, if that is you please log in

@topsecret-chat-admin topsecret-chat-admin added the bug Something isn't working label May 18, 2023
@topsecret-chat-admin topsecret-chat-admin added this to the 1.2.8 milestone May 18, 2023
@topsecret-chat-admin topsecret-chat-admin self-assigned this May 18, 2023
@topsecret-chat-admin topsecret-chat-admin added tested The issue has passed testing and ready to realese fixed The issue has been resolved and ready for testing released The issue has been released labels May 18, 2023
@topsecret-chat-admin
Copy link
Contributor Author

This issue has been resolved.
The resolution has also been extended to when a user attempts to change their email address, under their Profile >> Account page.

See below an example of the email received by the user when someone attempts registering using their email. A special THANKS again to Kunal Mhaske for the contribution and collaboration.

tempFileForShare_20230518-151256

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed The issue has been resolved and ready for testing released The issue has been released tested The issue has passed testing and ready to realese
Development

No branches or pull requests

1 participant