mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #17804
[Bug 1249858] A change has been merged
Reviewed: https://reviews.mahara.org/3169
Committed: http://gitorious.org/mahara/mahara/commit/bfaf9fa0b437ea097be5f6ea84c4b4b90b344e3b
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch: 1.6_STABLE
commit bfaf9fa0b437ea097be5f6ea84c4b4b90b344e3b
Author: Aaron Wells <aaronw@xxxxxxxxxxxxxxx>
Date: Wed Apr 2 16:41:22 2014 +1300
Pass original filename to file_mime_type()
Bug 1249858: When being called for file uploads, file_mime_type()
is getting the PHP temp upload file's filename, and so the attempt
to check the file extension is always unsuccessful because those
temp file names are gibberish
Change-Id: I631ca8c80316919b6aefc9712b68017d752ba942
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1249858
Title:
Mahara can't figure out mime types because of a finfo() bug
Status in Mahara ePortfolio:
In Progress
Status in Mahara 1.6 series:
In Progress
Status in Mahara 1.7 series:
In Progress
Status in Mahara 1.8 series:
In Progress
Status in Mahara 1.9 series:
In Progress
Bug description:
There have been several notable Mahara bugs based around the fact that
our current handling of mimetypes is broken.
See for instance:
- https://bugs.launchpad.net/mahara/+bug/1220639
- https://bugs.launchpad.net/mahara/+bug/1249166
The problem is this:
1. You can't really trust the mimetype that the browser sends to you, because different browsers send different wacky things
2. Mahara has long used its own function file_mime_type() in lib/file.php for this purpose
3. file_mime_type() preferentially uses finfo() to check the mimetype. But, there's a bug in finfo() with an external magic db, which is how it is distributed on Ubuntu presently: https://bugs.php.net/bug.php?id=61940
4. file_mime_type() falls back to mime_content_type(). But that's now deprecated
5. If neither of those works, we fall back to trusting what the browser told us, which isn't really the best (see #1)
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1249858/+subscriptions
References