Closed
Bug 7038
Opened 25 years ago
Closed 25 years ago
[Necko][PP] Trailing slash hack needs to be removed once Necko lands.
Categories
(Core Graveyard :: Embedding: APIs, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M9
People
(Reporter: SpamIrrer, Assigned: nisheeth_mozilla)
References
()
Details
The following error occured using viewer.exe of M5 on Windows NT4 Service Pack 3:
When I visit my Homepage (http://sites.inka.de/splinter/) an just type http://sites.inka.de/splinter instead http://sites.inka.de/splinter/ i get the forbidden error. Another problem there is, that the background of my website is displayed wrong. The error there it that there occures all 5 CM a big black strip which should not be there.
Updated•25 years ago
|
Component: Apprunner → Viewer App
Summary: Forbidden rrror and not the right display of the background → Forbidden error and not the right display of the background
Comment 1•25 years ago
|
||
1. This is a frames related bug. Viewer gets the redirect and loads the frameset
but possibly calculates the url of the frames from the original url,
and not from the destination of the redirect.
This bug is NOT present in Apprunner.
2. I've checked 1999052208 and I've seen no visual error in the
appearance of the background. (I'm running Win98 though.)
Changing component to: Viewer App (was: Apprunner)
Re-assigning this to match the change in component. It might be that apprunner
working is just a fluke due to the level of indirection (its content is loaded
in an iframe within navigator.xul). I suspect it's a webshell thing destined
for Nisheeth, et al.
Chris -- I'm starting with you on this in the (unlikely) event that you are
examing URL's. If not, then let's see who in the browser/networking team owns
URL resolution.
Updated•25 years ago
|
Assignee: karnaze → nisheeth
Comment 4•25 years ago
|
||
Nisheeth, please take a look at this.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Summary: Forbidden error and not the right display of the background → Trailing slash hack needs to be removed once Necko lands.
Target Milestone: M8
Assignee | ||
Comment 5•25 years ago
|
||
This bug is happening because of weirdness in netlib land. Code in webshell
depends on netlib adding a trailing slash to the URL for the case when the URL
points to a directory. This is only happening for the case when the frameset
document does not exist in the cache. So, if you nuke your cache and load the
above URL, everything works fine but all subsequent URL loads end up with the
forbidden error message.
When Necko lands, this url rewriting hack will need to be removed. Setting
status and marking target milestone M8.
Assignee | ||
Updated•25 years ago
|
Summary: Trailing slash hack needs to be removed once Necko lands. → [Necko] Trailing slash hack needs to be removed once Necko lands.
Assignee | ||
Comment 6•25 years ago
|
||
Updating summary to show dependency on Necko.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M8 → M9
Assignee | ||
Comment 7•25 years ago
|
||
Moving out Necko related bugs to M9...
Assignee | ||
Updated•25 years ago
|
Target Milestone: M9 → M10
Assignee | ||
Comment 8•25 years ago
|
||
Moving bugs blocked by Necko to M10...
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 9•25 years ago
|
||
This bug has been fixed by the Necko landing...
Comment 10•25 years ago
|
||
niseeth, is this testable in apprunner (and if it is how) or do I have to dig up viewer.
I follow the discussion but I'm not sure how to repro/test/verify in apprunner
Assignee | ||
Comment 11•25 years ago
|
||
Claudius, just verify that the bug described by SpamIrrer@gmx.de in the first
comment on this report is fixed. It isn't possible to verify the trailing slash
hack removal because that is a code level re-org that results in no behavioral
change.
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Hardware: PC → Other
Summary: [Necko] Trailing slash hack needs to be removed once Necko lands. → [Necko][PP] Trailing slash hack needs to be removed once Necko lands.
Target Milestone: M10 → M9
Comment 12•25 years ago
|
||
Marking bug as VERIFIED fixed for 1999081908 builds, resetting milestone to M9 since that's when
it was fixed. Please note for the eager to test or reproduce this particular
url will not appear to work but that's because of a different bug with the
javascript call for history.back(-1)
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•