Merge lp:~chad.smith/landscape-client/autoregistration-cacert-ignore-invalid-cert into lp:~landscape/landscape-client/trunk
Proposed by
Chad Smith
Status: | Merged |
---|---|
Approved by: | Mike Milner |
Approved revision: | 479 |
Merged at revision: | 480 |
Proposed branch: | lp:~chad.smith/landscape-client/autoregistration-cacert-ignore-invalid-cert |
Merge into: | lp:~landscape/landscape-client/trunk |
Diff against target: |
114 lines (+30/-6) 4 files modified
landscape/configuration.py (+1/-1) landscape/lib/fetch.py (+7/-1) landscape/lib/tests/test_fetch.py (+18/-0) landscape/tests/test_configuration.py (+4/-4) |
To merge this branch: | bzr merge lp:~chad.smith/landscape-client/autoregistration-cacert-ignore-invalid-cert |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mike Milner (community) | Approve | ||
Alberto Donato (community) | Approve | ||
Review via email: mp+96275@code.launchpad.net |
Description of the change
dpb found issues during integration testing that client wget & prcurl redirects http://
Since we can't put the cart before the horse:
This branch adds pycurl.
This insecure fetch option is only used during auto-discovery/
Added a test to ensure the proper curl.setopt is called if fetch(insecure=
To post a comment you must log in.
Nice fix! +1