Skip to content

fix(graphql): prevent user enumeration via error messages [M12]#565

Merged
lakhansamani merged 2 commits intomainfrom
fix/m12-user-enumeration
Apr 4, 2026
Merged

fix(graphql): prevent user enumeration via error messages [M12]#565
lakhansamani merged 2 commits intomainfrom
fix/m12-user-enumeration

Conversation

@lakhansamani
Copy link
Copy Markdown
Contributor

Summary

  • M12 (Medium): Different error messages for signup/login revealed whether accounts exist
  • Signup: now returns "signup failed. please check your credentials or try a different method"
  • Login: "user not found" and "user access has been revoked" → "invalid credentials"

Test plan

  • Package compiles
  • Verify signup/login errors are now generic

Signup and login returned different error messages depending on whether
a user/email/phone existed, was verified, or was revoked. Attackers
could enumerate valid accounts by observing these differences.

Now returns generic messages that don't reveal account state.

Fixes: M12 (Medium)
@lakhansamani lakhansamani merged commit 745c0c9 into main Apr 4, 2026
@lakhansamani lakhansamani deleted the fix/m12-user-enumeration branch April 4, 2026 11:59
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.

1 participant