You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pagespeed.apache.org by GitBox <gi...@apache.org> on 2019/07/22 10:51:14 UTC

[GitHub] [incubator-pagespeed-mod] imizwa edited a comment on issue #1933: webp images not being served in AMP pages?

imizwa edited a comment on issue #1933: webp images not being served in AMP pages?
URL: https://github.com/apache/incubator-pagespeed-mod/issues/1933#issuecomment-513741753
 
 
   Ok I have run some tests and can confirm that PageSpeed is not serving up webp images in AMP pages, and that's using a minimal config as shown below:
   
   ```
   ModPagespeedRewriteLevel CoreFilters
   ModPagespeedEnableFilters prioritize_critical_css
   ModPagespeedEnableFilters defer_javascript
   ModPagespeedEnableFilters sprite_images
   ModPagespeedEnableFilters convert_png_to_jpeg,convert_jpeg_to_webp
   ModPagespeedEnableFilters collapse_whitespace,remove_comments
   ```
   
   I also ran a diff check on both normal html and AMP html with PS on and off, it's working fine with standard HTML docs removing comments and rewriting image urls to webp. PS is changing/optimizing some code on the AMP pages (not much) but not touching the PNGs or JPGs and just serves the originals. In terms of content type, the document is HTML:
   
   `<!DOCTYPE html>
   <html amp>
   <head>....`
   
   I wonder if PS is even meant to serve up webp for AMP pages? Nothing I do seems to resolve it and this is happening on 3 different Ubuntu servers (2 cloud and 1 local).
   
   cheers, Richie
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services