Merge pull request #41 from City-of-Helsinki/UHF-5618_add_missing_xml…

…_content_type UHF-5618: rss feed would not be handled as xml
City-of-Helsinki · Sep 20, 2022 · 3ca44d3 · 3ca44d3
2 parents 6560bff + 0d1b532
commit 3ca44d3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/HttpMiddleware/AssetHttpMiddleware.php b/src/HttpMiddleware/AssetHttpMiddleware.php
@@ -97,10 +97,12 @@ private function isXmlResponse(Response $response) : bool {
     if (!$response->headers->has('content-type')) {
       return FALSE;
     }
-    return str_starts_with(
-      $response->headers->get('content-type') ?: '',
-      'application/xml',
-    );
+    foreach (['application/xml', 'application/rss+xml'] as $type) {
+      if (str_starts_with($response->headers->get('content-type'), $type)) {
+        return TRUE;
+      }
+    }
+    return FALSE;
   }
 
   /**

diff --git a/tests/src/Unit/AssetHttpMiddlewareTest.php b/tests/src/Unit/AssetHttpMiddlewareTest.php
@@ -69,6 +69,7 @@ public function xmlResponseProvider() : array {
     return [
       ['application/xml'],
       ['application/xml; charset=utf-8'],
+      ['application/rss+xml; charset=utf-8'],
     ];
   }