Closed Bug 715604 Opened 13 years ago Closed 12 years ago

Logging in does not follow original request

Categories

(Participation Infrastructure :: Phonebook, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED DUPLICATE of bug 810992

People

(Reporter: jdm, Unassigned)

References

Details

(Keywords: regression, Whiteboard: [regression])

If I visit a link to mozillians.org and I am not logged in, I would expect that I would be taken to the original link upon logging in. Instead, I am always redirected to my profile.
This sounds like
SUSPENSE.
This sounds like a user complaint on the 6-hour session cookie we've had on mozillians for awhile.
I don't see the connection. This occurs when I visit a link, get redirected to the login page, login immediately, and then am taken to my profile.
From digging through the code, I'm pretty sure this was a regression introduced along with BrowserID, but I couldn't find the place where the actual redirect occurs upon successful login.
Aha, the culprit is at https://github.com/mozilla/mozillians/blob/master/apps/browserid/views.py#L41 . This was introduced with BrowserID; previously we were presumably using the default django auth stuff that tied in with django.contrib.auth.redirect_to_login, which has some magic that redirects to the given next URL on successfully logging in.
Ah, I see what the issue is. Thanks for flagging down the issue. Interested in offering a commit, Josh?
Keywords: regression
Priority: -- → P2
Target Milestone: --- → 1.3
Ordinarily yes, but there's enough uncertainty in my understanding that I'll leave it to someone more experienced.
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [regression]
Component: mozillians.org → Phonebook
Product: Websites → Community Tools
QA Contact: mozillians-org → phonebook
Target Milestone: 1.3 → ---
Version: unspecified → other
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Bumping to QA verified duplicate.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.