MC-31987: Varnish graphql cache has to skip authenticated requests

Prabhu Ram · Prabhu Ram · commit ce1f70291b2f · 2020-03-10T16:14:23.000-05:00
- review fix
diff --git a/app/code/Magento/PageCache/etc/varnish4.vcl b/app/code/Magento/PageCache/etc/varnish4.vcl
@@ -108,7 +108,7 @@ sub vcl_recv {
         #unset req.http.Cookie;
     }
 
-     # Authenticated graphQL customers should not be cached by default
+     # Authenticated GraphQL requests should not be cached by default
     if (req.url ~ "/graphql" && req.http.Authorization ~ "^Bearer") {
         return (pass);
     }
diff --git a/app/code/Magento/PageCache/etc/varnish5.vcl b/app/code/Magento/PageCache/etc/varnish5.vcl
@@ -109,7 +109,7 @@ sub vcl_recv {
         #unset req.http.Cookie;
     }
 
-    # Authenticated graphQL customers should not be cached by default
+    # Authenticated GraphQL requests should not be cached by default
     if (req.url ~ "/graphql" && req.http.Authorization ~ "^Bearer") {
         return (pass);
     }
diff --git a/app/code/Magento/PageCache/etc/varnish6.vcl b/app/code/Magento/PageCache/etc/varnish6.vcl
@@ -109,7 +109,7 @@ sub vcl_recv {
         #unset req.http.Cookie;
     }
 
-    # Authenticated graphQL customers should not be cached by default
+    # Authenticated GraphQL requests should not be cached by default
     if (req.url ~ "/graphql" && req.http.Authorization ~ "^Bearer") {
         return (pass);
     }

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ sub vcl_recv {`
`108`	`108`	`#unset req.http.Cookie;`
`109`	`109`	`}`
`110`	`110`
`111`		`- # Authenticated graphQL customers should not be cached by default`
	`111`	`+ # Authenticated GraphQL requests should not be cached by default`
`112`	`112`	`if (req.url ~ "/graphql" && req.http.Authorization ~ "^Bearer") {`
`113`	`113`	`return (pass);`
`114`	`114`	`}`
Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ sub vcl_recv {`
`109`	`109`	`#unset req.http.Cookie;`
`110`	`110`	`}`
`111`	`111`
`112`		`- # Authenticated graphQL customers should not be cached by default`
	`112`	`+ # Authenticated GraphQL requests should not be cached by default`
`113`	`113`	`if (req.url ~ "/graphql" && req.http.Authorization ~ "^Bearer") {`
`114`	`114`	`return (pass);`
`115`	`115`	`}`