datadog 2.36.0 → 2.40.0

data/lib/datadog/symbol_database/file_hash.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-require 'digest/sha1'
+require "digest/sha1"
+require_relative "../di/fatal_exceptions"
 
 module Datadog
   module SymbolDatabase
@@ -29,7 +30,7 @@ module Datadog
         return nil unless file_path
         return nil unless File.exist?(file_path)
 
-        content = File.read(file_path, mode: 'rb')
+        content = File.read(file_path, mode: "rb")
         size = content.bytesize
         git_blob = "blob #{size}\0#{content}"
 
@@ -37,7 +38,8 @@ module Datadog
         # This is not a security vulnerability - we're computing file content hashes
         # to match against Git objects, not using SHA-1 for authentication/integrity.
         Digest::SHA1.hexdigest(git_blob)  # nosemgrep: ruby.lang.security.weak-hashes-sha1.weak-hashes-sha1
-      rescue => e
+      rescue Exception => e # standard:disable Lint/RescueException
+        Datadog::DI.reraise_if_fatal(e)
         logger.debug { "symdb: file hash failed for #{file_path}: #{e.class}: #{e.message}" }
         nil
       end

data/lib/datadog/symbol_database/logger.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'forwardable'
+require "forwardable"
 
 module Datadog
   module SymbolDatabase

data/lib/datadog/symbol_database/remote.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "../di/fatal_exceptions"
+
 module Datadog
   module SymbolDatabase
     # Provides remote configuration integration for symbol database.
@@ -22,7 +24,7 @@ module Datadog
     #
     # @api private
     module Remote
-      PRODUCT = 'LIVE_DEBUGGING_SYMBOL_DB'
+      PRODUCT = "LIVE_DEBUGGING_SYMBOL_DB"
 
       class << self
         # Declare products this receiver handles.
@@ -37,6 +39,21 @@ module Datadog
           []
         end
 
+        # Symbol Database follows DI when its enabled setting is nil — the
+        # default, or an explicit c.symbol_database.enabled = nil. In that
+        # follow-DI case the product is deferred and added/removed alongside DI
+        # so it is never advertised while DI is inactive. An explicit true/false
+        # is independent and manages its own product at startup.
+        def deferred_products(settings)
+          if settings.respond_to?(:symbol_database) &&
+              settings.symbol_database.enabled.nil? &&
+              Datadog::SymbolDatabase.supported_runtime?
+            [PRODUCT]
+          else
+            []
+          end
+        end
+
         # Create receivers for remote configuration.
         # @return [Array<Receiver>] Array of receivers
         def receivers(_telemetry)
@@ -44,9 +61,10 @@ module Datadog
             telemetry = lookup_telemetry
             component = begin
               Datadog.send(:components, allow_initialization: false)&.symbol_database
-            rescue => e
+            rescue Exception => e # standard:disable Lint/RescueException
+              Datadog::DI.reraise_if_fatal(e)
               Datadog.logger.debug { "symdb: failed to look up component in RC receiver: #{e.class}: #{e.message}" }
-              telemetry&.report(e, description: 'symdb: failed to look up component in RC receiver')
+              telemetry&.report(e, description: "symdb: failed to look up component in RC receiver")
               nil
             end
 
@@ -76,7 +94,8 @@ module Datadog
         # @api private
         def lookup_telemetry
           Datadog.send(:components, allow_initialization: false)&.telemetry
-        rescue
+        rescue Exception => e # standard:disable Lint/RescueException
+          Datadog::DI.reraise_if_fatal(e)
           nil
         end
 
@@ -107,9 +126,10 @@ module Datadog
             # the Repository::Change union type where `Deleted` lacks `content`.
             change.content.errored("Unrecognized change type: #{change.type}") if change.respond_to?(:content) # steep:ignore NoMethod
           end
-        rescue => e
+        rescue Exception => e # standard:disable Lint/RescueException
+          Datadog::DI.reraise_if_fatal(e)
           component.logger.debug { "symdb: error processing remote config change: #{e.class}: #{e.message}" }
