protocol-http2 0.19.2 → 0.19.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe5de9c523fbd986f47d3961f13d58e225f165f36ed322026d72ddbe95357c46
-  data.tar.gz: 04d1c6d47af4784c07b61d403efb040153b57b37d88c005f50cd0a6850d6e0f9
+  metadata.gz: b04111afb8bcb89c8866ccd13f1782ab2cd9ccf438210d5da17aecb384d2cf02
+  data.tar.gz: 7574a7a9595487e5ad574250e98a6662e5d2c588acffe8536b9d4f642cabfc4b
 SHA512:
-  metadata.gz: 0d869c91b0736f7eaaa4a57708c9ee8e060813df94d536f94de08844e68b78f7f72a1f5ecd860f1d392c7a1a49ed6a7c573084cb462f795c63c5b58357a38bc8
-  data.tar.gz: 1b9f8c945a2a8befaf2c07040dcd269637d304461e37732f22f1f5cafcea2bc3637f3ee32141a8d53d1be74bcaefb50a33d471a2f136e42aec2673e55518bbbf
+  metadata.gz: ec243f9398ad359deb1ca97341d6fca6c563158a40fa7e3617b39af599bb0ebce8c302ef7af3a378b944d1b8ada1689eff8f540e5c274a065fcd2c8420f531af
+  data.tar.gz: '08f92272a802133333232cac14b26568fdc5e95b4c0054caec717de403e0a5e22d41d3c2aa57ec08af40de11a75e4f2bb508d1b7b4c480513da81d352d58c072'

data/lib/protocol/http2/connection.rb

@@ -215,20 +215,18 @@ module Protocol
 			def write_frame(frame)
 				synchronize do
 					@framer.write_frame(frame)
-					
-					# I tried moving this outside the synchronize block but it caused a deadlock.
-					@framer.flush
 				end
+				
+				@framer.flush
 			end
 			
 			def write_frames
 				if @framer
 					synchronize do
 						yield @framer
-						
-						# See note above.
-						@framer.flush
 					end
+					
+					@framer.flush
 				else
 					raise EOFError, "Connection closed!"
 				end
@@ -476,7 +474,7 @@ module Protocol
 				# Return if there is no window to consume:
 				return unless size > 0
 				
-				# Console.logger.debug(self) do |buffer|
+				# Console.info(self) do |buffer|
 				# 	@dependencies.each do |id, dependency|
 				# 		buffer.puts "- #{dependency}"
 				# 	end

data/lib/protocol/http2/framer.rb

@@ -16,6 +16,8 @@ require_relative "goaway_frame"
 require_relative "window_update_frame"
 require_relative "continuation_frame"
 
+require "traces/provider"
+
 module Protocol
 	module HTTP2
 		# HTTP/2 frame type mapping as defined by the spec
@@ -107,6 +109,50 @@ module Protocol
 				
 				raise EOFError, "Could not read frame header!"
 			end
+			
+			Traces::Provider(self) do
+				def write_connection_preface
+					Traces.trace("protocol.http2.framer.write_connection_preface") do
+						super
+					end
+				end
+				
+				def read_connection_preface
+					Traces.trace("protocol.http2.framer.read_connection_preface") do
+						super
+					end
+				end
+				
+				def write_frame(frame)
+					attributes = {
+						"frame.length" => frame.length,
+						"frame.type" => frame.type,
+						"frame.flags" => frame.flags,
+						"frame.stream_id" => frame.stream_id,
+					}
+					
+					Traces.trace("protocol.http2.framer.write_frame", attributes: attributes) do
+						super
+					end
+				end
+				
+				def read_frame(maximum_frame_size = MAXIMUM_ALLOWED_FRAME_SIZE)
+					Traces.trace("protocol.http2.framer.read_frame") do |span|
+						super.tap do |frame|
+							span["frame.length"] = frame.length
+							span["frame.type"] = frame.type
+							span["frame.flags"] = frame.flags
+							span["frame.stream_id"] = frame.stream_id
+						end
+					end
+				end
+				
+				def flush
+					Traces.trace("protocol.http2.framer.flush") do
+						super
+					end
+				end
+			end
 		end
 	end
 end

data/lib/protocol/http2/stream.rb

@@ -275,7 +275,7 @@ module Protocol
 			end
 			
 			protected def ignore_headers(frame)
-				# Async.logger.warn(self) {"Received headers in state: #{@state}!"}
+				# Console.warn(self) {"Received headers in state: #{@state}!"}
 			end
 			
 			def receive_headers(frame)
@@ -316,7 +316,7 @@ module Protocol
 			end
 			
 			def ignore_data(frame)
-				# Async.logger.warn(self) {"Received headers in state: #{@state}!"}
+				# Console.warn(self) {"Received headers in state: #{@state}!"}
 			end
 			
 			# DATA frames are subject to flow control and can only be sent when a stream is in the "open" or "half-closed (remote)" state.  The entire DATA frame payload is included in flow control, including the Pad Length and Padding fields if present.  If a DATA frame is received whose stream is not in "open" or "half-closed (local)" state, the recipient MUST respond with a stream error of type STREAM_CLOSED.

data/lib/protocol/http2/version.rb

@@ -5,6 +5,6 @@
 
 module Protocol
 	module HTTP2
-		VERSION = "0.19.2"
+		VERSION = "0.19.4"
 	end
 end

data/lib/protocol/http2/window.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 # Released under the MIT License.
-# Copyright, 2019-2024, by Samuel Williams.
+# Copyright, 2024, by Samuel Williams.
 
 module Protocol
 	module HTTP2
@@ -72,8 +72,10 @@ module Protocol
 			end
 			
 			def inspect
-				"\#<#{self.class} used=#{@used} available=#{@available} capacity=#{@capacity}>"
+				"\#<#{self.class} available=#{@available} used=#{@used} capacity=#{@capacity}#{limited? ? " limited" : nil}>"
 			end
+			
+			alias to_s inspect
 		end
 		
 		# This is a window which efficiently maintains a desired capacity.
@@ -108,7 +110,7 @@ module Protocol
 			end
 			
 			def inspect
-				"\#<#{self.class} used=#{@used} available=#{@available} capacity=#{@capacity} desired=#{@desired}>"
+				"\#<#{self.class} available=#{@available} used=#{@used} capacity=#{@capacity} desired=#{@desired} #{limited? ? "limited" : nil}>"
 			end
 		end
 	end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: protocol-http2
 version: !ruby/object:Gem::Version
-  version: 0.19.2
+  version: 0.19.4
 platform: ruby
 authors:
 - Samuel Williams
@@ -40,7 +40,7 @@ cert_chain:
   Q2K9NVun/S785AP05vKkXZEFYxqG6EW012U4oLcFl5MySFajYXRYbuUpH6AY+HP8
   voD0MPg1DssDLKwXyt1eKD/+Fq0bFWhwVM/1XiAXL7lyYUyOq24KHgQ2Csg=
   -----END CERTIFICATE-----
-date: 2024-09-28 00:00:00.000000000 Z
+date: 2024-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: protocol-hpack
@@ -70,6 +70,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.18'
+- !ruby/object:Gem::Dependency
+  name: traces
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description:
 email:
 executables: []