Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Double Check Authentication Logic #22

Closed
bousquet opened this Issue Apr 4, 2013 · 2 comments

Comments

Projects
None yet
3 participants
Owner

bousquet commented Apr 4, 2013

There was previously a problem where you would somehow become another user. This commit (182ba06) was aimed at fixing, but I would like @phlipper to double check to understand what was going on and determine what the original issue was and verify that this is resolved.

@ghost ghost assigned phlipper Apr 4, 2013

Member

kjohnston commented Apr 4, 2013

This was happening upon login. In the case @bousquet, @elskwid and I were looking at the user (who could be redirected to one of two places, depending on the type of user) was being redirected to the place that made sense for the prior user we had logged in with.

Collaborator

phlipper commented Apr 8, 2013

This commit looks good to me. I am going to close out this issue and we can re-open a new issue with reproducible steps if this regression resurfaces.

@phlipper phlipper closed this Apr 8, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment