csandanov opened a new issue #1655: Clarification about downstream caching 
documentation
URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1655
 
 
   At https://www.modpagespeed.com/doc/downstream-caching#standard
   
    in the varnish example config the following lines added:
   ```vcl
   # Mark HTML as uncacheable.  If we can't send them purge requests they can't
   # cache our html.
   sub vcl_backend_response {
      if (beresp.http.Content-Type ~ "text/html") {
        unset beresp.http.Cache-Control;
        set beresp.http.Cache-Control = "no-cache, max-age=0";
      }
      return (deliver);
   }
   ```
   
   which is a bit confusing to me, because as I understand (please correct me 
if I'm wrong) the whole point of this section is to have cache enabled by 
allowing pagespeed to purge varnish when needed when a portion of hit/miss 
requests performed with beacons. But from these lines we still have no cache 
for text/html, also good varnish configuration would respect `no-cache` header 
from backend anyway
   
   Also, it's not clear how the html cache would be enabled on pagespeed side, 
does specifying `DownstreamCachePurgeLocationPrefix` and 
`DownstreamCacheRebeaconingKey`  options will signal to pagespeed to not set 
`no-cache; max-age=0` for html pages? It's not obvious.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to