Correctly merge mix of public and private responses

aschempp · aschempp · commit 5f3f951f20ef · 2018-04-15T08:28:27.000+02:00
If a response is flagged as public and others are flagged private,
the merged response can be considered private-cacheable.
diff --git a/src/Symfony/Component/HttpKernel/HttpCache/ResponseCacheStrategy.php b/src/Symfony/Component/HttpKernel/HttpCache/ResponseCacheStrategy.php
@@ -36,7 +36,7 @@ class ResponseCacheStrategy implements ResponseCacheStrategyInterface
     /**
      * Cache-Control headers that are sent to the final response if the appear in ALL of the responses.
      */
-    private static $inheritDirectives = array('public', 'private', 'immutable');
+    private static $inheritDirectives = array('immutable');
 
     private $embeddedResponses = 0;
     private $isNotCacheableResponseEmbedded = false;
@@ -76,6 +76,14 @@ public function add(Response $response)
             }
         }
 
+        if (false !== $this->flagDirectives['public']) {
+            $this->flagDirectives['public'] = $response->headers->hasCacheControlDirective('public');
+        }
+
+        if (false !== $this->flagDirectives['private']) {
+            $this->flagDirectives['private'] = $response->headers->hasCacheControlDirective('private') || $response->headers->hasCacheControlDirective('public');
+        }
+
         $age = $response->getAge();
         $this->age = max($this->age, $age);
 
@@ -124,7 +132,13 @@ public function update(Response $response)
             return;
         }
 
-        $response->headers->set('Cache-Control', implode(', ', array_keys(array_filter($this->flagDirectives))));
+        $flags = array_filter($this->flagDirectives);
+
+        if (isset($flags['public'], $flags['private'])) {
+            unset($flags['private']);
+        }
+
+        $response->headers->set('Cache-Control', implode(', ', array_keys($flags)));
 
         if ($this->ageDirectives['max-age']) {
             $response->headers->addCacheControlDirective('max-age', $this->ageDirectives['max-age'] + $this->age);
diff --git a/src/Symfony/Component/HttpKernel/Tests/HttpCache/ResponseCacheStrategyTest.php b/src/Symfony/Component/HttpKernel/Tests/HttpCache/ResponseCacheStrategyTest.php
@@ -112,8 +112,6 @@ public function testValidationOnMasterResponseIsNotPossibleWhenItContainsEmbedde
         $this->assertFalse($masterResponse->isValidateable());
         $this->assertFalse($masterResponse->headers->has('Last-Modified'));
         $this->assertFalse($masterResponse->headers->has('ETag'));
-//        $this->assertTrue($masterResponse->headers->hasCacheControlDirective('no-cache'));
-//        $this->assertTrue($masterResponse->headers->hasCacheControlDirective('must-revalidate'));
     }
 
     public function testMasterResponseWithValidationIsUnchangedWhenThereIsNoEmbeddedResponse()

Original file line number	Diff line number	Diff line change
`@@ -112,8 +112,6 @@ public function testValidationOnMasterResponseIsNotPossibleWhenItContainsEmbedde`
`112`	`112`	`$this->assertFalse($masterResponse->isValidateable());`
`113`	`113`	`$this->assertFalse($masterResponse->headers->has('Last-Modified'));`
`114`	`114`	`$this->assertFalse($masterResponse->headers->has('ETag'));`
`115`		`-// $this->assertTrue($masterResponse->headers->hasCacheControlDirective('no-cache'));`
`116`		`-// $this->assertTrue($masterResponse->headers->hasCacheControlDirective('must-revalidate'));`
`117`	`115`	`}`
`118`	`116`
`119`	`117`	`public function testMasterResponseWithValidationIsUnchangedWhenThereIsNoEmbeddedResponse()`