httpx 0.22.0 → 0.22.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 193a22f95b5e48159faf27f4bd060abae29fe5c22dcddc3c462177a57aff7a2c
-  data.tar.gz: 3574cb0a3392127275872186db0ef4cbef139573710a093d61c420544f359a7e
+  metadata.gz: 373c83e6252fb0439bfe307cacc18514741c9b8eaa1b9d055e1da59015c7a8f3
+  data.tar.gz: bc20567c842a6b6bd7d450fc6edd5ee1eb509940ffddab643a0c6a3b8ad48522
 SHA512:
-  metadata.gz: 8a72a33280f1d2bbe0a439e00bfdbcbcc70fef690e5d6cceb7f5f2a13bfec3db45e407733251bd03b47a02f138a96f059cefb1f35462a9dd049cc062c2b16c4e
-  data.tar.gz: cac069c954943fc756b81700c32e9a33eb76b4d27bd9e2ce29f83a9322c787012731b9f43298da64ea5a9f5df751405df209637fe9b069c4a49631a41abe9568
+  metadata.gz: ff4fd33b7614c410fb30b4916ad430fbf2091d43d6bec5dfa3784564a4824491baa89f33e491db6a4a76c5cf8f4e9e07e2d5192a1214ee0f8283e345b3b1387e
+  data.tar.gz: 4da51a3015ef7a7f32f8e24b7e14324acda8c6a735cd3038e0a7a224376c60775ed533ee936ba58ad51639adcc16906a8ef8b637d9ef84733ac4546ed4ddd7de

data/doc/release_notes/0_22_1.md

@@ -0,0 +1,11 @@
+# 0.22.1
+
+## Bugfixes
+
+* `:retries` plugin: fix `HTTPX::Response#response to point to last possible response in the redirection chain.
+* `:stream` plugin: Make `HTTPX::Session#request` public (as it is inn the main class) .
+* return 100 responses if the request didn't specifically ask for "100-continue" negotiation (via the "expect" header).
+
+## Improvements
+
+Wrap low-level socket errors in a `HTTPX::ConnectionError` exception.

data/doc/release_notes/0_22_2.md

@@ -0,0 +1,5 @@
+# 0.22.1
+
+## Chore
+
+Checking response class before calling `.status`, as this was being called in some places on error responses, thereby triggering the deprecation warning.

data/lib/httpx/adapters/faraday.rb

@@ -242,7 +242,8 @@ module Faraday
              Errno::EHOSTUNREACH,
              Errno::EINVAL,
              Errno::ENETUNREACH,
-             Errno::EPIPE => e
+             Errno::EPIPE,
+             ::HTTPX::ConnectionError => e
         raise CONNECTION_FAILED_ERROR, e
       end
 

data/lib/httpx/adapters/sentry.rb

@@ -63,7 +63,11 @@ module HTTPX::Plugins
 
         request_info = extract_request_info(req)
         sentry_span.set_description("#{request_info[:method]} #{request_info[:url]}")
-        sentry_span.set_data(:status, res.status)
+        if res.is_a?(HTTPX::ErrorResponse)
+          sentry_span.set_data(:error, res.message)
+        else
+          sentry_span.set_data(:status, res.status)
+        end
         sentry_span.set_timestamp(::Sentry.utc_now.to_f)
       end
 

data/lib/httpx/connection.rb

@@ -526,8 +526,14 @@ module HTTPX
            Errno::EHOSTUNREACH,
            Errno::EINVAL,
            Errno::ENETUNREACH,
-           Errno::EPIPE,
-           TLSError => e
+           Errno::EPIPE => e
+      # connect errors, exit gracefully
+      error = ConnectionError.new(e.message)
+      error.set_backtrace(e.backtrace)
+      handle_error(error)
+      @state = :closed
+      emit(:close)
+    rescue TLSError => e
       # connect errors, exit gracefully
       handle_error(e)
       @state = :closed

data/lib/httpx/errors.rb

@@ -5,6 +5,8 @@ module HTTPX
 
   class UnsupportedSchemeError < Error; end
 
+  class ConnectionError < Error; end
+
   class TimeoutError < Error
     attr_reader :timeout
 

data/lib/httpx/io/tcp.rb

@@ -74,8 +74,13 @@ module HTTPX
       try_connect
     rescue Errno::ECONNREFUSED,
            Errno::EADDRNOTAVAIL,
-           Errno::EHOSTUNREACH => e
-      raise e if @ip_index <= 0
+           Errno::EHOSTUNREACH,
+           SocketError => e
+      if @ip_index <= 0
+        error = ConnectionError.new(e.message)
+        error.set_backtrace(e.backtrace)
+        raise error
+      end
 
       log { "failed connecting to #{@ip} (#{e.message}), trying next..." }
       @ip_index -= 1

data/lib/httpx/plugins/digest_authentication.rb

@@ -46,6 +46,8 @@ module HTTPX
 
             probe_response = wrap { super(request).first }
 
+            return probe_response unless probe_response.is_a?(Response)
+
             if probe_response.status == 401 && digest.can_authenticate?(probe_response.headers["www-authenticate"])
               request.transition(:idle)
               request.headers["authorization"] = digest.authenticate(request, probe_response.headers["www-authenticate"])

data/lib/httpx/plugins/expect.rb

@@ -50,7 +50,8 @@ module HTTPX
         end
 
         def response=(response)
