ПРОЕКТЫ 


  АРХИВ 


Apache-Talk @lexa.ru 

Inet-Admins @info.east.ru 

Filmscanners @halftone.co.uk 

Security-alerts @yandex-team.ru 

nginx-ru @sysoev.ru 


  СТАТЬИ 


  ПЕРСОНАЛЬНОЕ 


  ПРОГРАММЫ 



ПИШИТЕ
ПИСЬМА












     АРХИВ :: nginx-ru
Nginx-ru mailing list archive (nginx-ru@sysoev.ru)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: allow/deny and return



On 18.10.2013 14:41, Maxim Dounin wrote:

Как уже и предлагалось - методом правильного документирования того
факта, что директивы модуля rewrite - это часть процесса выбора
конфигурации.  И всяческие директивы allow/deny/whatever -
применяются уже после того, как оный выбор случился.

а что делать, если необходимо, чтобы директивы модуля
rewrite отработали после проверок allow/deny/whatever?

То же самое, что и в других подобных ситуациях - использовать
другие механизмы.

а из других механизмов у нас в наличии есть только try_files.
и в этом случае добавление try_files решило проблему с if'ом.

Повторяю: добавление try_files не решило проблем с if'ом.  В
результате как раз с проблемами if'а - ничего не было сделано, а
вместо этого усилия были потрачены на try_files и борьбу с его
проблемами.  Если бы вместо добавления try_files соответствующие
усилия были потрачены именно на _решение_ проблем - было бы лучше.

никто и не обещал, что try_files решит все проблемы с if.

но многие случаи написания конфигурации try_files упрощает.

причем это вне зависимости от того, есть у if проблемы или их нет.

и во многих случаях можно обойтись try_files без использования if.

Да-да, именно так.  И в сухом остатке мы имеем больше проблем, чем
их было до введения директивы try_files.

если считать только количество записей в багтрекере, тогда да,
действительно так и будет: "в сухом остатке мы имеем больше
проблем, чем их было до введения директивы try_files".

если же посмотреть со стороны пользователей - в большинстве случаев написание конфигурации стало проще, удобнее и безглючнее, потому что
вместо фигурного выпиливания лобзиком работающей конфигурации
на основе директив if + return + error_page
+ recursive_error_pages + proxy_intercept_errors
+ fastcgi_intercept_errors + scgi_intercept_errors
+ uwsgi_intercept_errors + ...
можно взять директиву try_files с простой и понятной логикой работы,
которая подходит для большинства наиболее частоиспользуемых случаев.

даже если бы директива if всегда работала бы идеально - добавление
try_files всеравно имеет смысл из-за значительного упрощения конфига.

для решения всех проблем с if ресурсов надо потратить в десятки,
а то и в сотни раз больше, чем их было потрачено на добавление
новой директивы try_files и еще одной фазы обработки запроса.

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

если смотреть с точки зрения максимизации удобства работы пользователей,
то имеет смысл сначала добавить директиву try_files, и только после этого уже браться за более сложную задачу исправления всех ошибок в работе диркетивы if. (вообще не уверен, что эта задача имеет решение)

И это общая проблема всех подобных попыток "уменьшить количество
случаев" вместо реального решения проблемы.  Решать проблему всё
равно рано или поздно придётся, и откладывание этого решения -
лишь ухудшает ситуацию.

что в таком случае можно считать рельным решением проблем директивы if,
- чтобы не осталось ни одного глюка из http://wiki.nginx.org/IfIsEvil ?

про проблемы директивы if говорится очень давно, но видимо эти проблемы
нетривиальные, раз до сих пор никому так и не удалось их реально решить

Я описал правильное с моей точки зрения решение ещё при создании
этой статьи на wiki:

http://wiki.nginx.org/IfIsEvil#Why_this_happens_and_still_not_fixed

: Looks like the only correct fix would be to disable non-rewrite
: directives inside if completely. It would break many configuration
: out there though, so wasn't done yet.

Возможно, хорошим альтернативным вариантом будет вычищение всех
проблем, которые возникают при использовании неявных location'ов.

"вычищение всех проблем, которые возникают при использовании
неявных location'ов" - это будет очень нетривиальная задача.

кстати, если проблемы с директивой if решать способом
"disable non-rewrite directives inside if completely"

тогда точно таким же способом можно решить и все проблемы c try_files,
сказав что эта директива не совместима с директивой alias, нужен root.

--
Best regards,
 Gena

_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru


 




Copyright © Lexa Software, 1996-2009.