Skip to content
Snippets Groups Projects
Commit 4059d61e authored by Andrew Morgan's avatar Andrew Morgan
Browse files

Don't forget to ratelimit calls outside of RegistrationHandler

parent b33c4f7a
Branches
Tags
No related merge requests found
...@@ -168,7 +168,7 @@ class RegistrationHandler(BaseHandler): ...@@ -168,7 +168,7 @@ class RegistrationHandler(BaseHandler):
Raises: Raises:
RegistrationError if there was a problem registering. RegistrationError if there was a problem registering.
""" """
yield self._check_registration_ratelimit(address) yield self.check_registration_ratelimit(address)
yield self.auth.check_auth_blocking(threepid=threepid) yield self.auth.check_auth_blocking(threepid=threepid)
password_hash = None password_hash = None
...@@ -415,7 +415,7 @@ class RegistrationHandler(BaseHandler): ...@@ -415,7 +415,7 @@ class RegistrationHandler(BaseHandler):
ratelimit=False, ratelimit=False,
) )
def _check_registration_ratelimit(self, address): def check_registration_ratelimit(self, address):
"""A simple helper method to check whether the registration rate limit has been hit """A simple helper method to check whether the registration rate limit has been hit
for a given IP address for a given IP address
......
...@@ -75,6 +75,8 @@ class ReplicationRegisterServlet(ReplicationEndpoint): ...@@ -75,6 +75,8 @@ class ReplicationRegisterServlet(ReplicationEndpoint):
async def _handle_request(self, request, user_id): async def _handle_request(self, request, user_id):
content = parse_json_object_from_request(request) content = parse_json_object_from_request(request)
await self.registration_handler.check_registration_ratelimit(content["address"])
await self.registration_handler.register_with_store( await self.registration_handler.register_with_store(
user_id=user_id, user_id=user_id,
password_hash=content["password_hash"], password_hash=content["password_hash"],
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment