Closed Bug 1560699 Opened 5 years ago Closed 4 years ago

Consider downloading ftp:// resources rather than rendering them.


(Core :: Networking: FTP, enhancement, P1)




Tracking Status
firefox70 --- fixed


(Reporter: jan, Assigned: jan)


(Blocks 1 open bug, )


(4 keywords, Whiteboard: [necko-triaged][adv-main70-])


(1 file)

ckerschb commented on 13 Dec 2018
Firefox has been blocking FTP subresource loads within http(s) documents since Firefox 61 for a variety of reasons as a first step of deprecating FTP resource loads in the browser. It makes sense to me that we start treating FTP resources as binary data instead of rendering them as the next step.

See Also: → 1496725

Please correct me if I put to wrong component.

Component: Networking → Document Navigation
Component: Document Navigation → Networking: FTP
Keywords: site-compat

Loose P1 - let's try to fix this in this release.

Priority: -- → P1
Whiteboard: [necko-triaged]

Michal, this is mostly about changing the content type we expose on the ftp channel. You know this code well, do you think you could fix this for this release?

Flags: needinfo?(michal.novotny)

Tested with emulated Fennec and Desktop Nightly on Debian Testing. Please trigger a try build, if possible.

Downloaded: (This would be rendered if we didn't prevent MIME sniffing.)

Assignee: nobody → jan

Looks good to me. Thanks for the patch!

Keywords: checkin-needed

Pushed by
Download FTP resources instead of rendering them. r=michal

Keywords: checkin-needed
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70

bug 1475261: Connection errors are empty text instead of error pages. Example:

Therefore an empty file is now offered to download.

See Also: → 1475261
Regressions: 1576171
Whiteboard: [necko-triaged] → [necko-triaged][adv-main70-]
See Also: → 1591673
You need to log in before you can comment on or make changes to this bug.