P.S. this bug specifically affects users of the img_auth.php image authorisation script since it uses wfStreamFile().

Keywords: +Need-Review

Why not the docm dotm etc file extensions ? These are XML office files with Macro's enabled.

Yes, one could add further mime types as in the above link. But these file types do have mimetypes that start like the old ppt, doc, etc. types, so I reasoned that (1) they are very old and nobody has had any problem with them yet, and (2) the Microsoft tools may manage to open them anyway since the format might be compatible. If there are similar problems in these cases, one could of course use other mimetypes here as well.

For these specifically Microsoft-related issues, it might also be possible to use the infamous application/octet-stream as a default. This should at least prevent any renaming, and may suffice to let the operating system apply some more brains to figure out what the file is supposed to be.

Somewhat more authoritative source for MS mimetypes.

http://blogs.msdn.com/b/vsofficedeveloper/archive/2008/05/08/office-2007-open-xml-mime-types.aspx

I think we should just add all of them. I see no reason in doing half work in this case. They are documented, so we should implement it as such. We have had too many unnecessary problems with mime type and file extension mismatches already.

Done in r67196

docx is a zip format. Due to bug 23642 (MimeMagic::detectZipType does not recognize microsoft formats) they still won't be detected properly.