Closed Bug 33868 Opened 25 years ago Closed 25 years ago

URL encoding for Basic Realm authentication doesn't work

Categories

(Core :: Networking, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: rcummins, Assigned: andreas.otte)

References

()

Details

Attachments

(2 files)

From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; N; WinNT4.0; en-US; m14) BuildID: 2000032808 Including the username:password pair as part of the URL works, e.g. http://12345:67890@www.burlco.lib.nj.us/authtest/, however if the username and password are URL encoded, e.g. http://%31%32%33%34%35:%36%37%38%39%30@www.burlco.lib.nj.us/authtest/ the pair are sent to the server verbatim, not decoded. It works as expected in IE5, which is an observation, not an endorsement. Perhaps IE5 is in violation of an RFC I can't find. Reproducible: Always Steps to Reproduce: 1. Go to http://%31%32%33%34%35:%36%37%38%39%30@www.burlco.lib.nj.us/authtest/ Actual Results: You're told the authentication failed Expected Results: The page should load, instead you're told the authentication failed
-> Networking.
trying again ->Networking
Assignee: cbegle → gagan
Status: UNCONFIRMED → NEW
Component: Browser-General → Networking
Ever confirmed: true
QA Contact: asadotzler → tever
-> andreas
Assignee: gagan → andreas.otte
Target Milestone: --- → M18
Easy fix, just unescape the PreHost.
Attached patch fix for the problem (deleted) — Splinter Review
Attached patch real fix this time (deleted) — Splinter Review
Status: NEW → ASSIGNED
fix checked in. This fix worked some days ago, and I'm still sure it works, but authentification seems to be broken in general, so this can't be tested unless authentification is restored.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
authentification is restored, this can be verified now
VERIFIED: Mozilla 0.9 all plats. If someone has time, it might be worth checking to see if this works via http proxy.
QA Contact: tever → benc
verified now...
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: