Merge ~roadmr/canonical-identity-provider:call-me-saml-sha2-maybe into canonical-identity-provider:master
Proposed by
Daniel Manrique
Status: | Merged |
---|---|
Approved by: | Daniel Manrique |
Approved revision: | 73e7807f2a812b4a20681863517b6d08a3c5bf1f |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~roadmr/canonical-identity-provider:call-me-saml-sha2-maybe |
Merge into: | canonical-identity-provider:master |
Diff against target: |
201 lines (+104/-16) 5 files modified
requirements.txt (+3/-1) requirements_devel.txt (+3/-0) src/identityprovider/tests/cert/certificate-2048bit.pem (+21/-0) src/identityprovider/tests/cert/private-key-2048bit.pem (+27/-0) src/ubuntu_sso_saml/tests/test_processors.py (+50/-15) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Maximiliano Bertacchini | Approve | ||
Review via email: mp+388613@code.launchpad.net |
Commit message
Update saml2idp to 0.21 for proper, tested, working sha2 digest/signature support in SAML.
Also update the tests so we're sure the correct identifiers are used at the SSO level; correct signing itself is tested thoroughly in the saml2idp project proper.
Had a bit more repercussions than I expected (required adding bs4 and updating m2crypto which required a custom wheel instead of system package) but it works well in local tests....
To post a comment you must log in.
LGTM