fix error value when opening a new stream

Kriechi · Kriechi · commit 500a3488d492 · 2025-02-22T17:40:22.000+01:00
diff --git a/src/h2/connection.py b/src/h2/connection.py
@@ -793,8 +793,9 @@ def send_headers(self,
         # Check we can open the stream.
         if stream_id not in self.streams:
             max_open_streams = self.remote_settings.max_concurrent_streams
-            if (self.open_outbound_streams + 1) > max_open_streams:
-                msg = f"Max outbound streams is {max_open_streams}, {self.open_outbound_streams} open"
+            value = self.open_outbound_streams # take a copy due to the property accessor having side affects
+            if (value + 1) > max_open_streams:
+                msg = f"Max outbound streams is {max_open_streams}, {value} open"
                 raise TooManyStreamsError(msg)
 
         self.state_machine.process_input(ConnectionInputs.SEND_HEADERS)
@@ -1593,8 +1594,9 @@ def _receive_headers_frame(self, frame: HeadersFrame) -> tuple[list[Frame], list
         # stream ID is valid.
         if frame.stream_id not in self.streams:
             max_open_streams = self.local_settings.max_concurrent_streams
-            if (self.open_inbound_streams + 1) > max_open_streams:
-                msg = f"Max outbound streams is {max_open_streams}, {self.open_outbound_streams} open"
+            value = self.open_inbound_streams # take a copy due to the property accessor having side affects
+            if (value + 1) > max_open_streams:
+                msg = f"Max inbound streams is {max_open_streams}, {value} open"
                 raise TooManyStreamsError(msg)
 
         # Let's decode the headers. We handle headers as bytes internally up
