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

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

commit 7d1d689a2fdd273bc53c007c063a41dd6c5be594
Author: Randall Meyer <[email protected]>
AuthorDate: Wed Oct 1 16:21:56 2025 -0700

    Fixes sni.yaml-configured initial window size
    
    Without this fix, the sni.yaml configured http2_initial_window size is
    never sent to the client.
    
    (cherry picked from commit 3c4f97cd7a86b372631aa022c67d6da241df1423)
---
 src/proxy/http2/Http2ConnectionState.cc | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/proxy/http2/Http2ConnectionState.cc 
b/src/proxy/http2/Http2ConnectionState.cc
index 2bebac321e..4f2b37b2bf 100644
--- a/src/proxy/http2/Http2ConnectionState.cc
+++ b/src/proxy/http2/Http2ConnectionState.cc
@@ -1362,10 +1362,11 @@ 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.
+
+  // 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);
   }
 

Reply via email to