Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore Early Hints (5.x) #7444

Merged
merged 1 commit into from
Oct 16, 2022
Merged

Ignore Early Hints (5.x) #7444

merged 1 commit into from
Oct 16, 2022

Conversation

yschimke
Copy link
Collaborator

Ignore for now to solve #7430

Initial PR from #7196

@yschimke yschimke changed the title Ignore Early Hints Ignore Early Hints (5.x) Sep 10, 2022
@swankjesse
Copy link
Member

Yes, love this. PROOOOBABLY want this on 4.x also.

@swankjesse swankjesse merged commit b565eec into square:master Oct 16, 2022
@yschimke
Copy link
Collaborator Author

Yep, will do.

yschimke added a commit to yschimke/okhttp that referenced this pull request Oct 17, 2022
yschimke added a commit to yschimke/okhttp that referenced this pull request Jan 5, 2023
swankjesse pushed a commit that referenced this pull request Jan 12, 2023
* Ignore early hints for 4.9.x (#7444)

(cherry picked from commit b565eec)

* Expand 103 handling to other non-specific 1XX messages. (#7629)

(cherry picked from commit 7e4576c)

* Cleanup

* Add a test of 103 handling

* Add copyright

* Add copyright

* Fix cache test for 102/103 (#7633)

(cherry picked from commit 930f138)

* Disable test
@yschimke yschimke deleted the earlyhints_5 branch May 27, 2023 11:24
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants