Skip to content

Commit 42c985d

Browse files
committed
fix(detectfromheader): #6 update check on accept header when not present
1 parent da286d9 commit 42c985d

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

composer.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@
2626
},
2727
"require-dev": {
2828
"phpunit/phpunit": "^8.1",
29-
"zendframework/zend-diactoros": "^2.0.1",
3029
"friendsofphp/php-cs-fixer": "^2.0",
3130
"oscarotero/php-cs-fixer-config": "^1.0",
32-
"squizlabs/php_codesniffer": "^3.0"
31+
"squizlabs/php_codesniffer": "^3.0",
32+
"laminas/laminas-diactoros": "^2.2"
3333
},
3434
"autoload": {
3535
"psr-4": {

src/ContentType.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,8 +186,11 @@ private function detectFromExtension(ServerRequestInterface $request): ?string
186186
*/
187187
private function detectFromHeader(ServerRequestInterface $request): ?string
188188
{
189+
if (!$request->hasHeader('Accept')) {
190+
return $this->defaultFormat;
191+
}
189192
$headers = call_user_func_array('array_merge', array_column($this->formats, 'mime-type'));
190-
$accept = $request->getHeaderLine('Accept')?: '*/*';
193+
$accept = $request->getHeaderLine('Accept');
191194
$mime = $this->negotiateHeader($accept, new Negotiator(), $headers);
192195

193196
if ($mime !== null) {

0 commit comments

Comments
 (0)