httpx 0.19.7 → 0.19.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ca4deb7487ed711d1241a9d332fde1fbb5492d58497c26b4022aa255d3b0788b
-  data.tar.gz: 594ad514fff0cb9e98e9464b2091626583ecdd2f3f7b0754e2e5c14734560708
+  metadata.gz: 05a379db82c9c86f84a847680f4a784ed62a4cbcb8ce8d09a3d3c29abeb7d5d1
+  data.tar.gz: 5046f5d1e86f7d5c523a171b23fd5fb8126b63636a021c204a7c33a4ebf5e319
 SHA512:
-  metadata.gz: c852596dbab3faaf2c1849fd16c933879a452d82425fdbf398d289c4968f56a3e681f5ef25a5f64d5db2ed12a7f47469ce0bb3052e231514372db8d528e86e56
-  data.tar.gz: f79a5abca5e1bd3b26e729e69977e0324c1434cd740cdd177c8d058c6ac50f97c60a81e0d3cee50d336811b3c03a0f527e987bea423667ce8ddb42ed72db2f00
+  metadata.gz: d62d606f0688c485f79a38feed6b94841005fb6a70c5ab934c7b70cb7a7958066b85a496405959a7660da3f12f194d844f10660621ce6fb98c7aa898c38d855c
+  data.tar.gz: 42b46c44c4c864222f14235534be31db4eb5155716c542457178be010b227fdb391c095b009759e9c237d6320da7133123334cfc4b3d4dd6940ff58aa6df9f0e

data/doc/release_notes/0_19_8.md

@@ -0,0 +1,5 @@
+# 0.19.7
+
+## Bugfixes
+
+* `datadog` went v1, which broke `httpx` adapter. Now it supports both 1.0 and 0.x versions.

data/lib/httpx/adapters/datadog.rb

@@ -1,12 +1,52 @@
 # frozen_string_literal: true
 
-require "ddtrace/contrib/integration"
-require "ddtrace/contrib/rest_client/configuration/settings"
-require "ddtrace/contrib/rest_client/patcher"
+if defined?(::DDTrace) && ::DDTrace::VERSION::STRING >= "1.0.0"
+  require "datadog/tracing/contrib/integration"
+  require "datadog/tracing/contrib/configuration/settings"
+  require "datadog/tracing/contrib/patcher"
 
-module Datadog
+  TRACING_MODULE = Datadog::Tracing
+else
+
+  require "ddtrace/contrib/integration"
+  require "ddtrace/contrib/configuration/settings"
+  require "ddtrace/contrib/patcher"
+
+  TRACING_MODULE = Datadog
+end
+
+module TRACING_MODULE # rubocop:disable Naming/ClassAndModuleCamelCase
   module Contrib
     module HTTPX
+      if defined?(::DDTrace) && ::DDTrace::VERSION::STRING >= "1.0.0"
+        METADATA_MODULE = TRACING_MODULE::Metadata
+
+        TYPE_OUTBOUND = TRACING_MODULE::Metadata::Ext::HTTP::TYPE_OUTBOUND
+
+        TAG_PEER_SERVICE = TRACING_MODULE::Metadata::Ext::TAG_PEER_SERVICE
+
+        TAG_URL = TRACING_MODULE::Metadata::Ext::HTTP::TAG_URL
+        TAG_METHOD = TRACING_MODULE::Metadata::Ext::HTTP::TAG_METHOD
+        TAG_TARGET_HOST = TRACING_MODULE::Metadata::Ext::NET::TAG_TARGET_HOST
+        TAG_TARGET_PORT = TRACING_MODULE::Metadata::Ext::NET::TAG_TARGET_PORT
+
+        TAG_STATUS_CODE = TRACING_MODULE::Metadata::Ext::HTTP::TAG_STATUS_CODE
+
+      else
+
+        METADATA_MODULE = Datadog
+
+        TYPE_OUTBOUND = TRACING_MODULE::Ext::HTTP::TYPE_OUTBOUND
+        TAG_PEER_SERVICE = TRACING_MODULE::Ext::Integration::TAG_PEER_SERVICE
+        TAG_URL = TRACING_MODULE::Ext::HTTP::URL
+        TAG_METHOD = TRACING_MODULE::Ext::HTTP::METHOD
+        TAG_TARGET_HOST = TRACING_MODULE::Ext::NET::TARGET_HOST
+        TAG_TARGET_PORT = TRACING_MODULE::Ext::NET::TARGET_PORT
+        TAG_STATUS_CODE = Datadog::Ext::HTTP::STATUS_CODE
+        PROPAGATOR = TRACING_MODULE::HTTPPropagator
+
+      end
+
       # HTTPX Datadog Plugin
       #
       # Enables tracing for httpx requests. A span will be created for each individual requests,
