New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 9 users

Issue metadata

Status: Fixed
Buried. Ping if important.
Closed: Nov 27
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug

issue 333943
issue 636981

Sign in to add a comment

Issue 744499: Consider downloading `ftp://` resources rather than rendering them.

Reported by, Jul 17 2017 Project Member

Issue description

When even the linux kernel is migrating off FTP[1], it's really time for us to move on. :) I'd suggest that instead of rendering resources when folks navigate to `ftp://` origins, we instead download them.

That is, navigating to `` would download `a.html` rather than renderinf the HTML page, navigating to `` would download `a.mkv` rather than synthesizing an HTML page to play the video, and so on.


Comment 1 by, Jul 17 2017

From a practical standpoint, how would we implement this?  At the mime sniffing layer?  I don't think we can know if a path is a directory or not until around that point.

Comment 2 by, Aug 16 2017

Simplest way would probably be to give FTP non-directory listing responses a bogus binary/octet stream mime type at the URLRequestFTPJob layer, which would bypass mime sniffing entirely.  Anyhow, I support this change.

Comment 3 by, Sep 15 2017

Related  Issue 499063 . :)

Comment 4 by, Sep 17

Project Member
Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit - Your friendly Sheriffbot

Comment 5 by, Sep 17

Status: Available (was: Untriaged)
Hrm...I had thought we already did this.  I guess not.

Comment 6 by, Sep 17

This is something I really do want to do, it's just a question of bandwidth.

PMily: Where does FTP fall on Enamel's internal list of Things to get Rid Of? This is a nice baby step in the direction of removal that I'd like us to think about in Q4 or Q1.

Comment 7 by, Nov 16

Blocking: 333943

Comment 8 by, Nov 16

Status: Started (was: Available)

Comment 9 by, Nov 16

Blocking: 636981

Comment 10 by, Nov 27

Project Member
The following revision refers to this bug:

commit 80462a167555039c1d7276694b476466fa581e45
Author: Mike West <>
Date: Tue Nov 27 16:05:06 2018

Download FTP resources, do not render them.

With the exception of generated directory listing pages, resources
requested via FTP will be downloaded rather than rendered.

Intent thread:!topic/blink-dev/eopgOoY1QLs

Bug:  744499 
Change-Id: Ic04f17986cbceb511a19d541f4dd4c67685df81a
Commit-Queue: Mike West <>
Reviewed-by: Matt Menke <>
Reviewed-by: Jochen Eisinger <>
Cr-Commit-Position: refs/heads/master@{#611125}

Comment 11 by, Nov 27

Status: Fixed (was: Started)

Sign in to add a comment