Skip to content

sni: new hint 'http2_initial_window_size_in'#9734

Merged
randall merged 1 commit intoapache:masterfrom
randall:initial_window_size_in_per_sni
Jul 5, 2023
Merged

sni: new hint 'http2_initial_window_size_in'#9734
randall merged 1 commit intoapache:masterfrom
randall:initial_window_size_in_per_sni

Conversation

@randall
Copy link
Contributor

@randall randall commented May 24, 2023

Allow setting the H2 initial window size per SNI

@randall randall added the HTTP/2 label May 24, 2023
@randall randall added this to the 10.0.0 milestone May 24, 2023
@randall randall self-assigned this May 24, 2023
@randall randall force-pushed the initial_window_size_in_per_sni branch 3 times, most recently from a3cffe2 to 2bbcf6f Compare May 24, 2023 17:50
@randall randall marked this pull request as ready for review May 24, 2023 19:53
@bryancall bryancall self-requested a review June 5, 2023 22:22
@randall randall force-pushed the initial_window_size_in_per_sni branch from 2bbcf6f to 7da28ee Compare June 5, 2023 22:27
@randall
Copy link
Contributor Author

randall commented Jun 5, 2023

[approve ci autest]

@randall
Copy link
Contributor Author

randall commented Jun 13, 2023

@bryancall Can I get a review here?

@randall randall force-pushed the initial_window_size_in_per_sni branch from 7da28ee to 93771bc Compare June 30, 2023 16:30
Allow setting the H2 initial window size per SNI
@randall randall force-pushed the initial_window_size_in_per_sni branch from 93771bc to 8374759 Compare June 30, 2023 17:45
containing CA certs. The value for ``dir`` must be a file path for an OpenSSL
X509 hashed directory containing CA certs. If a given file path does not begin
with ``/`` , it must be relative to the |TS| configuration directory.
``verify_client_ca_certs`` can only be used with capabilities provided by
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the misspellings.

Copy link
Contributor

@bneradt bneradt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay in review. Looks great to me. Thank you for adding this. I could see why this would be helpful.

@randall randall merged commit 71a438d into apache:master Jul 5, 2023
@randall randall deleted the initial_window_size_in_per_sni branch July 5, 2023 17:02
zwoop added a commit that referenced this pull request Jul 6, 2023
* master:
  Build binaries for bigobj AuTest (#9965)
  Build regex_remap plugin in CMake build (#9963)
  Add libloader plugin to CMake build (#9953)
  Build TSVConnFD.so and place in correct directory (#9956)
  Add parser.cc to header_rewrite sources in CMake (#9957)
  Build traffic_crashlog executable in CMake build (#9958)
  Build s3_auth plugin in CMake build (#9962)
  Remove obsolete ink_inet_addr function. (#9948)
  more tests for checking rule enforcement (#9936)
  Build prefetch plugin and test in CMake build (#9954)
  Check CMakeLists.txt files in RAT (#9955)
  Converts core ats:: namespace to ts:: (#9950)
  Clean up: Remove mgmt/api folder. This was unused code, functions were (#9945)
  sni: new hint 'http2_initial_window_size_in' (#9734)
  build_h3_tools: update for lib64 (#9946)
  OpenSSL 3.0 tls autest updates (#9947)
bneradt added a commit to bneradt/trafficserver that referenced this pull request Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments