Make WordPress Core

Opened 14 years ago

Closed 10 years ago

Last modified 2 years ago

#11824 closed defect (bug) (maybelater)

Erroneous MIME type for .ico files

Reported by: davidevicario's profile davide.vicario Owned by: cnorris23's profile cnorris23
Milestone: Priority: low
Severity: minor Version: 2.9.1
Component: Media Keywords: has-patch dev-feedback
Focuses: Cc:

Description

wp-includes/functions.php

line 2286

$mimes = apply_filters( ...
...
'ico' => 'image/x-icon',
...)

this is an erroneous label for ico file.
the right one is:

$mimes = apply_filters( ...
...
'ico' => 'image/vnd.microsoft.icon',
...)

read : http://www.iana.org/assignments/media-types/image/vnd.microsoft.icon

or: http://en.wikipedia.org/wiki/ICO_(file_format)

Attachments (1)

11824.001.diff (457 bytes) - added by cnorris23 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @cnorris23
14 years ago

  • Component changed from General to Media
  • Keywords has-patch added
  • Milestone set to Future Release
  • Owner set to cnorris23
  • Status changed from new to accepted
  • Version set to 2.9.1

@cnorris23
14 years ago

#2 @RyanMurphy
14 years ago

possible for 3.1?

#3 @RyanMurphy
13 years ago

  • Cc ryan@… added
  • Keywords dev-feedback added

Patch still applies cleanly. Possible for 3.2?

#4 @Otto42
12 years ago

Related: #16434

#5 @wonderboymusic
10 years ago

  • Milestone Future Release deleted
  • Resolution set to maybelater
  • Status changed from accepted to closed

I just checked a bunch of major websites, all serve .ico as image/x-icon

#6 @Otto42
10 years ago

Windows itself uses image/x-icon and always has. The "vnd.microsoft.icon" may be what IANA says, but it's not actually used by anybody.

Note: See TracTickets for help on using tickets.