Last modified: 2014-09-15 01:28:41 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T71437, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69437 - image/vnd.microsoft.icon not recognized as MIME type for .ico (can trigger error message when trying to overwrite existing icons)
image/vnd.microsoft.icon not recognized as MIME type for .ico (can trigger er...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
1.23.2
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-12 15:39 UTC by James Haley
Modified: 2014-09-15 01:28 UTC (History)
8 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
This patches the issue for uploading .ico files in Mediawiki 1.23.x. (805 bytes, patch)
2014-09-11 18:44 UTC, Alexia E. Smith
Details

Description James Haley 2014-08-12 15:39:56 UTC
The IANA standard image/vnd.microsoft.icon MIME type is not configured to correspond to the .ico file extension in includes/mime.types, which instead associates it with the Microsoft-preferred but non-standard image/x-icon.

This causes attempts to overwrite already existing icons on our wiki, which were previously allowed to uploaded by MediaWiki 1.16.2, to fail with the error message "File extension '.ico' does not match the detected MIME type 'image/vnd.microsoft.icon'.

A quick Google search shows many other users are affected, including the ShoutWiki farm.
Comment 1 Alexia E. Smith 2014-09-11 18:44:54 UTC
Created attachment 16442 [details]
This patches the issue for uploading .ico files in Mediawiki 1.23.x.

This patches the issue for uploading .ico files in Mediawiki 1.23.x.
Comment 2 Andre Klapper 2014-09-11 19:20:34 UTC
Hi! Thanks for your patch!

You are welcome to use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial

Putting your branch in Git makes it easier to review it quickly. If you don't want to set up Git/Gerrit, you can also use https://tools.wmflabs.org/gerrit-patch-uploader/
Thanks again! We appreciate your contribution.
Comment 3 Alexia E. Smith 2014-09-11 20:14:14 UTC
Patch submitted through the uploader.
https://gerrit.wikimedia.org/r/#/c/159824/
Comment 4 Andre Klapper 2014-09-12 10:11:33 UTC
Thank you! For future reference, adding "Bug: 69437" as the last line in the commit message (see https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines ) creates an automatic notification here. :)
Comment 5 Gerrit Notification Bot 2014-09-13 16:24:41 UTC
Change 159824 had a related patch set uploaded by Brian Wolff:
Make image/vnd.microsoft.icon be an alias for image/x-icon mime type.

https://gerrit.wikimedia.org/r/159824
Comment 6 Bawolff (Brian Wolff) 2014-09-15 01:22:54 UTC
(In reply to Gerrit Notification Bot from comment #5)
> Change 159824 had a related patch set uploaded by Brian Wolff:
> Make image/vnd.microsoft.icon be an alias for image/x-icon mime type.
> 
> https://gerrit.wikimedia.org/r/159824

Note, gerrit notification bot has a bug where it reports the wrong user when people change the commit message. I'm not trying to steal credit :)

-----

Patch is merged. Will appear in next major version of mediawiki. Thanks.
Comment 7 Gerrit Notification Bot 2014-09-15 01:28:41 UTC
Change 159824 merged by jenkins-bot:
Make image/vnd.microsoft.icon be an alias for image/x-icon mime type.

https://gerrit.wikimedia.org/r/159824

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links