Я уже писал про грабли HttpWebRequest, но тут нашёл ещё одну забавную, и местами неприятную. Правда наполовину она относится к HttpWebResponse, но классы связаны метровым канатом, так что не принципиально.
Вкратце: работа с куками организована очень оригинально, и не очень логично.
Для удобства распишу по условным пунктам. Для начала диспозиция:
Но! У Headers можно взять значения через .GetValues(), вернётся честный массив из двух элементов. И вроде бы всё хорошо, и пост можно жакончить, но тут приходит сервер, и выдаёт нам:
Set-Cookie: ABC=123; expires=Fri, 31 Dec 2010 23:59:59 GMT; path=/;
Вкратце: работа с куками организована очень оригинально, и не очень логично.
Для удобства распишу по условным пунктам. Для начала диспозиция:
- Сервер возвращает куки в Http-заголовке Set-Cookie
- Если надо установить две куки, сервер передаёт два заголовка (не очень логично, но ок)
- HttpWebResponse имеет пропертю Headers, и по имени заголовка можно получить значение
Тут начинаются проблемы, ибо куки две, а заголовок один. Как вы думаете что вернётся? Не буду томить, скажу, что вернётся в этом методе содержимое двух кук через запятую. Очень, блин удобно. Считаем, что куки у нас разломаны в этом виде.
Но! У Headers можно взять значения через .GetValues(), вернётся честный массив из двух элементов. И вроде бы всё хорошо, и пост можно жакончить, но тут приходит сервер, и выдаёт нам:
Set-Cookie: ABC=123; expires=Fri, 31 Dec 2010 23:59:59 GMT; path=/;
Вы заметили, что между пятницей и 31-ым числом есть запятая? HttpWebResponse тоже заметил, и вместо этой куки честно вернул нам две, обе разломанные. Всё, приехали.
Но обойти это надо, поэтому можно сделать следующие вещи:
- Вручную распарсить значения кук, зная, что запятая, по стандарту, запрещённый символ. Т.е. встретится она может только в expires
- Взять у HttpWebResponse пропертю Cookies, с уже обработанными куками
Вроде второй вариант самый правильный и логичный, за исключением случаев, когда вам хочется посмотреть на изначальные данные от сервера, а не обработанные странным кодом. Но чтобы он работал, надо обязательно, у HttpWebReqest установить пропертю CookieContainer, иначе вам вернётся пустой массив в респонзе.
request.CookieContainer = new CookieContainer()
Немного нелогично (нам ведь только ответные нужны), но в принципе допустимо. И всё из-за весьма странной реализации работы с заголовками.
На этом всё, буду ловить очередные грабли данного класса.
На этом всё, буду ловить очередные грабли данного класса.
ИМХО, .NET тут не причем, сам стандарт корявый, посмотри на Appendix A2.3 в http://arxiv.org/pdf/cs/0105018v1.pdf
ОтветитьУдалитьСтандарт кривой во всех местах, это да.
УдалитьНо есть нюансы:
By convention, duplicate HTTP headers can be “folded” into a single header,
with the duplicate values separated by comma.
Такую свёртку не понимает никто. Но .NET решил сворачивать.
) If comma were to be allowed to separate cookies according to the HTTP convention,
then a “quoting” mechanism would be necessary for attribute values,
especially expires, for which two of the acceptable formats contain embedded
commas, and for cookie values (although they were specified to exclude
comma).
Тот же .NET забывает квортирование и сам же встаёт на грабли со сплитом по запятой.
Так что, моё мнение, что реализовывали абстрактный "стандарт", но не думали о том, как работает мир.