@@ -15,6 +55,8 @@ module Datadog
       #
       module Plugin
         class RequestTracer
+          include Contrib::HttpAnnotationHelper
+
           SPAN_REQUEST = "httpx.request"
 
           def initialize(request)
@@ -22,43 +64,37 @@ module Datadog
           end
 
           def call
-            return if skip_tracing?
+            return unless tracing_enabled?
 
             @request.on(:response, &method(:finish))
 
             verb = @request.verb.to_s.upcase
             uri = @request.uri
 
-            @span = datadog_pin.tracer.trace(SPAN_REQUEST)
-            service_name = datadog_config[:split_by_domain] ? uri.host : datadog_pin.service_name
+            @span = build_span
 
-            begin
-              @span.service = service_name
-              @span.span_type = Datadog::Ext::HTTP::TYPE_OUTBOUND
-              @span.resource = verb
+            @span.resource = verb
 
-              Datadog::HTTPPropagator.inject!(@span.context, @request.headers) if datadog_pin.tracer.enabled && !skip_distributed_tracing?
+            # Add additional request specific tags to the span.
 
-              # Add additional request specific tags to the span.
+            @span.set_tag(TAG_URL, @request.path)
+            @span.set_tag(TAG_METHOD, verb)
 
-              @span.set_tag(Datadog::Ext::HTTP::URL, @request.path)
-              @span.set_tag(Datadog::Ext::HTTP::METHOD, verb)
+            @span.set_tag(TAG_TARGET_HOST, uri.host)
+            @span.set_tag(TAG_TARGET_PORT, uri.port.to_s)
 
-              @span.set_tag(Datadog::Ext::NET::TARGET_HOST, uri.host)
-              @span.set_tag(Datadog::Ext::NET::TARGET_PORT, uri.port.to_s)
+            # Tag as an external peer service
+            @span.set_tag(TAG_PEER_SERVICE, @span.service)
 
-              # Tag as an external peer service
-              @span.set_tag(Datadog::Ext::Integration::TAG_PEER_SERVICE, @span.service)
+            propagate_headers if @configuration[:distributed_tracing]
 
-              # Set analytics sample rate
-              if Contrib::Analytics.enabled?(datadog_config[:analytics_enabled])
-                Contrib::Analytics.set_sample_rate(@span, datadog_config[:analytics_sample_rate])
-              end
-            rescue StandardError => e
-              Datadog.logger.error("error preparing span for http request: #{e}")
+            # Set analytics sample rate
+            if Contrib::Analytics.enabled?(@configuration[:analytics_enabled])
+              Contrib::Analytics.set_sample_rate(@span, @configuration[:analytics_sample_rate])
             end
           rescue StandardError => e
-            Datadog.logger.debug("Failed to start span: #{e}")
+            Datadog.logger.error("error preparing span for http request: #{e}")
+            Datadog.logger.error(e.backtrace)
           end
 
           def finish(response)
@@ -67,7 +103,7 @@ module Datadog
             if response.is_a?(::HTTPX::ErrorResponse)
               @span.set_error(response.error)
             else
-              @span.set_tag(Datadog::Ext::HTTP::STATUS_CODE, response.status.to_s)
+              @span.set_tag(TAG_STATUS_CODE, response.status.to_s)
 
               @span.set_error(::HTTPX::HTTPError.new(response)) if response.status >= 400 && response.status <= 599
             end
@@ -77,40 +113,48 @@ module Datadog
 
           private
 
-          def skip_tracing?
-            return true if @request.headers.key?(Datadog::Ext::Transport::HTTP::HEADER_META_TRACER_VERSION)
-
-            return false unless @datadog_pin
-
-            span = @datadog_pin.tracer.active_span
+          if defined?(::DDTrace) && ::DDTrace::VERSION::STRING >= "1.0.0"
 
