Merge lp://qastaging/~mterry/ubuntuone-couch/no-ssl-validation into lp://qastaging/ubuntuone-couch
Proposed by
Michael Terry
Status: | Rejected |
---|---|
Rejected by: | Michael Terry |
Proposed branch: | lp://qastaging/~mterry/ubuntuone-couch/no-ssl-validation |
Merge into: | lp://qastaging/ubuntuone-couch |
Diff against target: |
11 lines (+1/-1) 1 file modified
ubuntuone/couch/auth.py (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~mterry/ubuntuone-couch/no-ssl-validation |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John O'Brien (community) | Needs Information | ||
dobey (community) | Needs Information | ||
Review via email:
|
Description of the change
See bug 797281 for details.
To post a comment you must log in.
Unmerged revisions
- 14. By Michael Terry
-
don't validate ssl certficates
Interesting. I'm not sure we should go this "temporary workaround" route by explicitly disabling cert validation, rather than just fixing it so that cert validation works.
Can we either pass in the path to the correct cert file to check against, or somehow fix installation of the CA certs we have to provide in ubuntuone- storage- protocol to be automatically loaded by everything? We will need to change package level dependencies either way, but I think it's the right thing to do, and not much more work. Or is it an insane amount of work to do that?