Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Бага в proxy_method
Hello!
On Mon, Jan 14, 2013 at 04:50:52PM +0200, Gena Makhomed wrote:
> On 14.01.2013 16:24, Maxim Dounin wrote:
>
> >>>>>А, нет, вру, должно быть всё нормально и без пробела, это
> >>>>>действительно бага.
> >>>>
> >>>>>У тебя proxy_method задан на уровне http{}, да?
> >>>>
> >>>>Да, на уровне http{}.
> >>
> >>MD> Патч.
> >>
> >>Неправильный патч.
> >>Правильно будет делать trim() и добавлять пробел всегда.
>
> >А не по^Wвсё ли равно? Цель схлопнуть несколько пробелов в один,
> >если их там вдруг больше одного, мне представляется старнной и
> >малоосмысленной. Пробел - разделитель, сколько их там будет, если
> >пользователь написал в конфиге метод с пробелами - неважно.
>
> никто не может гарантировать, что все http backend`ы будут правильно
> работать, если вместо ожидаемого ими одного пробела придет
> несколько.
Для этого надо явно написать более одного пробела в конфиге, что
как бы позволяет предположить, что пользователь этого
действительно хотел.
> да и просто - некрасиво, что в этом месте nginx будет нарушать RFC.
>
> http://www.w3.org/Protocols/rfc2616/rfc2616-sec5.html#sec5.1
>
> Request-Line = Method SP Request-URI SP HTTP-Version CRLF
>
> SP = <US-ASCII SP, space (32)>
>
> P.S. кроме того, http://en.wikipedia.org/wiki/Robustness_principle
Читать про "implied *LWS" там тут:
http://tools.ietf.org/html/rfc2616#section-2.1
Несколько пробелов - это допустимо.
--
Maxim Dounin
http://nginx.com/support.html
_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru
|