oschaaf commented on issue #1556: Combined css returns truncated, mis-encoded 
body on cache hit
URL: 
https://github.com/apache/incubator-pagespeed-ngx/issues/1556#issuecomment-391059716
 
 
   @morlovich Ah..I forgot about that backtrace.. To be honest my memory of 
that is bit blurry right now because I've been looking at so many things 
between then and now.
   Maybe back then the content associated to the headers looked inconsistent 
there? Unfortunately, I didn't capture that in the comment :(
   
   Looking at the first backtrace in 
https://github.com/apache/incubator-pagespeed-ngx/issues/1556#issuecomment-389066123
 again, what strikes me is that the response headers already have the gzip 
header set in that trace, while `ResourceCombiner::WriteCombination` works with 
uncompressed contents right? In the second backtrace, 
`InflatingFetch::GzipValue` gets invoked to add it (and perform actual 
compression).

----------------------------------------------------------------
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

Reply via email to