search

mozilla / django-browserid

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

Fix #141: Remove email from browserid_info if user authed normally. #158

Closed Osmose closed 11 years ago

Osmose commented 11 years ago

If the user authenticates using a different method than BrowserID, the browserid_info function should not include the user's email for use by the JavaScript.

Osmose commented 11 years ago

@callahad @willkg r?

callahad commented 11 years ago

r+ from skimming the code - seems like a good way to do it. I have not actually tested the code.

willkg commented 11 years ago

I'll look through this tomorrow for sure.

willkg commented 11 years ago

Tests pass. The code looks ok to me. r+