Closed
Bug 296369
Opened 19 years ago
Closed 19 years ago
Digest auth broken
Categories
(Firefox :: General, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: pavel.penaz, Assigned: dougt)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
1.91 KB,
patch
|
darin.moz
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050602 Firefox/1.0+ Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050602 Firefox/1.0+ Digest authentification has stopped working after checkin for bug 292368. Reproducible: Always Steps to Reproduce: 1. go to http://84.242.65.71/test/ 2. input login/password: guest/guest 3. see what happens (doesn't work) 4. try the same steps in internet explorer (works) Actual Results: In firefox I get the following error message: Authorization Required This server could not verify that you are authorized to access the document requested. Either you supplied the wrong credentials (e.g., bad password), or your browser doesn't understand how to supply the credentials required. Expected Results: Successful login
Reporter | ||
Updated•19 years ago
|
Reporter | ||
Updated•19 years ago
|
Flags: blocking1.8b3?
Comment 1•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050602 Firefox/1.0+ ID:2005060202 confirmed this works in the 20050601-07:04 pdt build fails in the 20050601-09:40pdt build and any after that CC-DougT
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 3•19 years ago
|
||
simple fix -- need to change the contract id to get the right object. doh.
Updated•19 years ago
|
Attachment #185154 -
Flags: review+
Assignee | ||
Comment 4•19 years ago
|
||
Checking in nsHttpDigestAuth.cpp; /cvsroot/mozilla/netwerk/protocol/http/src/nsHttpDigestAuth.cpp,v <-- nsHttpDigestAuth.cpp new revision: 1.21; previous revision: 1.20 done Checking in nsHttpDigestAuth.h; /cvsroot/mozilla/netwerk/protocol/http/src/nsHttpDigestAuth.h,v <-- nsHttpDigestAuth.h new revision: 1.7; previous revision: 1.6 done Thanks for the bug report. Sorry for breaking you today. Tomorrow's build will have this fix. Doug
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•19 years ago
|
||
Thanks for the quick fix, WFM with latest trunk build -> VERIFIED
Status: RESOLVED → VERIFIED
Updated•19 years ago
|
Flags: blocking1.8b3?
You need to log in
before you can comment on or make changes to this bug.
Description
•