[HttpFoundation] fixed the check of 'proxy-revalidate' in Response::mustRevalidate()

axiac · axiac · commit c980c3f6905d · 2015-07-30T11:42:41.000+03:00
'proxy-revalidate' is not a header on its own but a 'Cache-Control' directive See http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.9 This fixes issue #15262
diff --git a/src/Symfony/Component/HttpFoundation/Response.php b/src/Symfony/Component/HttpFoundation/Response.php
@@ -587,7 +587,7 @@ public function setPublic()
      */
     public function mustRevalidate()
     {
-        return $this->headers->hasCacheControlDirective('must-revalidate') || $this->headers->has('proxy-revalidate');
+        return $this->headers->hasCacheControlDirective('must-revalidate') || $this->headers->hasCacheControlDirective('proxy-revalidate');
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -587,7 +587,7 @@ public function setPublic()`
`587`	`587`	`*/`
`588`	`588`	`public function mustRevalidate()`
`589`	`589`	`{`
`590`		`- return $this->headers->hasCacheControlDirective('must-revalidate') \|\| $this->headers->has('proxy-revalidate');`
	`590`	`+ return $this->headers->hasCacheControlDirective('must-revalidate') \|\| $this->headers->hasCacheControlDirective('proxy-revalidate');`
`591`	`591`	`}`
`592`	`592`
`593`	`593`	`/**`