Merge pull request #855 from libgraviton/feature/no-etag

flex etag
libgraviton · Feb 21, 2024 · 411e05c · 411e05c
2 parents 0e21b00 + 82d5507
commit 411e05c
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 13 deletions.
diff --git a/composer.json b/composer.json
@@ -73,7 +73,7 @@
         "squizlabs/php_codesniffer": "~3",
 
         "graviton/test-services-bundle": "^2.15.0",
-        "graviton/mongodb-fixtures-bundle": "dev-develop",
+        "graviton/mongodb-fixtures-bundle": "^4.0.1",
         "libgraviton/codesniffer": "~2",
 
         "symfony/phpunit-bridge": "^6.0",

diff --git a/composer.lock b/composer.lock
diff --git a/src/Graviton/CacheBundle/Listener/ETagResponseListener.php b/src/Graviton/CacheBundle/Listener/ETagResponseListener.php
@@ -5,6 +5,7 @@
 
 namespace Graviton\CacheBundle\Listener;
 
+use Symfony\Component\HttpFoundation\StreamedResponse;
 use Symfony\Component\HttpKernel\Event\ResponseEvent;
 
 /**
@@ -27,6 +28,10 @@ public function onKernelResponse(ResponseEvent $event)
     {
         $response = $event->getResponse();
 
+        if ($response instanceof StreamedResponse || empty($response->getContent())) {
+            return;
+        }
+
         /**
          * the "W/" prefix is necessary to qualify it as a "weak" Etag.
          * only then a proxy like nginx will leave the tag alone because a strong cannot