Re: реакция image_filter на return

[GD <gd@xxxxxxxxxxx>]

On Fri, 13 Jan 2012 00:36:45 +0400
Maxim Dounin <mdounin@xxxxxxxxxx> wrote:

> Hello!
> 
> On Fri, Jan 13, 2012 at 12:25:34AM +0400, GD wrote:
> 
> > On Thu, 12 Jan 2012 23:58:07 +0400
> > Maxim Dounin <mdounin@xxxxxxxxxx> wrote:
> > 
> > > Hello!
> > > 
> > > On Thu, Jan 12, 2012 at 09:43:44PM +0400, GD wrote:
> > > 
> > > > On Thu, 12 Jan 2012 21:00:03 +0400
> > > > Maxim Dounin <mdounin@xxxxxxxxxx> wrote:
> > > > 
> > > > > Hello!
> > > > > 
> > > > > On Thu, Jan 12, 2012 at 12:15:03AM +0400, GD wrote:
> 
> [...]
> 
> > > > > > Смотрел на ngx_http_addition_module, там реакция на return
> > > > > > полностью соответсвует док-ции. Т.е. если срабатывает return 403,
> > > > > > то add_after_body уже не отрабатывает.
> > > > > > 
> > > > > > Получилось полечить image_filter патчем (см. аттач).
> > > > > > Хочется услышать мненеие разработчиков.
> > > > > 
> > > > > Image filter расчитан на то, что он обрабатывает в т.ч. untrusted 
> > > > > ответы со сторонних серверов, и поэтому какая-либо фильтрация по 
> > > > > кодам ответов - не производится.
> > > > 
> > > > не очень логично на мой взгляд, но понятно
> > > > и, да, мой патч уже не кажется мне решением
> > > > 
> > > > не понятно все же почему return не завершает любую дальнейшую обработку
> > > > код ответа здесь не играет роли
> > > 
> > > Потому что image_filter - это фильтр, а return - возвращает ответ, 
> > > который потом проходит через этот фильтр.  Чтобы ответ через 
> > > фильтр не проходил - надо передать обработку в другой location, 
> > > где этот фильтр выключен.
> > 
> > ок, с этим ясно.
> > 
> > а почему в ngx_http_addition_module наоборот?
> > он не "расчитан на то, что он обрабатывает в т.ч. untrusted
> > ответы со сторонних серверов"?
> 
> Нет.  Image filter в этом плане совсем специальный, т.к. делался 
> под специальную задачу.

тогда вопросов больше нет, спасибо большое :)

> 
> Maxim Dounin


-- 
GD <gd@xxxxxxxxxxx>

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