Closed (works as designed)
Project:
Drupal core
Version:
8.6.x-dev
Component:
file system
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 May 2016 at 09:55 UTC
Updated:
21 Feb 2018 at 12:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
damienmckennaThis is patch that a client site was using to add the MIME type.
Comment #3
damienmckennaDuh, Damien.
Comment #4
damienmckennaComment #5
damienmckennaAnd this patch is for D8.
Comment #7
damienmckennaRerunning the tests against the correct branch.
Comment #8
andrewmacpherson commentedThe patch in #5 was missing a comma, fixed here.
Comment #9
andrewmacpherson commentedI'm in favour of this - all we're doing here is adding it to a list of known types. The
.brfformat has been around for a long time (20 years?), though oddly it doesn't appear in the IANA MIME type registry.Comment #10
andrewmacpherson commentedHmm, I spoke too soon.
@Damien - can you provide a citation for the
application/brailleMIME-type? The.brffile extension is well known, but I'm having trouble finding information about the MIME-type. I've only managed to find one article about BRF which mentions Mime-type at all, and it suggestsapplication/octet-stream.See https://datatypes.net/open-brf-files
The logic for
Drupal\Core\File\MimeType\ExtensionMimeTypeGuesser::guess()returnsapplication/octet-streamas a fallback, so unless we can find evidence of an official (or common?) MIME-type, we don't need to add this file extension at all.Comment #11
andrewmacpherson commentedComment #12
mgiffordNothing on the mime-type, but the application/octet-stream was something I was thinking about to. Found this other reference: http://www.nbp.org/ic/nbp/publications/ebraille.html
Comment #13
mgiffordoops.
Comment #14
damienmckennaBack in 2006 the Apache developers suggested using "text/plain". So yeah, I'm pushing back on my client to get clarification on why they want it this way.
Comment #17
damienmckennaI finally heard back from my client's contact, and it doesn't seem to be needed anymore, if it was ever really needed in the first place.
Thanks.