git-cl-try: print proper access denied error msg
git-cl-try does not properly print error message when the response
status code is 403, because response parsing is broken.
Fix response parsing.
R=sheyang@chromium.org, tandrii@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1773963002
git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@299137 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/git_cl.py b/git_cl.py
index d236e85..7e9e4ae 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -252,9 +252,12 @@
# Buildbucket could return an error even if status==200.
if content_json and content_json.get('error'):
+ error = content_json.get('error')
+ if error.get('code') == 403:
+ raise BuildbucketResponseException(
+ 'Access denied: %s' % error.get('message', ''))
msg = 'Error in response. Reason: %s. Message: %s.' % (
- content_json['error'].get('reason', ''),
- content_json['error'].get('message', ''))
+ error.get('reason', ''), error.get('message', ''))
raise BuildbucketResponseException(msg)
if response.status == 200: