

michael
-
Posts
107 -
Joined
-
Last visited
Content Type
Profiles
Forums
Gallery
Downloads
Articles
Store
Blogs
Blog Comments posted by michael
-
-
Quote
Oh, I take this back. There's a bug in thirtybees core that actually does not do the check correctly -- so you are right, this config can fail for some old browsers.
How to fix it?
-
Do you know how to fix it? When you will fix it please share. Thank you. I will double check and test.
Further reading about Vary issue:
https://community.cloudflare.com/t/vary-user-agent-ignored/23988
-
Cloudflare free will ignore, with Cloudflare pro you should not use webp as it will take care by Cloudflare polish option available in pro.
So with Cloudflare pro you should delete all generated webp.
Cloudflare free - no support Vary header, older browser will not works
I need to test this with Amaozon AWS Cloudfront.
-
This is not a Thirtybees bug this is a Cloudflare "feature".
Cloudfare anyway will ignore any Vary header.
-
Hello @datakick
have you test your solution with older browser or Safari e.g 12, which do not support webP?
"The problem with free Cloudflare it doesn't support HTTP Vary header for cache so when Chrome/Firefox webP supported browser populates Cloudflare cache and then next visitor uses Safari browser which doesn't support webP, then Cloudflare will serve a webP image for Safari user which will be a broken image. "
"Cloudflare cache not supporting HTTP Vary header"
With Chrome and Firefox everything will work. With older Safari no.
I have your code, I have changed it little and I see that it not works with Safari 12, with Safari 13 works.
Do not use any emulator or testing websites as there are false positive.
MAC or browser.
Have you test it? Safari browser:
http://appldnld.apple.com/Safari5/041-5487.20120509.INU8B/SafariSetup.exe
I am looking a solution:
location ~ (.+)\.(png|jpe?g)$ { if ( $http_accept ~* webp ) { set $webp "A"; } if ( $request_filename ~ (.+)\.(png|jpe?g)$ ) { set $file_without_ext $1; } if ( -f $file_without_ext.webp ) { set $webp "${webp}E"; } if ( $webp = AE ) { add_header Vary Accept; rewrite (.+)\.(png|jpe?g)$ $1.webp break; } }
https://gist.github.com/sergejmueller/7672727
-
thank you
-
@Peter do you use any configuration for Prestashop or Thirtybees for https://www.modpagespeed.com/doc/build_ngx_pagespeed_from_source ???
@data have you test ngx_pagespeed Nginx module?
I am looking for config for ngx_pagespeed. Can anyone share?
-
use original IP address changed by cloudflare
what is for? what is the purpose?
-
From my knowledge.
Easiest way is to use Cloudflare Pro or higher paid plan and enable Polish with webP on the fly conversion. Cloudflare free plan doesn't support webP nor does it support Nginx level webP
Cloudflare can't properly cache webP files and different between supported webP browsers and non-supported like Safari if done at Nginx level.
With Cloudflare pro and higher it make automatically conversion to WebP.
Do you agree with it?
How your config works?
Which Cloudflare plan do you use?
# use original IP address changed by cloudflare
for what it is this option? How it works?
-
Hello @datakick
Does Thirtybees support WebP images?
Is there any module?
Please have a look at centminmod LEMP it is very poverful and out of box LEMP.
nginx configuration for thirtybees
in Datakick's Tips and Tricks
A blog by datakick in General
Posted
Hi @data
Please advise if this bug is corrected?