-          telemetry&.report(e, description: 'symdb: error processing remote config change')
+          telemetry&.report(e, description: "symdb: error processing remote config change")
           # Rescue runs regardless of which branch raised — Steep cannot narrow the
           # union type from a respond_to? check.
           content_obj = change.respond_to?(:content) ? change.content : change.previous # steep:ignore NoMethod
@@ -128,7 +148,7 @@ module Datadog
             return
           end
 
-          if config['upload_symbols']
+          if config["upload_symbols"]
             component.logger.debug { "symdb: upload enabled via remote config" }
             component.start_upload
           else
@@ -162,7 +182,7 @@ module Datadog
             return nil
           end
 
-          unless config.key?('upload_symbols')
+          unless config.key?("upload_symbols")
             logger.debug { "symdb: missing 'upload_symbols' key in config" }
             return nil
           end

data/lib/datadog/symbol_database/scope.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'json'
+require "json"
 
 module Datadog
   module SymbolDatabase
@@ -85,7 +85,7 @@ module Datadog
         # Always emit has_injectible_lines (even when false) on METHOD scopes.
         # Wire format keeps the historical spelling +injectible+; Ruby identifier
         # is +targetable_lines+.
-        if scope_type == 'METHOD'
+        if scope_type == "METHOD"
           h[:has_injectible_lines] = targetable_lines? # steep:ignore ArgumentTypeMismatch
           h[:injectible_lines] = targetable_lines if targetable_lines && !targetable_lines.empty?
         end

data/lib/datadog/symbol_database/scope_batcher.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-require 'set'
+require "set"
+require_relative "../di/fatal_exceptions"
 
 module Datadog
   module SymbolDatabase
@@ -122,7 +123,8 @@ module Datadog
 
         # Upload outside mutex (if batch was full)
         perform_upload(scopes_to_upload) if scopes_to_upload
-      rescue => e
+      rescue Exception => e # standard:disable Lint/RescueException
+        Datadog::DI.reraise_if_fatal(e)
         @logger.debug { "symdb: failed to add scope: #{e.class}: #{e.message}" }
         # Don't propagate, continue operation
       end
@@ -267,7 +269,8 @@ module Datadog
             flush
           end
         end
-      rescue => e
+      rescue Exception => e # standard:disable Lint/RescueException
+        Datadog::DI.reraise_if_fatal(e)
         @logger.debug { "symdb: timer thread error: #{e.class}: #{e.message}" }
       end
 
@@ -279,7 +282,8 @@ module Datadog
 
         @uploader.upload_scopes(scopes)
         @on_upload&.call(scopes)  # Notify tests after upload
-      rescue => e
+      rescue Exception => e # standard:disable Lint/RescueException
+        Datadog::DI.reraise_if_fatal(e)
         @logger.debug { "symdb: upload failed: #{e.class}: #{e.message}" }
         # Don't propagate, uploader handles retries
       end

data/lib/datadog/symbol_database/service_version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'json'
+require "json"
 
 module Datadog
   module SymbolDatabase
@@ -28,13 +28,13 @@ module Datadog
       # @param final [Boolean, nil] true if this is the last batch of the upload
       # @raise [ArgumentError] if service empty or scopes not an array
       def initialize(service:, env:, version:, scopes:, upload_id: nil, batch_num: nil, final: nil)
-        raise ArgumentError, 'service is required' if service.nil? || service.empty?
-        raise ArgumentError, 'scopes must be an array' unless scopes.is_a?(Array)
+        raise ArgumentError, "service is required" if service.nil? || service.empty?
+        raise ArgumentError, "scopes must be an array" unless scopes.is_a?(Array)
 
         @service = service
         @env = env
         @version = version
-        @language = 'ruby'
+        @language = "ruby"
         @scopes = scopes
         @upload_id = upload_id
         @batch_num = batch_num

data/lib/datadog/symbol_database/symbol.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'json'
+require "json"
 
 module Datadog
   module SymbolDatabase

data/lib/datadog/symbol_database/transport/http/endpoint.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative '../../../core/transport/http/api/endpoint'
+require_relative "../../../core/transport/http/api/endpoint"
 
 module Datadog
   module SymbolDatabase

