[GitHub] [incubator-pagespeed-mod] jmarantz commented on issue #1933: webp images not being served in AMP pages?

[GitBox <gi...@apache.org>]

jmarantz commented on issue #1933: webp images not being served in AMP pages?
URL: https://github.com/apache/incubator-pagespeed-mod/issues/1933#issuecomment-513683931
 
 
   Can you check the content-type served when the jpg is requested from
   Chrome? I think what you are seeing is that because AMP pages are cached
   without regard to user-agent, PageSpeed will not rewrite URLs for images in
   a browser-specific way. But it will optimize images in-place from the
   original URL and will serve these in an AMP context.
   
   -Josh
   
   
   On Mon, Jul 22, 2019 at 9:46 AM Richie Brereton <no...@github.com>
   wrote:
   
   > Hi there, I have an issue but not sure if it's my config or PS. I have PS
   > installed on my server running WordPress with an AMP plugin. When I visit a
   > non-AMP page with Chrome I see the images are served as webp as they should
   > be, but when I visit an AMP page with either my Chrome mobile simulator
   > plug-in or a physical tablet, images aren't served as webp but are instead
   > served as srcset JPGs, is this normal?. It could very well be my own fault
   > as I'm pretty new to PS so I will include my config below:
   >
   > `##############################################################
   > ModPagespeedDisableFilters inline_css,inline_javascript
   >
   > ModPagespeedEnableFilters collapse_whitespace,remove_comments
   > ModPagespeedEnableFilters rewrite_css,rewrite_javascript
   > ModPagespeedEnableFilters combine_css,combine_javascript
   > ModPagespeedEnableFilters defer_javascript
   > ModPagespeedEnableFilters prioritize_critical_css
   > ModPagespeedEnableFilters jpeg_subsampling
   > ModPagespeedEnableFilters convert_gif_to_png
   > ModPagespeedEnableFilters convert_png_to_jpeg,convert_jpeg_to_webp
   > ModPagespeedEnableFilters convert_jpeg_to_progressive
   > ModPagespeedEnableFilters inline_images
   > ModPagespeedEnableFilters responsive_images,resize_images
   > ModPagespeedEnableFilters responsive_images_zoom
   > ModPagespeedEnableFilters lazyload_images
   > ModPagespeedEnableFilters move_css_above_scripts
   > ModPagespeedEnableFilters canonicalize_javascript_libraries
   >
   > ModPagespeedEnableFilters make_show_ads_async
   > ModPagespeedEnableFilters make_google_analytics_async
   > ModPagespeedEnableFilters insert_dns_prefetch
   > ModPagespeedEnableFilters inline_google_font_css
   >
   > ModPagespeedMemcachedServers localhost:11211
   > ModPagespeedCreateSharedMemoryMetadataCache "/var/cache/mod_pagespeed/" 51200
   > ModPagespeedImageRecompressionQuality 85
   > ModPagespeedJpegRecompressionQualityForSmallScreens 70
   > ModPagespeedWebpRecompressionQualityForSmallScreens 70
   > ################################################################`
   >
   > Would appreciate any help thanks.
   >
   > —
   > You are receiving this because you are subscribed to this thread.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/incubator-pagespeed-mod/issues/1933?email_source=notifications&email_token=AAO2IPL36436DKWEOTJ7OVDQAVQWNA5CNFSM4IFVYVW2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HASBCGA>,
   > or mute the thread
   > <https://github.com/notifications/unsubscribe-auth/AAO2IPJHCCED627S3EIKOALQAVQWNANCNFSM4IFVYVWQ>
   > .
   >
   

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