Closed Bug 296369 Opened 19 years ago Closed 19 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.1Splinter 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: 19 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: