hyper icon indicating copy to clipboard operation
hyper copied to clipboard

Option to allow leading whitespace in first HTTP/1 response header

Open bagder opened this issue 1 year ago • 4 comments

Version current git master

Platform Linux

Description hyper rejects HTTP responses where the first header has leading whitespace. This is spec compliant.

The popular browsers accept such headers and therefore we are now converting curl to (again) also allow them (in https://github.com/curl/curl/pull/11607).

hyper's current behavior prevents it from getting content other widely used HTTP clients can fetch. Do you think that is perhaps reason to reconsider this stance and allow this violation? It would make life in curl easier as right now curl+hyper builds have different behavior than curl with its native HTTP implementation.

bagder avatar Aug 07 '23 08:08 bagder

I guess we could allow it in hyper too, perhaps when also enabled to allow obsolete line-folding. For anyone wanting to do this, the fix would need to be in httparse.

seanmonstar avatar Aug 30 '23 19:08 seanmonstar

I would be interested in picking this up edit: once this is unblocked from httpparse.

JordanllHarper avatar Nov 04 '23 15:11 JordanllHarper

I am interested in picking this up, can make the change in httparse

emirakdere avatar Jan 17 '24 18:01 emirakdere

This issue was resolved at https://github.com/seanmonstar/httparse/pull/147

vkobinski avatar Mar 29 '24 04:03 vkobinski