Re: allow/deny and return

[Gena Makhomed <gmm@xxxxxxxxx>]

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