On Wed, Oct 07, 2020 at 06:50:30PM +0200, Marcin Wanat wrote:

Hi there,

> i am doing simple webp client support check using map and then using
> try_files to check if file exists and serve it.

$1 may not mean what you want it to mean, when more than one regex-thing
is involved. And "map" can be a regex-thing.

> location ~ ^/imgs/([0-9]*)/(.*)$ {
> add_header X-webp $webp_suffix;
> try_files /imgs/$1$webp_suffix
> /imgs/$1.jpg =404;
> }

If you change the location regex to whatever your engine's version of
"save this in a named variable" is, then use that named variable in the
try_files line, you may have better luck.

Perhaps:

location ~ ^/imgs/(?P<numbers>[0-9]*)/(.*)$ {

and use $numbers instead of $1.

> What am i missing ?
>
> 1) Why it does not serve .webp file at all ?
> 2) Why when try_files has webp check on first position it DO NOT serve .jpg
> file but 404 and when i swap order of parameters i DO serve .jpg ?

When nginx needs to know the value of $webp_suffix, the "map" is used,
which messes with your $1.

Then when nginx substitutes in the value of $1, it is not what it was
from the regex location.

So it all Just Works when $webp_suffix does not need to be read.

f
--
Francis Daly francis@daoine.org
_______________________________________________
nginx mailing list
nginx@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx