Closed Bug 1600490 Opened 5 years ago Closed 5 years ago

FTP listings show "File:" alt text instead of icon

Categories

(Core Graveyard :: Networking: FTP, defect)

70 Branch
defect
Not set
normal

Tracking

(firefox-esr68 wontfix, firefox70 wontfix, firefox71 wontfix, firefox72 wontfix)

RESOLVED WONTFIX
Tracking Status
firefox-esr68 --- wontfix
firefox70 --- wontfix
firefox71 --- wontfix
firefox72 --- wontfix

People

(Reporter: arnaud.adamy, Unassigned)

References

(Regression, )

Details

(Keywords: regression)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:70.0) Gecko/20100101 Firefox/70.0

Steps to reproduce:

Browse FTP site.

Actual results:

The file icon is broken since a few versions. Firefox display "File: " text instead.

Expected results:

An icon of a blank page of paper with a corner creased should be displayed.

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:72.0) Gecko/20100101 Firefox/72.0
20191130213831

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=ea747bb2ffb77d1fd62b5fa6217cbee15b73d31f&tochange=781485c695e1f07b8782427d556f6570e4a8072f

mozregression-gui can't narrow the range further because this regression is about 2 years old. Still, bug 1222924 looks like the prime suspect because this is what Inspector shows:
<img src="moz-icon://.zip?size=16" alt="File:">

Status: UNCONFIRMED → NEW
Has Regression Range: --- → yes
Has STR: --- → yes
Component: Untriaged → ImageLib
Ever confirmed: true
Flags: needinfo?(gijskruitbosch+bugs)
Keywords: regression
Product: Firefox → Core
Regressed by: 1222924
Summary: File icon for FTP browsing → FTP listings show "File:" alt text instead of icon

Indeed. moz-icon is surely the problem here as every file type icons from the view depends on it.
The folder picture works as it does not depends on it, but from the chrome skin.
I don't know where moz-icon collection should be located, but I think embedded icons as resources is the solution to avoid such (not very important, but highly unpleasant) problem.
Hope it can be solved this way.
Regards.

Breaking app/filetype-specific icons was kind of an explicit decision. See https://bugzilla.mozilla.org/show_bug.cgi?id=1222924#c6 (quoting the relevant bit of comment #5) and comment 7.

I think if we care about FTP we can fix the icons for the file listing. But I'm pretty sure we're going to be removing support eventually, so I don't know how much of a priority fixing this will be.

Component: ImageLib → Networking: FTP
Flags: needinfo?(gijskruitbosch+bugs)

This regression is 2 years old and was basically intended. Insecure ftp:// is dead.
(bug 1593595 comment 2 has tips for ftps:// with vsftpd+FileZilla (bug 85464) and https:// directory listing with nginx (bug 1574475). The best we all can do is sending ftp server operators a short email asking them for switching to https.)

Status: NEW → RESOLVED
Closed: 5 years ago
OS: Unspecified → All
Hardware: Unspecified → All
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.