commit b451fc7489c18ff5c7587169eeb965640f8cba66
author Stefan Zager <szager@chromium.org> Sun Sep 01 23:41:52 2013 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Mon Sep 02 20:04:17 2013 +0000
tree 9dba29d802a2f8391ca938fb2e0cab688518bc5f
parent ade27ce41bb1814faf0a2ce6ffd2dc09471068db

Capture failing http requests.

BUG=chromium:283051
TEST=None

Change-Id: I827ef21c0c31c667a2aab448612dbd26b93f5d86
Reviewed-on: https://chromium-review.googlesource.com/167751
Reviewed-by: Chase Phillips <cmp@chromium.org>
Tested-by: Chase Phillips <cmp@chromium.org>
Commit-Queue: Chase Phillips <cmp@chromium.org>

lib/gob_util.py

diff --git a/lib/gob_util.py b/lib/gob_util.py
index 11350ea..c053cb7 100755
--- a/lib/gob_util.py
+++ b/lib/gob_util.py
@@ -127,7 +127,11 @@
   if o_params:
     q.extend(['o=%s' % p for p in o_params])
   path = 'changes/?%s' % '&'.join(q)
-  result = ReadHttpJsonResponse(CreateHttpConn(host, path), ignore_404=False)
+  try:
+    result = ReadHttpJsonResponse(CreateHttpConn(host, path), ignore_404=False)
+  except GOBError as e:
+    msg = '%s:\n%s' % (e.message, path)
+    raise GOBError(e.http_status, msg)
   return result if len(change_list) == 1 else sum(result, [])