Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(207)

Unified Diff: appengine/swarming/swarming_bot/bot_code/bot_auth.py

Issue 2963103002: Always treat TokenError as fatal error. (Closed)
Patch Set: Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | client/tests/auth_server_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/swarming/swarming_bot/bot_code/bot_auth.py
diff --git a/appengine/swarming/swarming_bot/bot_code/bot_auth.py b/appengine/swarming/swarming_bot/bot_code/bot_auth.py
index b4e3891384dae9fd43a9bc43eadbdaccbf10e89f..7c8f853e9d7540977308ecf848f5c851faa7c6b5 100644
--- a/appengine/swarming/swarming_bot/bot_code/bot_auth.py
+++ b/appengine/swarming/swarming_bot/bot_code/bot_auth.py
@@ -329,7 +329,7 @@ class AuthSystem(object):
# See remote_client.AUTH_HEADERS_EXPIRATION_SEC.
bot_auth_hdr = auth_params.swarming_http_headers.get('Authorization') or ''
if not bot_auth_hdr.startswith('Bearer '):
- raise auth_server.TokenError(2, 'The bot is not using OAuth', fatal=True)
+ raise auth_server.TokenError(2, 'The bot is not using OAuth')
tok = bot_auth_hdr[len('Bearer '):]
# Default to some safe small expiration in case bot_main doesn't report it
« no previous file with comments | « no previous file | client/tests/auth_server_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698