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

http3: remove RoundTripOpt.CheckSettings #4416

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

marten-seemann
Copy link
Member

@marten-seemann marten-seemann commented Apr 6, 2024

Depends on #4424. Part of #3522.

@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from 448fe69 to 7a71fc4 Compare April 6, 2024 17:41
Copy link

codecov bot commented Apr 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.68%. Comparing base (9bc7bd8) to head (a2c01d7).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4416      +/-   ##
==========================================
+ Coverage   84.60%   84.68%   +0.08%     
==========================================
  Files         152      152              
  Lines       14382    14372      -10     
==========================================
+ Hits        12167    12170       +3     
+ Misses       1710     1699      -11     
+ Partials      505      503       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from 7a71fc4 to 25307b5 Compare April 6, 2024 17:53
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from 25307b5 to 8427215 Compare April 8, 2024 00:40
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch 2 times, most recently from 41456df to 7021b84 Compare April 8, 2024 03:06
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch 2 times, most recently from dd82d91 to 0274a23 Compare April 11, 2024 16:57
@marten-seemann marten-seemann changed the base branch from http3-remove-hijackers to master April 11, 2024 16:57
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from 0274a23 to fe0509f Compare April 11, 2024 22:30
@marten-seemann marten-seemann changed the title http3: add Settings to SingleDestinationRoundTripper, remove RoundTripOpt.CheckSettings http3: remove RoundTripOpt.CheckSettings Apr 11, 2024
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from fe0509f to eb22ca7 Compare April 11, 2024 22:49
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from eb22ca7 to 4af97cb Compare April 20, 2024 09:15
@marten-seemann marten-seemann added this to the v0.43 milestone Apr 20, 2024
The settings can be obtained from the SingleDestinationRoundTripper.
@marten-seemann marten-seemann force-pushed the http3-single-roundtripper-settings branch from 4af97cb to a2c01d7 Compare April 20, 2024 09:23
@marten-seemann marten-seemann merged commit 18422ad into master Apr 20, 2024
34 checks passed
@marten-seemann marten-seemann deleted the http3-single-roundtripper-settings branch April 21, 2024 09:07
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.

None yet

1 participant