Closed
Bug 443295
Opened 16 years ago
Closed 16 years ago
Login occasionally fails
Categories
(Webtools Graveyard :: Verbatim, defect, P1)
Webtools Graveyard
Verbatim
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dschafer, Assigned: dschafer)
Details
Attachments
(1 file, 1 obsolete file)
2.88 KB,
patch
|
clouserw
:
review+
|
Details | Diff | Splinter Review |
Occasionally, login will fail, returning no error message and without apparent cause. Unfortunately, no other information is known on this issue at this time. Rebooting the browser seemed to fix the problem, even when rebooting the server did not, so it might have something to do with the session string.
Assignee | ||
Updated•16 years ago
|
Priority: -- → P1
Assignee | ||
Comment 1•16 years ago
|
||
Before, no paths were set for cookies, which led to trouble with removing cookies and switching localizations. This patch ensures all cookies are sent with the proper path, so that all cookies have the same path regardless of page or localization.
Attachment #328406 -
Flags: review?(clouserw)
Comment 2•16 years ago
|
||
Comment on attachment 328406 [details] [diff] [review] Changes the path for the cookies to the highest-level domain of the site 1) You assume this isn't running on SSL when you determine your cookie path. I know we talked on IRC but let's just hardcode this to '/'. It's fine to have the cookie from the root. 2) If a person doesn't have a session you're setting the value to '-'. I think you said that's how jToolkit did it but is there a good reason? Setting it to "" would delete the cookie from the client which seems cleaner to me.
Attachment #328406 -
Flags: review?(clouserw) → review-
Assignee | ||
Comment 3•16 years ago
|
||
Same as previous patch, but path is always '/', the cookie is now set to '' when we want to remove the login, and expires is set to POSIX time 1 on that remove-login cookie.
Attachment #328406 -
Attachment is obsolete: true
Attachment #328518 -
Flags: review?(clouserw)
Updated•16 years ago
|
Attachment #328518 -
Flags: review?(clouserw) → review+
Assignee | ||
Comment 4•16 years ago
|
||
This is in revision 7735. Marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•