diff options
author | Roxana Ciobanu <roxana.ciobanu@intel.com> | 2014-08-04 15:16:39 +0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-08-06 10:23:39 +0100 |
commit | 6a77a1dd220bf8f8fbaba4fcd6ceca9da27559d2 (patch) | |
tree | 68af7b199e258c2899e75855e1a678f65fa11a1d | |
parent | 2dab9e62125504009ea6798f7746b13097cb6ce6 (diff) | |
download | poky-6a77a1dd220bf8f8fbaba4fcd6ceca9da27559d2.tar.gz |
scripts/send-error-report: fetch /Errors/ instead of /.
If HTTP_PROXY or http_proxy is set when the send-error-report script
is run, it will check to see if fetching / on the specified server
returns 200 without the proxy set. If it does it will assume that the
proxy is not needed. However this check can never work because
fetching / always redirects to /Errors/ in the current code and
thus returns code 301. This is fixed by fetching /Errors/ instead of /.
[YOCTO #YB6576]
(From OE-Core rev: af93c89febcd186d7e31f1d15affc15f38e3379d)
Signed-off-by: Roxana Ciobanu <roxana.ciobanu@intel.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-x | scripts/send-error-report | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/send-error-report b/scripts/send-error-report index 48d983bc0e..c99d3876d7 100755 --- a/scripts/send-error-report +++ b/scripts/send-error-report | |||
@@ -20,7 +20,7 @@ def handle_connection(server, data): | |||
20 | # we need to check that the server isn't a local one, as in no_proxy | 20 | # we need to check that the server isn't a local one, as in no_proxy |
21 | try: | 21 | try: |
22 | temp = httplib.HTTPConnection(server, strict=True, timeout=5) | 22 | temp = httplib.HTTPConnection(server, strict=True, timeout=5) |
23 | temp.request("GET", "/") | 23 | temp.request("GET", "/Errors/") |
24 | tempres = temp.getresponse() | 24 | tempres = temp.getresponse() |
25 | if tempres.status == 200: | 25 | if tempres.status == 200: |
26 | proxyrequired = False | 26 | proxyrequired = False |