Closed Bug 1224046 Opened 4 years ago Closed 4 years ago

Following links from resource://* listings fails

Categories

(Core :: Networking, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla45
Tracking Status
firefox45 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

STR:
- Open resource://app/
- Click on one of the links

Expected result:
- Link opened

Actual result:
- Nothing happens. Browser console mentions a Security Error: Content at resource://app/ may not load or link to jar:file:///...

mozregression points me to:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=fc5272ac152e8d0fea38ad2e4bd38e6d8aafb744&tochange=6d3e153efe5bff3b3cc4f3690924491aec6856b2

which points to bug 1184387 as the starting point of the breakage. Now, obviously, that was done for a good reason, so the question is how we should work around it.

A simple thing that works is to remove the <base href> from the directory listing page, but I'm not sure what implications that would have for non-resource:// listings.
So, there was an attempt to remove the base href from the directory listings in bug 358128, but that was backed out because of ftp listings, where, aiui, a 'mozilla' link on e.g. the ftp://ftp.mozilla.org/pub listing would go to ftp://ftp.mozilla.org/pubmozilla instead of ftp://ftp.mozilla.org/pub/mozilla.

I don't know if ftp.mozilla.org had something special back then, but trying a few ftp servers, I'm always "redirected" to a url ending with a / when going to a ftp url of a directory that doesn't end with a /. And it looks like this wasn't actually happening back in the 1.5 days (I tested FF 1.5 on windows).

So I guess we may be safe to remove that base href now.
This doesn't seem to cause regressions in the test suite.
Assignee: nobody → mh+mozilla
Attachment #8686485 - Flags: review?(mcmanus)
Comment on attachment 8686485 [details] [diff] [review]
Remove <base href> from directory listings

Review of attachment 8686485 [details] [diff] [review]:
-----------------------------------------------------------------

I'm ok with trying it out
Attachment #8686485 - Flags: review?(mcmanus) → review+
https://hg.mozilla.org/mozilla-central/rev/19e419ab3792
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Depends on: 1232287
Depends on: 1367583
You need to log in before you can comment on or make changes to this bug.