Merge lp://staging/~canonical-isd-hackers/canonical-identity-provider/bug_530271_csrf into lp://staging/canonical-identity-provider/release

Proposed by Łukasz Czyżykowski
Status: Merged
Merged at revision: 49
Proposed branch: lp://staging/~canonical-isd-hackers/canonical-identity-provider/bug_530271_csrf
Merge into: lp://staging/canonical-identity-provider/release
Diff against target: 233 lines (+122/-19)
4 files modified
identityprovider/middleware/csrf.py (+6/-2)
identityprovider/templates/403-csrf.html (+21/-0)
identityprovider/tests/test_middleware.py (+92/-16)
identityprovider/views/ui.py (+3/-1)
To merge this branch: bzr merge lp://staging/~canonical-isd-hackers/canonical-identity-provider/bug_530271_csrf
Reviewer Review Type Date Requested Status
Łukasz Czyżykowski (community) Approve
Review via email: mp+26801@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
David Owen (dsowen) wrote :

(ready for review)

Revision history for this message
Łukasz Czyżykowski (lukasz-czyzykowski) wrote :

Looks OK

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.