From ce84b2ddae06871d9267be28045ac55bbb54ab03 Mon Sep 17 00:00:00 2001 From: Lars Wirzenius Date: Mon, 22 Jul 2019 21:04:20 +0300 Subject: Change: return instead of raising --- api.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'api.py') diff --git a/api.py b/api.py index 972b334..54e7da3 100755 --- a/api.py +++ b/api.py @@ -130,9 +130,7 @@ class AccessCheckerPlugin: logging.debug('authz_needed: %r', authz_needed) if authz_needed: logging.debug('checking if access is allowed: %r', scopes) - x = self._checker.access_is_allowed(r.headers, scopes) - logging.debug('x: %r', x) - if not x: + if not self._checker.access_is_allowed(r.headers, scopes): logging.error('Request denied %s %s', r.method, r.path) return bottle.HTTPError(400) @@ -144,7 +142,7 @@ class AccessCheckerPlugin: logging.error( 'Could not handle request: %s %s: %s', r.method, r.path, str(e)) - raise bottle.HTTPError(500) + return bottle.HTTPError(500) return access_checker_wrapper -- cgit v1.2.1