Closed
Bug 6193
Opened 25 years ago
Closed 25 years ago
NECKO: M5 Mozilla will not load Yahoo! News Stories links on Yahoo main page
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: thully, Assigned: gagan)
References
()
Details
Kernel 2.2.6, Mozilla M5, KDE 1.1, Red Hat 5.2 Yahoo! News stories Links on the main Yahoo! page will not load when clicked in Mozilla M5.
Comment 1•25 years ago
|
||
Well it looks like the problem is with the url being given which is something like '/homer/?http://...'. Looking further to see if I'm munging the URL or if this is a netlib problem.
Updated•25 years ago
|
Assignee: joki → gagan
Comment 2•25 years ago
|
||
The problem seems to be in NS_MakeAbsoluteURL. Reassigning to netlib.
Comment 3•25 years ago
|
||
With the latest Linux Build (1999063008), I can't reproduce the problem described. Clicking on the New stories link goes to correct page.
Summary: M5 Mozilla will not load Yahoo! News Stories links on Yahoo main page → NECKO: M5 Mozilla will not load Yahoo! News Stories links on Yahoo main page
Comment 6•25 years ago
|
||
Windows NT 4 build 7/25/08 My Netscape seems to have the same problem. Clicking on any of the links in the Netcenter Apps box yields "Error Loading URL http://" I looked at the source from the page and the markup looks like this: <A HREF="z.t?cp=myna/http://home.netscape.com/netcenter/mail/index.html">
Known problem with parsing. Marking for M10.
Target Milestone: M9 → M10
Kipp wants to help. Kipp: from what I see we are parsing for :// and since this is a relative link that portion throws off our logic. So we first need to check for ? to detect any random characters after it as query. Let me know if you need more help.
Can you explain what all the '?' cases are for nsStdURL::Parse? I don't recall ? being allowed in so many places...thanks
Comment 10•25 years ago
|
||
Once more, with gagan on the cc list: Can you explain what all the '?' cases are for nsStdURL::Parse? I don't recall ? being allowed in so many places...thanks
Comment 11•25 years ago
|
||
Ok, I've done *some* work to make the url parsing work better. I updated urltest to test the cases implied by this bug and they seem to work (note: there are a few test cases in urltest that don't pass -- try "urltest -abs foo" to see them). Basically I made it so that "abc;param?query#ref" will always parse with "#" taking precedence over "?" which takes precedence over ";". Therefore, if there are funny characters in refs or queries, they will retain them. Which makes this sort of thing: "?http://one-two-three;hello" do the "right" thing. It's not clear if the ref handling is exactly right, however.
Assignee | ||
Comment 12•25 years ago
|
||
*** Bug 14697 has been marked as a duplicate of this bug. ***
Comment 13•25 years ago
|
||
can this be marked fixed
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Comment 14•25 years ago
|
||
links to news stories off www.yahoo.com work for me on 11/5 linux and win32 builds. marking fixed.
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 15•25 years ago
|
||
With the Nov 15th Linux build, this problem is fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•