Skip to content

E
edx-platform
Commit 28575fb3 by Michael Frey Committed by GitHub
Options

Merge pull request #16137 from edx/mjfrey/enrollment-rate-limit 

increase staff user rate limit
parents 6fbea7f1 3e245726
Showing with 2 additions and 2 deletions
common/djangoapps/enrollment/tests/test_views.py
...@@ -575,7 +575,7 @@ class EnrollmentTest(EnrollmentTestMixin, ModuleStoreTestCase, APITestCase, Ente ...@@ -575,7 +575,7 @@ class EnrollmentTest(EnrollmentTestMixin, ModuleStoreTestCase, APITestCase, Ente
self.assert_enrollment_status(username=staff_user.username, expected_status=status.HTTP_200_OK) self.assert_enrollment_status(username=staff_user.username, expected_status=status.HTTP_200_OK)
# Once the limit is reached, subsequent requests should fail # Once the limit is reached, subsequent requests should fail
for attempt in xrange(rate_limit + 10): for attempt in xrange(rate_limit + 50):
self.assert_enrollment_status(username=staff_user.username, expected_status=status. HTTP_429_TOO_MANY_REQUESTS) self.assert_enrollment_status(username=staff_user.username, expected_status=status. HTTP_429_TOO_MANY_REQUESTS)
def test_enrollment_throttle_for_service(self): def test_enrollment_throttle_for_service(self):
......
common/djangoapps/enrollment/views.py
...@@ -75,7 +75,7 @@ class EnrollmentUserThrottle(UserRateThrottle, ApiKeyPermissionMixIn): ...@@ -75,7 +75,7 @@ class EnrollmentUserThrottle(UserRateThrottle, ApiKeyPermissionMixIn):
"""Limit the number of requests users can make to the enrollment API.""" """Limit the number of requests users can make to the enrollment API."""
THROTTLE_RATES = { THROTTLE_RATES = {
'user': '40/minute', 'user': '40/minute',
'staff': '400/minute', 'staff': '600/minute',
} }
def allow_request(self, request, view): def allow_request(self, request, view):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment