Closed
Bug 11648
Opened 25 years ago
Closed 25 years ago
nsStdUrl parsing is broken when prehost is provided
Categories
(Core :: Networking, defect, P3)
Tracking
()
VERIFIED
FIXED
M10
People
(Reporter: jud, Assigned: gagan)
References
()
Details
I'm not sure if it's related to the % being in the prehost or not (I don't think
it is). But, this url will cause url->GetPath() to spit back
"there.com/aPath/a.html", instead of the expected "/aPath/a.html".
Updated•25 years ago
|
Target Milestone: M10
Comment 1•25 years ago
|
||
Do different rules apply for escaping in the hostname section? Or is %h
supposed to just be treated as an "h"? The url looks malformed, but we should
cope with it nonetheless.
Reporter | ||
Comment 2•25 years ago
|
||
uname%hostname.com is commonly used as a "username" on systems that want to use
the user's email address as the username. using the straight up
uname@hostname.com would break parsing rules for scheme://prehost@foo.com, thus
if you're logging in somewhere that requires your email address as a pwd
(basically any web based login form, or the ftp uname/pwd dialog) the service
you're using requests that you replace the @ with a %, and their system
interprets it the same way. So, to make a long story short, the '%' should not
be escaped in the prehost.
Comment 3•25 years ago
|
||
I have a fix for this, waiting for review ...
Comment 4•25 years ago
|
||
fix checked in ...
Updated•25 years ago
|
QA Contact: paulmac → tever
Reporter | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•25 years ago
|
||
works now. Thanks andreas!
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Bulk move of all Necko (to be deleted component) bugs to new Networking
component.
You need to log in
before you can comment on or make changes to this bug.
Description
•