Add HttpHeaderCollection #202
Conversation
| { | ||
| $this->headers[$this->normalizeName($name)]= | ||
| array_values((array) $value); | ||
|
|
There was a problem hiding this comment.
http://www.ietf.org/rfc/rfc2616.txt p.4.2 последний абзац.
Это не обязывает нас при получении $value делать explode по \r\n и ";" ?
Насколько я понимаю, с точки зрения протокола
x-foo: bar;baz
равнозначно с
x-foo: bar
x-foo: baz
There was a problem hiding this comment.
Там речь про comma (","), а не semicolon (";"). Мне неясен это абзац совершенно. Если это так, то как различать значение и список значений? Экранирования нет, запятая не запрещена.
There was a problem hiding this comment.
Тесты показали, что популярным веб-серверам и языкам разработки пофиг на это требование.
Если возражений от сообщества нет, то предлагаю вливать.
|
C остальным вроде бы согласен. Да и предыдущий комментарий - это скорее паранойя, чем реальная необходимость. |
Как я уже отмечал в #197, для
HttpRequestтребуется какой-то контейнер для заголовков, потому что в текущей реализации имя заголовка регистрозависимо.HttpHeaderCollectionвыдернул из #194.