Closed
Bug 18134
Opened 25 years ago
Closed 25 years ago
DOGFOOD: relative URLs do not resolve correctly from web-based pages (FILE: access o.k.)
Categories
(Core :: Networking, defect, P3)
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: rxsherm, Assigned: jud)
References
()
Details
(Whiteboard: [PDT+])
Attachments
(2 files)
3.36 KB,
patch
|
Details | Diff | Splinter Review | |
1.40 KB,
patch
|
Details | Diff | Splinter Review |
Given the following HTML :
<html>
<a href="../../helloworld.html">hello world that doesnt resolve URL
properly</a><br>
</html>
The relative link above will not resolve properly if this page is retrieved from
a web server (HTTP:-prefixed) URL. (If this page is retrieved from a FILE: path,
it works o.k.)
In Mozilla, the URL not only displays incorrectly when highlighted (it appears
as http://helloworld.html) but when clicked, the browser attempts to connect to
the bad URL.
Updated•25 years ago
|
Assignee: leger → gagan
Component: Browser-General → Necko
Comment 5•25 years ago
|
||
Oooppss ... ok this is very much likely the new resolve stuff. We no longer
handle the path independly before exposing it to RelpaceMess. So the ../..
leakes into the host, when there are more .. than directory depth.
Comment 6•25 years ago
|
||
Comment 7•25 years ago
|
||
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 8•25 years ago
|
||
andreas' latest patch fixes this (I verified it). Test case (internal) at URL in
URL field. You should be able to click on the link and goto a page two dirs deep
with the text "Got it!"
Updated•25 years ago
|
Summary: relative URLs do not resolve correctly from web-based pages (FILE: access o.k.) → DOGFOOD: relative URLs do not resolve correctly from web-based pages (FILE: access o.k.)
Comment 9•25 years ago
|
||
This breaks links inside of tinderbox if you're accessing it through
<http://tinderbox.mozilla.org/> rather than
<http://cvs-mirror.mozilla.org/tinderbox/>. This probably has the potential to
break a non-trivial number of web pages... seems like good M11 fodder, given
that it's a regression and Andreas has already been kind enough to submit a
patch.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M11
Assignee | ||
Comment 10•25 years ago
|
||
agreed. m11 approval?
Assignee | ||
Updated•25 years ago
|
Assignee | ||
Comment 11•25 years ago
|
||
added new test case url (old http server wasn't working properly). same steps to
verify
Assignee | ||
Updated•25 years ago
|
Assignee: gagan → valeski
Status: ASSIGNED → NEW
Comment 12•25 years ago
|
||
Putting on PDT+ radar. chofmann believes the fix was checked in. Is this
fixed?
Comment 13•25 years ago
|
||
The tinderbox case, at least, now works correctly.
Assignee | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 14•25 years ago
|
||
fix checked in a few hours ago. sorry about that.
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 15•25 years ago
|
||
Verified on 11/23 build. Various levels of relative URLs tested.
Updated•25 years ago
|
Status: VERIFIED → REOPENED
Comment 16•25 years ago
|
||
Shows correctly, but when cliking on some link in the page the image dissapers.
Load http://www.avp.ee
Look at the banner on the top of the screen, click a link on the page, the
banner dissapers, when all other is still shown.
I do not know may I or not to make chainges in status, but I do it. If not write
to me, please.
Updated•25 years ago
|
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Comment 17•25 years ago
|
||
What you describe is an entirely different problem, it is covered in bug 18790.
The problem described in this bug is still fixed. So I'm closing this one again.
Comment 18•25 years ago
|
||
Bulk move of all Necko (to be deleted component) bugs to new Networking
component.
Comment 19•25 years ago
|
||
works fine on all platforms (14th's build). marking verified.
Comment 20•25 years ago
|
||
works fine on all platforms (14th's build). marking verified.
You need to log in
before you can comment on or make changes to this bug.
Description
•