data/lib/datadog/symbol_database/transport/http.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative '../../core/encoding'
-require_relative '../../core/transport/http'
-require_relative 'http/endpoint'
-require_relative '../transport'
+require_relative "../../core/encoding"
+require_relative "../../core/transport/http"
+require_relative "http/endpoint"
+require_relative "../transport"
 
 module Datadog
   module SymbolDatabase
@@ -14,7 +14,7 @@ module Datadog
         # Multipart form-data is dispatched via `env.form` from the
         # `Symbols::Client` subclass.
         SYMBOLS_ENDPOINT = API::Endpoint.new(
-          '/symdb/v1/input',
+          "/symdb/v1/input",
           Datadog::Core::Encoding::JSONEncoder,
         )
 
@@ -34,7 +34,7 @@ module Datadog
             agent_settings: agent_settings,
             headers: headers,
           ) do |transport|
-            transport.api 'symbols', SYMBOLS_ENDPOINT, default: true
+            transport.api "symbols", SYMBOLS_ENDPOINT, default: true
 
             yield(transport) if block_given?
           end.to_transport(SymbolDatabase::Transport::Symbols::Transport)

data/lib/datadog/symbol_database/transport.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative '../core/transport/request'
-require_relative '../core/transport/transport'
-require_relative '../core/transport/http/client'
-require_relative '../core/transport/http/env'
+require_relative "../core/transport/request"
+require_relative "../core/transport/transport"
+require_relative "../core/transport/http/client"
+require_relative "../core/transport/http/env"
 
 module Datadog
   module SymbolDatabase

data/lib/datadog/symbol_database/uploader.rb

@@ -1,13 +1,15 @@
 # frozen_string_literal: true
 
-require 'json'
-require 'securerandom'
-require 'zlib'
-require 'stringio'
-require_relative '../core/environment/identity'
-require_relative '../core/vendor/multipart-post/multipart/post/composite_read_io'
-require_relative 'service_version'
-require_relative 'transport/http'
+require "json"
+require "securerandom"
+require "zlib"
+require "stringio"
+require_relative "../core/environment/identity"
+require_relative "../core/vendor/multipart-post/multipart/post/composite_read_io"
+require_relative "../tracing/ext"
+require_relative "service_version"
+require_relative "transport/http"
+require_relative "../di/fatal_exceptions"
 
 module Datadog
   module SymbolDatabase
@@ -68,7 +70,11 @@ module Datadog
         compressed_data = Zlib.gzip(json_data)
 
         # Emitted unconditionally so the rare oversized case is observable.
-        @telemetry&.distribution('tracers', 'symbol_database.payload_size', compressed_data.bytesize)
+        @telemetry&.distribution(
+          Tracing::Ext::TELEMETRY_METRICS_NAMESPACE,
+          "symbol_database.payload_size",
+          compressed_data.bytesize
+        )
 
         # Symbols for very large applications (>50MB after gzip) are dropped:
         # the upload is skipped and the customer sees no autocomplete /
@@ -81,9 +87,10 @@ module Datadog
         end
 
         perform_http_upload(compressed_data, scopes.size, upload_id: upload_id, batch_num: batch_num)
-      rescue => e
+      rescue Exception => e # standard:disable Lint/RescueException
+        Datadog::DI.reraise_if_fatal(e)
         @logger.debug { "symdb: upload failed: #{e.class}: #{e.message}" }
-        @telemetry&.report(e, description: 'symdb: upload failed')
+        @telemetry&.report(e, description: "symdb: upload failed")
       end
 
       private
@@ -132,19 +139,19 @@ module Datadog
 
         event_upload = Datadog::Core::Vendor::Multipart::Post::UploadIO.new(
           event_io,
-          'application/json',
-          'event.json',
+          "application/json",
+          "event.json",
         )
 
         file_upload = Datadog::Core::Vendor::Multipart::Post::UploadIO.new(
           file_io,
-          'application/gzip',
+          "application/gzip",
           "symbols_#{Process.pid}.json.gz",
         )
 
         {
-          'event' => event_upload,
-          'file' => file_upload,
+          "event" => event_upload,
+          "file" => file_upload,
         }
       end
 
@@ -155,13 +162,13 @@ module Datadog
       # @return [String] JSON string for event metadata
       def build_event_metadata(attachment_size, upload_id:, batch_num:)
         JSON.generate(
-          ddsource: 'ruby',
+          ddsource: "ruby",
           service: @settings.service,
           version: @settings.version,
-          language: 'ruby',
+          language: "ruby",
           runtimeId: Datadog::Core::Environment::Identity.id,
           parentId: nil,  # Fork tracking deferred for MVP
-          type: 'symdb',
+          type: "symdb",
           uploadId: upload_id,
           batchNum: batch_num,
           # Always false: the Ruby tracer continuously uploads new code
@@ -187,8 +194,8 @@ module Datadog
           end
           @batch_num += 1 # steep:ignore NoMethod
           # @upload_id and @batch_num are non-nil after the branch above; refine for Steep.
-          upload_id = @upload_id or raise('unreachable: @upload_id should be set')
-          batch_num = @batch_num or raise('unreachable: @batch_num should be set')
+          upload_id = @upload_id or raise("unreachable: @upload_id should be set")
+          batch_num = @batch_num or raise("unreachable: @batch_num should be set")
           [upload_id, batch_num]
         end
       end

data/lib/datadog/symbol_database.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative "ruby_version"
+
 module Datadog
   # Namespace for Datadog symbol database upload.
   #
@@ -45,5 +47,25 @@ module Datadog
     # Reference: Symbol Database Backend RFC, section "Scope" and "Edge Cases"
     # @see https://www.postgresql.org/docs/current/datatype-numeric.html
     UNKNOWN_MAX_LINE = 2147483647
+
+    # Collapses the symbol_database.enabled tri-state setting to a boolean.
+    # An explicit true/false wins; nil (unconfigured) yields the caller-supplied
+    # fallback.
+    # @param setting_value [Boolean, nil] the symbol_database.enabled setting
+    # @param di_fallback [Boolean] value used when the setting is unconfigured
+    # @return [Boolean]
+    def self.resolve_enabled(setting_value, di_fallback)
+      setting_value.nil? ? di_fallback : setting_value
+    end
+
+    # Whether the current Ruby runtime can run symbol database extraction:
+    # MRI (CRuby) on Ruby 2.7 or later. Used by the remote-config layer to avoid
+    # advertising the product on runtimes where Component.build would return nil
+    # (e.g. Ruby 2.6, which Dynamic Instrumentation supports but Symbol Database
+    # does not).
+    # @return [Boolean]
+    def self.supported_runtime?
+      RUBY_ENGINE == "ruby" && RubyVersion.is?(">= 2.7")
+    end
   end
 end

data/lib/datadog/tracing/analytics.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'metadata/ext'
+require_relative "metadata/ext"
 
 module Datadog
   module Tracing

data/lib/datadog/tracing/buffer.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative '../core/environment/ext'
-require_relative '../core/buffer/thread_safe'
-require_relative '../core/buffer/cruby'
-require_relative '../core/diagnostics/health'
+require_relative "../core/environment/ext"
+require_relative "../core/buffer/thread_safe"
+require_relative "../core/buffer/cruby"
+require_relative "../core/diagnostics/health"
 
 module Datadog
   module Tracing
@@ -120,7 +120,7 @@ module Datadog
     #
     # TODO We should restructure this module, so that classes are not declared at top-level ::Datadog.
     # TODO Making such a change is potentially breaking for users manually configuring the tracer.
-    TraceBuffer = if Core::Environment::Ext::RUBY_ENGINE == 'ruby'
+    TraceBuffer = if Core::Environment::Ext::RUBY_ENGINE == "ruby"
       CRubyTraceBuffer
     else
       ThreadSafeTraceBuffer

data/lib/datadog/tracing/client_ip.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative '../core/configuration'
-require_relative '../core/utils/network'
-require_relative 'metadata/ext'
-require_relative 'span'
+require_relative "../core/configuration"
+require_relative "../core/utils/network"
+require_relative "metadata/ext"
+require_relative "span"
 
 module Datadog
   module Tracing

data/lib/datadog/tracing/component.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require_relative 'tracer'
-require_relative 'flush'
-require_relative 'sync_writer'
-require_relative 'sampling/span/rule_parser'
-require_relative 'sampling/span/sampler'
-require_relative 'diagnostics/environment_logger'
-require_relative 'contrib/component'
+require_relative "tracer"
+require_relative "flush"
+require_relative "sync_writer"
+require_relative "sampling/span/rule_parser"
+require_relative "sampling/span/sampler"
+require_relative "diagnostics/environment_logger"
+require_relative "contrib/component"
 
 module Datadog
   module Tracing
@@ -104,9 +104,30 @@ module Datadog
           return writer
         end
 
+        if settings.tracing.native_transport && (transport = build_native_transport(agent_settings))
+          options = options.merge(transport: transport)
+        end
+
         Tracing::Writer.new(agent_settings: agent_settings, **options)
       end
 
+      def build_native_transport(agent_settings)
+        require_relative "transport/native"
+
+        unless Transport::Native.supported?
+          Datadog.logger.warn(
+            "Native transport requested but not available: #{Transport::Native::UNSUPPORTED_REASON}. " \
+            "Falling back to default HTTP transport."
+          )
+          return nil
+        end
+
+        Transport::Native::Transport.new(
+          agent_settings: agent_settings,
+          logger: Datadog.logger
+        )
+      end
+
       def subscribe_to_writer_events!(writer, sampler_delegator, test_mode)
         return unless writer.respond_to?(:events) # Check if it's a custom, external writer
 

data/lib/datadog/tracing/configuration/dynamic.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require_relative 'dynamic/option'
+require_relative "dynamic/option"
 
 module Datadog
   module Tracing
@@ -11,14 +11,14 @@ module Datadog
         # Dynamic configuration for `DD_LOGS_INJECTION`.
         class LogInjectionEnabled < SimpleOption
           def initialize
-            super('log_injection_enabled', 'DD_LOGS_INJECTION', :log_injection)
+            super("log_injection_enabled", "DD_LOGS_INJECTION", :log_injection)
           end
         end
 
         # Dynamic configuration for `DD_TRACE_HEADER_TAGS`.
         class TracingHeaderTags < SimpleOption
           def initialize
-            super('tracing_header_tags', 'DD_TRACE_HEADER_TAGS', :header_tags)
+            super("tracing_header_tags", "DD_TRACE_HEADER_TAGS", :header_tags)
           end
 
           def call(tracing_header_tags)
@@ -35,7 +35,7 @@ module Datadog
         # Dynamic configuration for `DD_TRACE_SAMPLE_RATE`.
         class TracingSamplingRate < SimpleOption
           def initialize
-            super('tracing_sampling_rate', 'DD_TRACE_SAMPLE_RATE', :default_rate)
+            super("tracing_sampling_rate", "DD_TRACE_SAMPLE_RATE", :default_rate)
           end
 
           # Ensures sampler is rebuilt and new configuration is applied
@@ -54,7 +54,7 @@ module Datadog
         # Dynamic configuration for `DD_TRACE_SAMPLING_RULES`.
         class TracingSamplingRules < SimpleOption
           def initialize
-            super('tracing_sampling_rules', 'DD_TRACE_SAMPLING_RULES', :rules)
+            super("tracing_sampling_rules", "DD_TRACE_SAMPLING_RULES", :rules)
           end
 
           # Ensures sampler is rebuilt and new configuration is applied
@@ -63,15 +63,15 @@ module Datadog
             # local environment variable it configures.
             if tracing_sampling_rules
               tracing_sampling_rules.each do |rule|
-                next unless (tags = rule['tags'])
+                next unless (tags = rule["tags"])
 
                 # Tag maps come in as arrays of 'key' and `value_glob`.
                 # We need to convert them into a hash for local use.
                 tag_array = tags.map! do |tag|
-                  [tag['key'], tag['value_glob']]
+                  [tag["key"], tag["value_glob"]]
                 end
 
-                rule['tags'] = tag_array.to_h
+                rule["tags"] = tag_array.to_h
               end
 
               # The configuration is stored as JSON, so we need to convert it back