Description
Preconditions and environment
- 2.4.2+ most likely present in earlier versions as well.
Steps to reproduce
- Create a password request graphql query for an existing account
"mutation requestPasswordResetEmail() { requestPasswordResetEmail(email: 'existing@example.org')}
- Create a password request graphql query for a non-existing account
"mutation requestPasswordResetEmail() { requestPasswordResetEmail(email: 'nonexisting@example.org')}
Expected result
Both return a successful password reset message
{"data":{"requestPasswordResetEmail":true}}
Actual result
The existing one returns:
{"data":{"requestPasswordResetEmail":true}}
The non existing one returns:
{ "errors": [ { "message": "Cannot reset the customer's password", "extensions": { "category": "graphql-input" }, "locations": [ { "line": 2, "column": 3 } ], "path": [ "requestPasswordResetEmail" ], "code": 502 } ], "data": { "requestPasswordResetEmail": null } }
This means that the reset password graphql endpoint can be used to identify if accounts exist or not. Which is potentially harmful to the account owner.
Additional information
Preferably both calls should take the same amount of time to send a response. As more advanced bots will use the difference in response time to determine if an account exists or not.
Release note
No response
Triage and priority
- Severity: S0 - Affects critical data or functionality and leaves users without workaround.
- Severity: S1 - Affects critical data or functionality and forces users to employ a workaround.
- Severity: S2 - Affects non-critical data or functionality and forces users to employ a workaround.
- Severity: S3 - Affects non-critical data or functionality and does not force users to employ a workaround.
- Severity: S4 - Affects aesthetics, professional look and feel, “quality” or “usability”.
Metadata
Metadata
Assignees
Type
Projects
Status