Народ, нам нужен ваш feedback по обработке дублирующихся заголовков в

Question

Народ, нам нужен ваш feedback по обработке дублирующихся заголовков в

http.client.
https://github.com/tarantool/tarantool/issues/2658

Изначально мы сделали, что http заголовки добавляются в hash map и дубликаты тупо перетирают предыдущую версию. Как оказалось, это не очень удобно для Set-Cookie. В итоге мы переделали как в nodejs:

Duplicates in raw headers are handled in the following ways, depending on the header name:

Duplicates of age, authorization, content-length, content-type, etag, expires, from, host, if-modified-since, if-unmodified-since, last-modified, location, max-forwards, proxy-authorization, referer, retry-after, or user-agent are discarded.
set-cookie is always an array. Duplicates are added to the array.
For all other headers, the values are joined together with ', '.

Однако есть мысль, что конкатенировать строки не особо правильно и лучше все дублирующие хедеры вставлять как массив. Однако тогда в коде везде придется проверять массив ли каждый конкретный хедер или нет.

Что думаете по этому поводу? Как лучше сделать?

@intrax @OrangeChaton @Totktonada @bigbes

#database #programming #russian #software #tarantool

0

28.09.2017

1 ответов

19 просмотров

Mons Anderson · Accepted Answer

Mons Anderson

seems ugly часть так, часть этак сделайте все дублирующиеся через "," - это вписывается в RFC

0

02.10.2017

215 похожих чатов

Народ, нам нужен ваш feedback по обработке дублирующихся заголовков в

1 ответов

Похожие вопросы