Ignore early hints for 4.9.x

Open unlucku opened this issue 3 years ago • 1 comments

Does the rfc mention if :status: has to be the first response header or not? If it does not, then a for loop to look for the status might be needed.

Alternatively this change can be put in Http2Connection.headers and we can call .get(":status:") on the headers but I think that might end up requiring more changes than it should.

Mar 29 '22 17:03 unlucku

We’re gonna revamp 1xx handling in 5.0.0. I think we get that working with our new 1xx testing stuff in MockWebServer, then backport to 4.9.x.

FYI @yschimke

Apr 02 '22 14:04 swankjesse

I don't think the plan is to land this, but it didn't fix the issue, so probably indicates we need to be able to test it to land the eventual fix.

Sep 04 '22 14:09 yschimke

Rebased onto 4.10.x and fixed the bug plus adding a test

https://github.com/square/okhttp/pull/7443

Sep 10 '22 10:09 yschimke