Ticket #7438 (closed: fixed)
support of proxy for scriptrepository
| Reported by: | Gesner Passos | Owned by: | Gesner Passos |
|---|---|---|---|
| Priority: | critical | Milestone: | Release 2.6 |
| Component: | Framework | Keywords: | |
| Cc: | ricardo.leal@… | Blocked By: | |
| Blocking: | Tester: | Nick Draper |
Description (last modified by Gesner Passos) (diff)
Add support for proxy to scriptrepository
feature asked by Ricardo
Change History
comment:2 Changed 7 years ago by Gesner Passos
Implementation of proxy support for ScriptRepository
re #7438
Changeset: eaf50b7e70087a1c1bd2c2c4c9ad23e8646df141
comment:3 Changed 7 years ago by Gesner Passos
Implementation of proxy support for windows
It does correct the parsing of proxy_http in linux also
re #7438
Changeset: 7ec1de2fef9a2a892baadf55a1853d4335900174
comment:4 Changed 7 years ago by Gesner Passos
- Cc ricardo.leal@… added
- Status changed from accepted to verify
- Resolution set to fixed
- Description modified (diff)
Tester:
if possible, I would like Ricardo to test this feature. It is important that the installation, download and upload all work nicely. For sure, can not be anyone from ISIS, unless you know how to setup an external computer under a proxy to use the proxy implementation. Currently, a notice logging will alert that the connection is under proxy, and this string must be correct. Before releasing, this message should be moved to information level.
Suggestions on how to test you may find install and download: http://trac.mantidproject.org/mantid/ticket/6177#comment:24 delete: http://trac.mantidproject.org/mantid/ticket/7031 upload: http://trac.mantidproject.org/mantid/ticket/6176
You better not use the sandbox, as recommended by nick: http://trac.mantidproject.org/mantid/ticket/7031#comment:13
comment:5 Changed 7 years ago by Ricardo Leal
Using ubuntu 12.04 x64 with http_proxy variable set. The proxy appears to work correctly.
comment:6 Changed 7 years ago by Nick Draper
- Status changed from verify to verifying
- Tester set to Nick Draper
comment:7 Changed 7 years ago by Nick Draper
This has been tested by Ricardo, I'm just going to do the merge
