Closed Bug 11648 Opened 25 years ago Closed 25 years ago

nsStdUrl parsing is broken when prehost is provided

Categories

(Core :: Networking, defect, P3)

All
Windows NT
defect

Tracking

()

VERIFIED FIXED

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".
Target Milestone: M10
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.
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.
Blocks: 13449
I have a fix for this, waiting for review ...
Blocks: 12833
Blocks: 12834
fix checked in ...
QA Contact: paulmac → tever
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
works now. Thanks andreas!
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.