ПРОЕКТЫ 


  АРХИВ 


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: Бага в proxy_method



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

--
Best regards,
 Gena

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


 




Copyright © Lexa Software, 1996-2009.