search

mozilla / django-browserid

Django application for adding BrowserID support.
Mozilla Public License 2.0
180 stars 80 forks source link

Django1.8 #284

Closed abompard closed 9 years ago

abompard commented 9 years ago

Fix the CSRF view for Django 1.8+.

Osmose commented 9 years ago

Looks good except for the one note. Address that concern and I think this is good to merge, thanks!

abompard commented 9 years ago

OK, I changed the code to use the first configured template engine instead of Django's default. I tried to setup session_csrf to test it but didn't manage to. However it works fine with the default CSRF library, and all tests pass.

Osmose commented 9 years ago

LGTM, nice work! r+

Osmose commented 9 years ago

Merged in c904e7d...fe0a40b.