diff options
author | Paul "LeoNerd" Evans <paul@matrix.org> | 2015-03-05 16:24:13 +0000 |
---|---|---|
committer | Paul "LeoNerd" Evans <paul@matrix.org> | 2015-03-05 16:24:13 +0000 |
commit | 9d9d39536b8be554eec2b16b2846ad31ac643721 (patch) | |
tree | 7fdff4d91d75f00e3df180cd46866e09d527e345 /synapse/http | |
parent | Rename rooms_to_listeners to room_to_listeners, for consistency with user_ an... (diff) | |
download | synapse-9d9d39536b8be554eec2b16b2846ad31ac643721.tar.xz |
Slightly reduce the insane amounts of indentation in main http server response path, by 'continue'ing around a non-match or falling through
Diffstat (limited to 'synapse/http')
-rw-r--r-- | synapse/http/server.py | 44 |
1 files changed, 23 insertions, 21 deletions
diff --git a/synapse/http/server.py b/synapse/http/server.py index 74a101a5d7..767c3ef79b 100644 --- a/synapse/http/server.py +++ b/synapse/http/server.py @@ -124,27 +124,29 @@ class JsonResource(HttpServer, resource.Resource): # and path regex match for path_entry in self.path_regexs.get(request.method, []): m = path_entry.pattern.match(request.path) - if m: - # We found a match! Trigger callback and then return the - # returned response. We pass both the request and any - # matched groups from the regex to the callback. - - args = [ - urllib.unquote(u).decode("UTF-8") for u in m.groups() - ] - - logger.info( - "Received request: %s %s", - request.method, request.path - ) - - code, response = yield path_entry.callback( - request, - *args - ) - - self._send_response(request, code, response) - return + if not m: + continue + + # We found a match! Trigger callback and then return the + # returned response. We pass both the request and any + # matched groups from the regex to the callback. + + args = [ + urllib.unquote(u).decode("UTF-8") for u in m.groups() + ] + + logger.info( + "Received request: %s %s", + request.method, request.path + ) + + code, response = yield path_entry.callback( + request, + *args + ) + + self._send_response(request, code, response) + return # Huh. No one wanted to handle that? Fiiiiiine. Send 400. raise UnrecognizedRequestError() |