-            return true if span && (span.name == SPAN_REQUEST)
+            def build_span
+              TRACING_MODULE.trace(
+                SPAN_REQUEST,
+                service: service_name(@request.uri.host, configuration, Datadog.configuration_for(self)),
+                span_type: TYPE_OUTBOUND
+              )
+            end
 
-            false
-          end
+            def propagate_headers
+              TRACING_MODULE::Propagation::HTTP.inject!(TRACING_MODULE.active_trace, @request.headers)
+            end
 
-          def skip_distributed_tracing?
-            return !datadog_pin.config[:distributed_tracing] if datadog_pin.config && datadog_pin.config.key?(:distributed_tracing)
+            def configuration
+              @configuration ||= Datadog.configuration.tracing[:httpx, @request.uri.host]
+            end
 
-            !Datadog.configuration[:httpx][:distributed_tracing]
-          end
+            def tracing_enabled?
+              TRACING_MODULE.enabled?
+            end
+          else
+            def build_span
+              service_name = configuration[:split_by_domain] ? @request.uri.host : configuration[:service_name]
+              configuration[:tracer].trace(
+                SPAN_REQUEST,
+                service: service_name,
+                span_type: TYPE_OUTBOUND
+              )
+            end
 
-          def datadog_pin
-            @datadog_pin ||= begin
-              service = datadog_config[:service_name]
-              tracer = datadog_config[:tracer]
+            def propagate_headers
+              Datadog::HTTPPropagator.inject!(@span.context, @request.headers)
+            end
 
-              Datadog::Pin.new(
-                service,
-                app: "httpx",
-                app_type: Datadog::Ext::AppTypes::WEB,
-                tracer: -> { tracer }
-              )
+            def configuration
+              @configuration ||= Datadog.configuration[:httpx, @request.uri.host]
             end
-          end
 
-          def datadog_config
-            @datadog_config ||= Datadog.configuration[:httpx, @request.uri.host]
+            def tracing_enabled?
+              configuration[:tracer].enabled
+            end
           end
         end
 
@@ -125,7 +169,11 @@ module Datadog
       module Configuration
         # Default settings for httpx
         #
-        class Settings < Datadog::Contrib::Configuration::Settings
+        class Settings < TRACING_MODULE::Contrib::Configuration::Settings
+          DEFAULT_ERROR_HANDLER = lambda do |response|
+            Datadog::Ext::HTTP::ERROR_RANGE.cover?(response.status)
+          end
+
           option :service_name, default: "httpx"
           option :distributed_tracing, default: true
           option :split_by_domain, default: false
@@ -145,14 +193,14 @@ module Datadog
             o.lazy
           end
 
-          option :error_handler, default: Datadog::Tracer::DEFAULT_ON_ERROR
+          option :error_handler, default: DEFAULT_ERROR_HANDLER
         end
       end
 
       # Patcher enables patching of 'httpx' with datadog components.
       #
       module Patcher
-        include Datadog::Contrib::Patcher
+        include TRACING_MODULE::Contrib::Patcher
 
         module_function
 
@@ -192,8 +240,14 @@ module Datadog
           super && version >= MINIMUM_VERSION
         end
 
-        def default_configuration
-          Configuration::Settings.new
+        if defined?(::DDTrace) && ::DDTrace::VERSION::STRING >= "1.0.0"
+          def new_configuration
+            Configuration::Settings.new
+          end
+        else
+          def default_configuration
+            Configuration::Settings.new
+          end
         end
 
         def patcher

data/lib/httpx/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: httpx
 version: !ruby/object:Gem::Version
-  version: 0.19.7
+  version: 0.19.8
 platform: ruby
 authors:
 - Tiago Cardoso
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-04-08 00:00:00.000000000 Z
+date: 2022-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http-2-next
@@ -78,6 +78,7 @@ extra_rdoc_files:
 - doc/release_notes/0_19_5.md
 - doc/release_notes/0_19_6.md
 - doc/release_notes/0_19_7.md
+- doc/release_notes/0_19_8.md
 - doc/release_notes/0_1_0.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md
@@ -149,6 +150,7 @@ files:
 - doc/release_notes/0_19_5.md
 - doc/release_notes/0_19_6.md
 - doc/release_notes/0_19_7.md
+- doc/release_notes/0_19_8.md
 - doc/release_notes/0_1_0.md
 - doc/release_notes/0_2_0.md
 - doc/release_notes/0_2_1.md