Commited391df

committed

Don't assume that file binary exists on *nix OS

Certain lightweight distributions such as Alpine Linux (popular for smaller Docker images) do not include it by default.

1 parent1067468 commited391dfCopy full SHA for ed391df

File tree

+15

-1

lines changed

src/Symfony/Component/HttpFoundation/File/MimeType
- FileBinaryMimeTypeGuesser.php

+15

-1

lines changed

Lines changed: 15 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,21 @@ public function __construct($cmd = 'file -b --mime %s 2>/dev/null')`
`43`	`43`	`*/`
`44`	`44`	`publicstaticfunctionisSupported()`
`45`	`45`	`{`
`46`		`-return'\\' !==DIRECTORY_SEPARATOR &&function_exists('passthru') &&function_exists('escapeshellarg');`
	`46`	`+static$supported;`
	`47`	`+`
	`48`	`+if (isset($supported)) {`
	`49`	`+return$supported;`
	`50`	`+ }`
	`51`	`+`
	`52`	`+if ('\\' ===DIRECTORY_SEPARATOR \|\| !function_exists('passthru') \|\| !function_exists('escapeshellarg')) {`
	`53`	`+return$supported =false;`
	`54`	`+ }`
	`55`	`+`
	`56`	`+ob_start();`
	`57`	`+passthru('command -v file',$exitStatus);`
	`58`	`+$binPath =trim(ob_get_clean());`
	`59`	`+`
	`60`	`+return$supported =0 ===$exitStatus &&'' !==$binPath;`
`47`	`61`	`}`
`48`	`62`
`49`	`63`	`/**`

Comments

(0)