WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 12 years ago

Last modified 5 years ago

#11824 closed defect (bug) (maybelater)

Erroneous MIME type for .ico files

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

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 17 years ago.

Download all attachments as: .zip

Change History (7)

#1 @cnorris23
17 years ago

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

@cnorris23
17 years ago

#2 @RyanMurphy
16 years ago

possible for 3.1?

#3 @RyanMurphy
16 years ago

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

Patch still applies cleanly. Possible for 3.2?

#4 @Otto42
15 years ago

Related: #16434

#5 @wonderboymusic
12 years ago

  • Milestone Future Release
  • Resolution → maybelater
  • Status accepted → closed

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

#6 @Otto42
12 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.