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
changes/01/332501/1
nodir@chromium.org 9 years ago
parent 88be9a1c5a
commit baff4e1c8c

@ -252,9 +252,12 @@ def _buildbucket_retry(operation_name, http, *args, **kwargs):
# 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:

Loading…
Cancel
Save