(trafficserver) branch master updated: Fixes sni.yaml-configured initial window size (#12531)

[rrm]

This is an automated email from the ASF dual-hosted git repository.

rrm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 7f1c000bd8 Fixes sni.yaml-configured initial window size (#12531)
7f1c000bd8 is described below

commit 7f1c000bd89e690df21debc01fecab7df6f651d7
Author: Randall Meyer <r...@apache.org>
AuthorDate: Sat Oct 4 14:07:03 2025 -0700

    Fixes sni.yaml-configured initial window size (#12531)
    
    Without this fix, the sni.yaml configured http2_initial_window size is
    never sent to the client.
---
 src/proxy/http2/Http2ConnectionState.cc | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/proxy/http2/Http2ConnectionState.cc 
b/src/proxy/http2/Http2ConnectionState.cc
index 49e9cf8be7..4b38579574 100644
--- a/src/proxy/http2/Http2ConnectionState.cc
+++ b/src/proxy/http2/Http2ConnectionState.cc
@@ -1369,10 +1369,7 @@ Http2ConnectionState::send_connection_preface()
   configured_settings.set(HTTP2_SETTINGS_MAX_CONCURRENT_STREAMS, 
_adjust_concurrent_stream());
 
   uint32_t const configured_initial_window_size = 
this->_get_configured_receive_session_window_size();
-  if (this->_has_dynamic_stream_window()) {
-    // Since this is the beginning of the connection and there are no streams
-    // yet, we can just set the stream window size to fill the entire session
-    // window size.
+  if (configured_initial_window_size > HTTP2_INITIAL_WINDOW_SIZE) {
     configured_settings.set(HTTP2_SETTINGS_INITIAL_WINDOW_SIZE, 
configured_initial_window_size);
   }