Close #82

neomerx · Feb 1, 2019 · 8ff0934 · 8ff0934
1 parent 070fa47
commit 8ff0934
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/Http/Headers/HeaderParametersParser.php b/src/Http/Headers/HeaderParametersParser.php
@@ -84,11 +84,16 @@ public function parseContentTypeHeader(string $mediaType): MediaTypeInterface
         $parameters = null;
         $count      = count($fields);
         for ($idx = 1; $idx < $count; ++$idx) {
-            if (strpos($fields[$idx], '=') === false) {
+            $fieldValue = $fields[$idx];
+            if (empty($fieldValue) === true) {
+                continue;
+            }
+
+            if (strpos($fieldValue, '=') === false) {
                 throw new InvalidArgumentException('mediaType');
             }
 
-            list($key, $value) = explode('=', $fields[$idx], 2);
+            list($key, $value) = explode('=', $fieldValue, 2);
             $parameters[trim($key)] = trim($value, ' "');
         }
 

diff --git a/tests/Http/Headers/HeaderParametersParserTest.php b/tests/Http/Headers/HeaderParametersParserTest.php
@@ -81,6 +81,11 @@ public function testParseHeadersNoParams2(): void
         self::assertEquals(self::MEDIA_TYPE, $contentType->getMediaType());
         self::assertNull($contentType->getParameters());
 
+        /** @var MediaTypeInterface $contentType */
+        $contentType = $this->parser->parseContentTypeHeader(self::MEDIA_TYPE . ';');
+        self::assertEquals(self::MEDIA_TYPE, $contentType->getMediaType());
+        self::assertNull($contentType->getParameters());
+
         /** @var AcceptMediaTypeInterface $accept */
         $accept = $this->first($this->parser->parseAcceptHeader(self::MEDIA_TYPE . ';'));
         self::assertEquals(self::MEDIA_TYPE, $accept->getMediaType());