httpx 0.23.2 → 0.23.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1198b505a01534dd942900f5cd82654459fbab18e89e8c4e3291717d36cfa777
-  data.tar.gz: 02c767977043d0d80faa74d1175de23931ab5bb097fd11004df145fd1e3c2216
+  metadata.gz: e3869ae05ccfa2903477391c3ed918d9022ebd628795a09ad9681511a2d46f77
+  data.tar.gz: c7d296cdbaed94327582b409463677ad549038d5785d8e296e837c243183bf43
 SHA512:
-  metadata.gz: 0e8bfbf580750b9fa04fd116e9a1a2b341ed488771aa10021c496d98412b0d9bac87ee5d4963d48648b8df2f85445060953e43548bb0ffab7a8f126001365aad
-  data.tar.gz: 49297a78674a918c0cd3e10e40748bad72347fd2e4e3df9df43f16b0e179238a7271d64306e2db4eefa2160e38e8ee2f5768c9da2f2f8cebc1c1db97b8fc0821
+  metadata.gz: 80b74ee360f1db9632a0e1317e3a3ed5ae3ac863e5c6c637e4ba85719027e68bf1e3fcc2103b35b1fc11221bd832af911d985a7dd51eb9030cd9392bf310b591
+  data.tar.gz: b424d2a3e8c796cd04747bd446a06ee690f1fa5965f397b4e7ab98880ca2d34f97d8846ecccf6973b42351b15543d610dac6ca8217ba68fb53388610071e17e1

data/doc/release_notes/0_23_3.md

@@ -0,0 +1,6 @@
+# 0.23.3
+
+## Bugfixes
+
+* native resolver: fix missing exception variable in the DNS error code path.
+* native resolver: fixed short DNS packet handling when using TCP.

data/doc/release_notes/0_23_4.md

@@ -0,0 +1,5 @@
+# 0.23.4
+
+## Bugfixes
+
+* fix `Response::Body#read` which rewinds on every call.

data/lib/httpx/plugins/response_cache/file_store.rb

@@ -0,0 +1,34 @@
+# # frozen_string_literal: true
+
+# require "mutex_m"
+# require "forwardable"
+
+# module HTTPX::Plugins
+#   module ResponseCache
+#     class FileStore < Store
+#       def initialize(dir = Dir.tmpdir)
+#         @dir = Pathname.new(dir)
+#         @store = {}
+#         @hash_store.extend(Mutex_m)
+#       end
+
+#       def clear
+#         @store.synchronize { @store.clear }
+#       end
+
+#       def _get(request)
+#         return unless File.exist?(@dir.join(request.response_cache_key))
+
+#         @store.synchronize { @store[key] }
+#       end
+
+#       def set(key)
+#         @store.synchronize do
+#           resps = (@store[key] ||= [])
+
+#           yield resps
+#         end
+#       end
+#     end
+#   end
+# end

data/lib/httpx/resolver/native.rb

@@ -198,15 +198,16 @@ module HTTPX
             return
           else
             size = @read_buffer[0, 2].unpack1("n")
+            buffer = @read_buffer.byteslice(2..-1)
 
             if size > @read_buffer.bytesize
               # only do buffer logic if it's worth it, and the whole packet isn't here already
               @large_packet = Buffer.new(size)
-              @large_packet << @read_buffer.byteslice(2..-1)
+              @large_packet << buffer
 
               next
             else
-              parse(@read_buffer)
+              parse(buffer)
             end
           end
         else # udp
@@ -266,7 +267,6 @@ module HTTPX
         reset_hostname(hostname)
         @connections.delete(connection)
         ex = NativeResolveError.new(connection, connection.origin.host, "unknown DNS error (error code #{result})")
-        ex.set_backtrace(e.backtrace)
         raise ex
       when :decode_error
         hostname, connection = @queries.first

data/lib/httpx/response.rb

@@ -107,6 +107,8 @@ module HTTPX
 
       raise Error, "no decoder available for \"#{transcoder}\"" unless decoder
 
+      @body.rewind
+
       decoder.call(self, *args)
     end
 
@@ -152,9 +154,12 @@ module HTTPX
       def read(*args)
         return unless @buffer
 
-        rewind
+        unless @reader
+          rewind
+          @reader = @buffer
+        end
 
-        @buffer.read(*args)
+        @reader.read(*args)
       end
 
       def bytesize
@@ -249,14 +254,17 @@ module HTTPX
       end
       # :nocov:
 
-      private
-
       def rewind
         return unless @buffer
 
+        # in case there's some reading going on
+        @reader = nil
+
         @buffer.rewind
       end
 
+      private
+
       def transition
         case @state
         when :idle

data/lib/httpx/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HTTPX
-  VERSION = "0.23.2"
+  VERSION = "0.23.4"
 end

data/sig/response.rbs

@@ -58,6 +58,7 @@ module HTTPX
       @window_size: Integer
       @length: Integer
       @buffer: StringIO | Tempfile | nil
+      @reader: StringIO | Tempfile | nil
 
       def write:(String chunk) -> Integer?
 
@@ -72,10 +73,11 @@ module HTTPX
       def close: () -> void
       def closed?: () -> bool
 
+      def rewind: () -> void
+
       private
 
-      def initialize: (Response, Options) -> untyped
-      def rewind: () -> void
+      def initialize: (Response, Options) -> void
       def transition: () -> void
       def _with_same_buffer_pos: [A] () { () -> A } -> A
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: httpx
 version: !ruby/object:Gem::Version
-  version: 0.23.2
+  version: 0.23.4
 platform: ruby
 authors:
 - Tiago Cardoso
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-05 00:00:00.000000000 Z
+date: 2023-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http-2-next
@@ -97,6 +97,8 @@ extra_rdoc_files:
 - doc/release_notes/0_23_0.md
 - doc/release_notes/0_23_1.md
 - doc/release_notes/0_23_2.md
+- doc/release_notes/0_23_3.md
+- doc/release_notes/0_23_4.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md
 - doc/release_notes/0_3_0.md
@@ -186,6 +188,8 @@ files:
 - doc/release_notes/0_23_0.md
 - doc/release_notes/0_23_1.md
 - doc/release_notes/0_23_2.md
+- doc/release_notes/0_23_3.md
+- doc/release_notes/0_23_4.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md
 - doc/release_notes/0_3_0.md
@@ -273,6 +277,7 @@ files:
 - lib/httpx/plugins/push_promise.rb
 - lib/httpx/plugins/rate_limiter.rb
 - lib/httpx/plugins/response_cache.rb
+- lib/httpx/plugins/response_cache/file_store.rb
 - lib/httpx/plugins/response_cache/store.rb
 - lib/httpx/plugins/retries.rb
 - lib/httpx/plugins/stream.rb