Re: Бага в proxy_method

Maxim Dounin mdounin at mdounin.ru
Sun Jan 20 02:48:58 UTC 2013


Hello!

On Tue, Jan 15, 2013 at 02:12:47AM +0400, Andrey Repin wrote:

> Здравствуйте, Уважаемый(-ая, -ое) Maxim Dounin!
> >> >> > А, нет, вру, должно быть всё нормально и без пробела, это
> >> >> > действительно бага.
> >> >> 
> >> >> > У тебя proxy_method задан на уровне http{}, да?
> >> >> 
> >> >> Да, на уровне http{}.
> >> 
> >> MD> Патч.
> >> 
> >> Неправильный патч.
> >> Правильно будет делать trim() и добавлять пробел всегда.
> 
> MD> А не по^Wвсё ли равно?
> 
> Не всё. Есть стандарты. Чем точнее ты им следуешь, тем меньше мест трубуется
> перепроверять. Я, например, с трудом продрался через код вашего патча, и мне
> помогло только то, что я знал, с чего всё началось. Это уже говорит о том, что
> "что-то здесь не так".
> 
> MD> Цель схлопнуть несколько пробелов в один, 
> MD> если их там вдруг больше одного, мне представляется старнной и 
> MD> малоосмысленной.  Пробел - разделитель, сколько их там будет, если 
> MD> пользователь написал в конфиге метод с пробелами - неважно.  
> 
> В стандарте указан один пробел.

Это, к сожалению, неправда.  Ссылку я уже в этом треде приводил, 
но таки мне не жалко и второй раз её же дать.  Читать про "implied 
*LWS" тут:

http://tools.ietf.org/html/rfc2616#section-2.1

-- 
Maxim Dounin
http://nginx.com/support.html



Подробная информация о списке рассылки nginx-ru