Chromium Code Reviews| Index: presubmit_canned_checks.py |
| =================================================================== |
| --- presubmit_canned_checks.py (revision 78512) |
| +++ presubmit_canned_checks.py (working copy) |
| @@ -655,8 +655,12 @@ |
| if not input_api.change.issue: |
| return [] |
| - path = '/api/%s?messages=true' |
| - url = (input_api.host_url + path) % input_api.change.issue |
| + # TODO(dpranke): Should figure out if input_api.host_url is supposed to |
|
M-A Ruel
2011/03/17 19:12:54
There's also the problem that some issues needs to
|
| + # be a host or a scheme+host and normalize it there. |
| + host = input_api.host_url |
| + if not host.startswith('http://'): |
|
M-A Ruel
2011/03/17 19:43:34
Some host are https:// only, please take that in a
|
| + host = 'http://' + host |
| + url = '%s/api/%s?messages=true' % (host, input_api.change.issue) |
| f = input_api.urllib2.urlopen(url) |
| issue_props = input_api.json.load(f) |