Closed Bug 296369 Opened 20 years ago Closed 20 years ago

Digest auth broken

Categories

(Firefox :: General, defect)

x86
Windows XP
defect
Not set
major

Tracking

()

VERIFIED FIXED

People

(Reporter: pavel.penaz, Assigned: dougt)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

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
Flags: blocking1.8b3?
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
I will look at it today.
Assignee: nobody → dougt
Attached patch patch v.1 (deleted) — Splinter Review
simple fix -- need to change the contract id to get the right object. doh.
Attachment #185154 - Flags: review+
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: 20 years ago
Resolution: --- → FIXED
Thanks for the quick fix, WFM with latest trunk build -> VERIFIED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: