Skip to content

Ratelimit the registration endpoint - #24664

Merged
uzairr merged 1 commit into
masterfrom
ratelimit-registration-api
Aug 21, 2020
Merged

Ratelimit the registration endpoint#24664
uzairr merged 1 commit into
masterfrom
ratelimit-registration-api

Conversation

@uzairr

@uzairr uzairr commented Aug 3, 2020

Copy link
Copy Markdown
Contributor

The registration pages display whether usernames and email addresses exist or have already been registered. An attacker can therefore guess or brute force a series of names to enumerate valid users.To overcome this, registration validation endpoints are rate-limited up to 30 requests/user in a week.

https://openedx.atlassian.net/browse/PROD-880

@uzairr
uzairr force-pushed the ratelimit-registration-api branch from d86f962 to 9ba6f27 Compare August 5, 2020 12:20
@uzairr

uzairr commented Aug 5, 2020

Copy link
Copy Markdown
Contributor Author

jenkins run python

@uzairr
uzairr force-pushed the ratelimit-registration-api branch 5 times, most recently from de399ab to dce6953 Compare August 7, 2020 11:27
@waheedahmed

Copy link
Copy Markdown
Contributor

jenkins run python

@waheedahmed
waheedahmed force-pushed the ratelimit-registration-api branch from dce6953 to 883d42f Compare August 19, 2020 11:43
Comment thread openedx/core/djangoapps/user_authn/views/tests/test_register.py Outdated
Comment thread cms/envs/common.py Outdated
Comment thread lms/envs/common.py Outdated
Comment thread openedx/core/djangoapps/user_authn/views/tests/test_register.py Outdated
@uzairr
uzairr force-pushed the ratelimit-registration-api branch from 883d42f to 4bd09a0 Compare August 20, 2020 12:49
@uzairr
uzairr force-pushed the ratelimit-registration-api branch from 4bd09a0 to 7bc17c7 Compare August 20, 2020 13:38
@uzairr
uzairr merged commit ea69e0d into master Aug 21, 2020
@uzairr
uzairr deleted the ratelimit-registration-api branch August 21, 2020 12:39
@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants