Почему в Nginx не работают переменные для try_files?

Question

[monochromer]

Пытаюсь решить такую задачу - вместо картинок png и jpeg раздавать заранее созданные варианты для avif и webp, если браузер их поддерживает. Например, браузер запрашивает картинку `image.png`. Если поддерживает и avif и webp, то пробуем сначала отдать картинку `image.png.avif`, если её нет, то картинку `image.png.webp`. Если и её нет, то отдаём саму картинку `image.png`.

Основаная логика такая:

location ~* \.(png|jpe?g|gif)$  {
  set $avif_image_path "";
  if ($http_accept ~* "image/avif") {
    set $avif_image_path "$uri.avif";
  }

  set $webp_image_path "";
  if ($http_accept ~* "image/webp") {
    set $webp_image_path "$uri.webp";
  }

  add_header Vary Accept;

  try_files $avif_image_path $webp_image_path $uri =404;
}

Но Nginx всегда возвращает оригинальный файл изображения.

Полный листинг конфига:

spoiler

worker_processes auto;

events {
  worker_connections 1024;
  multi_accept on;
}

http {
  access_log /dev/stdout;
  error_log /dev/stdout info;

  charset utf-8;
  server_tokens off;
  sendfile on;
  tcp_nodelay on;
  tcp_nopush on;

  include mime.types;
  default_type application/octet-stream;

  server {
    listen 80;

    root /usr/share/nginx/public;
    index index.html index.xml;

    location ~* \.(png|jpe?g|gif)$  {
      set $avif_image_path "";
      if ($http_accept ~* "image/avif") {
        set $avif_image_path "$uri.avif";
      }

      set $webp_image_path "";
      if ($http_accept ~* "image/webp") {
        set $webp_image_path "$uri.webp";
      }

      add_header Vary Accept;
      add_header X-Debug "avif: `$avif_image_path`, webp: `$webp_image_path`, uri: `$uri`";

      try_files $avif_image_path $webp_image_path $uri =404;
    }
  }
}

Репозиторий с примером - https://github.com/monochromer/nginx-image-test

Comments

[Aetae]

Основной совет всегда один: не использовать if.

Расширено: конфиг nginx это не скрипт, а декларативный собственно конфиг - порядок исполнения команд и заполнения переменных зависит не от того, в каком порядке ты их написал, а от внутренней логики nginx.

Увы, я не достаточно гуру, чтоб точно сказать в каком порядке тут всё происходит, но либо переменные заполняются после обработки try_files, либо ещё какая фигня с порядком.:)

Скорее всего поможет использовать map заранее задекларирав переменные по условиям.

[Ivan Ustûžanin]

я не видел такой реализации, и видимо неспроста
как пример: https://github.com/cdowdy/Nginx-Content-Negotiatio... и там используется map

Aetae, я тоже не гуру, но подозреваю, что инициализация переменной вне if перебивает установку внутри

[dodo512]

How nginx "location if" works.

[monochromer]

я занёс данные в заголовок `X-Debug`, туда попадают ожидаемые значения, но как-то они неверно интерпретируются внутри `try_files`.

[Aetae]

monochromer, ещё раз:
У тебя написано допустим так:

set
add_header 
try_files

а работает оно как-то так:

try_files
set
add_header

Тебе привели пример с map - используй его.

Answer 1

[dodo512]

Когда истинно уловие if исполняется только содержимое блока if.
И то что унаследовано от основной конфигурации, например, add_header.
А вот try_files не наследуется и исполняется только когда уловие if ложно.

Эти проблемы возникают только в контексте location.
При использовании if в контексте server таких проблем нет.
Достаточно перенести if на уровень server и всё заработает как ожидалось.

server {
  set $avif_image_path "";
  if ($http_accept ~* "image/avif") {
    set $avif_image_path "$uri.avif";
  }

  set $webp_image_path "";
  if ($http_accept ~* "image/webp") {
    set $webp_image_path "$uri.webp";
  }

  location ~* \.(png|jpe?g|gif)$  {
    add_header Vary Accept;

    try_files $avif_image_path $webp_image_path $uri =404;
  }

Или как советовали в комментариях избавиться от if и переписать на map.

Comments

[monochromer]

Переменные будут разделяться, получается, между всеми пользователями?

[dodo512]

monochromer, что значит разделяться между всеми пользователями?
Каждый запрос пользователя обрабатывается отдельно.
В процессе обработки переменным заново присваиваются значения.
На каждый запрос будут новые $http_accept, $uri, $avif_image_path, $webp_image_path.

Answer 2

[Lynn «Кофеман»]

Вариант с map:

map $http_accept $avif_image_path {
  default "";
  ~image/avif "$uri.avif";
}
map $http_accept $webp_image_path {
  default "";
  ~image/webp "$uri.webp";
}

server {
  # ...
  location ~* \.(png|jpe?g|gif)$  {
    add_header Vary Accept;

    try_files $avif_image_path $webp_image_path $uri =404;
  }
  # ...
}

Comments

[monochromer]

Кажется, необязательно писать `default ""` в данном случае.

[Lynn «Кофеман»]

monochromer, наверное, но мне привычнее так. И сразу явно видно.