stufan commented on issue #1322: ModifyCachingHeaders changes headers for all resources URL: https://github.com/apache/incubator-pagespeed-ngx/issues/1322#issuecomment-398064610 I dig a bit into the code and I came to some findings but I am not quite sure how good are they to be applied. Basically if I comment ```C if (preserve_caching_headers == kPreserveAllCachingHeaders) { if (StringCaseEqual(name_gs, "ETag") || StringCaseEqual(name_gs, "Expires") || StringCaseEqual(name_gs, "Date") || StringCaseEqual(name_gs, "Last-Modified") || StringCaseEqual(name_gs, "Cache-Control")) { continue; } } else if (preserve_caching_headers == kPreserveOnlyCacheControl) { // Retain the original Cache-Control header, but send the recomputed // values for all other cache-related headers. if (StringCaseEqual(name_gs, "Cache-Control")) { continue; } } // else we don't preserve any headers. ``` in `copy_response_headers_to_ngx` , then caching headers appear for pagespeed resources appear together with the HTML original cache control headers. It looks like when `preserve_caching_headers == kPreserveAllCachingHeaders` or `preserve_caching_headers == kPreserveOnlyCacheControl` the caching headers generated by Pagespeed (for Pagespeed resources) are skipped / not added to the response.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