-          if response && response.status == 100 &&
+          if response.is_a?(Response) &&
+             response.status == 100 &&
              !@headers.key?("expect") &&
              (@state == :body || @state == :done)
 
@@ -92,7 +93,7 @@ module HTTPX
           response = @responses.delete(request)
           return unless response
 
-          if response.status == 417 && request.headers.key?("expect")
+          if response.is_a?(Response) && response.status == 417 && request.headers.key?("expect")
             response.close
             request.headers.delete("expect")
             request.transition(:idle)

data/lib/httpx/plugins/follow_redirects.rb

@@ -44,6 +44,7 @@ module HTTPX
 
           max_redirects = redirect_request.max_redirects
 
+          return response unless response.is_a?(Response)
           return response unless REDIRECT_STATUS.include?(response.status) && response.headers.key?("location")
           return response unless max_redirects.positive?
 
@@ -121,6 +122,12 @@ module HTTPX
           @redirect_request || self
         end
 
+        def response
+          return super unless @redirect_request
+
+          @redirect_request.response
+        end
+
         def max_redirects
           @options.max_redirects || MAX_REDIRECTS
         end

data/lib/httpx/plugins/ntlm_authentication.rb

@@ -39,6 +39,8 @@ module HTTPX
               request.headers["authorization"] = ntlm.negotiate
               probe_response = wrap { super(request).first }
 
+              return probe_response unless probe_response.is_a?(Response)
+
               if probe_response.status == 401 && ntlm.can_authenticate?(probe_response.headers["www-authenticate"])
                 request.transition(:idle)
                 request.headers["authorization"] = ntlm.authenticate(request, probe_response.headers["www-authenticate"])

data/lib/httpx/plugins/proxy/http.rb

@@ -23,6 +23,7 @@ module HTTPX
             response = super
 
             if response &&
+               response.is_a?(Response) &&
                response.status == 407 &&
                !request.headers.key?("proxy-authorization") &&
                response.headers.key?("proxy-authenticate")
@@ -113,13 +114,14 @@ module HTTPX
 
           def __http_on_connect(request, response)
             @inflight -= 1
-            if response.status == 200
+            if response.is_a?(Response) && response.status == 200
               req = @pending.first
               request_uri = req.uri
               @io = ProxySSL.new(@io, request_uri, @options)
               transition(:connected)
               throw(:called)
-            elsif response.status == 407 &&
+            elsif response.is_a?(Response) &&
+                  response.status == 407 &&
                   !request.headers.key?("proxy-authorization") &&
                   @options.proxy.can_authenticate?(response.headers["proxy-authenticate"])
 

data/lib/httpx/plugins/rate_limiter.rb

@@ -23,6 +23,8 @@ module HTTPX
         end
 
         def retry_on_rate_limited_response(response)
+          return false unless response.is_a?(Response)
+
           status = response.status
 
           RATE_LIMIT_CODES.include?(status)

data/lib/httpx/plugins/retries.rb

@@ -23,6 +23,7 @@ module HTTPX
         Parser::Error,
         TLSError,
         TimeoutError,
+        ConnectionError,
         Connection::HTTP2::GoawayError,
       ].freeze
       DEFAULT_JITTER = ->(interval) { interval * (0.5 * (1 + rand)) }

data/lib/httpx/plugins/stream.rb

@@ -95,8 +95,6 @@ module HTTPX
     #
     module Stream
       module InstanceMethods
-        private
-
         def request(*args, stream: false, **options)
           return super(*args, **options) unless stream
 

data/lib/httpx/plugins/upgrade.rb

@@ -35,7 +35,9 @@ module HTTPX
           response = super
 
           if response
-            return response unless response.respond_to?(:headers) && response.headers.key?("upgrade")
+            return response unless response.is_a?(Response)
+
+            return response unless response.headers.key?("upgrade")
 
             upgrade_protocol = response.headers["upgrade"].split(/ *, */).first
 

data/lib/httpx/plugins/webdav.rb

@@ -32,6 +32,8 @@ module HTTPX
                 "</D:lockinfo>"
           response = request(:lock, path, headers: headers, xml: xml)
 
+          return response unless response.is_a?(Response)
+
           return response unless blk && response.status == 200
 
           lock_token = response.headers["lock-token"]

data/lib/httpx/request.rb

@@ -86,7 +86,7 @@ module HTTPX
     def response=(response)
       return unless response
 
-      if response.is_a?(Response) && response.status == 100
+      if response.is_a?(Response) && response.status == 100 && @headers.key?("expect")
         @informational_status = response.status
         return
       end

data/lib/httpx/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: httpx
 version: !ruby/object:Gem::Version
-  version: 0.22.0
+  version: 0.22.2
 platform: ruby
 authors:
 - Tiago Cardoso
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-12-21 00:00:00.000000000 Z
+date: 2022-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http-2-next
@@ -89,6 +89,8 @@ extra_rdoc_files:
 - doc/release_notes/0_21_0.md
 - doc/release_notes/0_21_1.md
 - doc/release_notes/0_22_0.md
+- doc/release_notes/0_22_1.md
+- doc/release_notes/0_22_2.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md
 - doc/release_notes/0_3_0.md
@@ -170,6 +172,8 @@ files:
 - doc/release_notes/0_21_0.md
 - doc/release_notes/0_21_1.md
 - doc/release_notes/0_22_0.md
+- doc/release_notes/0_22_1.md
+- doc/release_notes/0_22_2.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md
 - doc/release_notes/0_3_0.md