Closed (duplicate)
Project:
Drupal core
Version:
8.3.x-dev
Component:
config.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Mar 2014 at 17:57 UTC
Updated:
16 Jun 2017 at 19:25 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
berdirHm, fatal error isn't a minor bug. We should probably add fileinfo to the list of extensions that we need, surprised that this didn't fail before..
Comment #2
Jamie Holly commented+1. php_fileinfo isn't enabled on all distributions by default. Given this is a core item, it should probably be checked during prerequisites.
Comment #3
jhodgdonThis looks like config.module, not the generic config system?
Comment #4
jhedstromHmm, fileinfo has been enabled by default since php 5.3.0. However, it's easy enough to add this to the list.
Comment #5
Jamie Holly commented+1 to including this very minor check, especially for Win users, where you still must include the php_fileinfo.dll in your php.ini, even after 5.3.
Comment #6
chi commentedThis extension cannot be disabled on Linux based platforms, so that only windows users can test properly this patch.
Comment #10
Anonymous (not verified) commentedI can not reproduce this error on Windows with disabled php_fileinfo extension. And looks like from #4 was just an analysis, without confirmation of the problem. Because it was fixed in #2387443: BinaryFileResponse can fail because the core MIME guessing is not added to the MimeType singleton.