datadog 2.46.0 → 2.48.0

data/lib/datadog/opentelemetry/sdk/span_processor.rb

@@ -38,7 +38,7 @@ module Datadog
               Datadog::Tracing::SpanEvent.new(
                 event.name,
                 attributes: event.attributes,
-                time_unix_nano: event.timestamp
+                time_unix_nano: event.timestamp,
               )
             end
           end
@@ -112,7 +112,7 @@ module Datadog
                   trace_state: tracestate && ::Datadog::Core::Utils.utf8_encode(tracestate, placeholder: nil),
                   span_remote: link.span_context.remote?,
                 ),
-                attributes: link.attributes
+                attributes: link.attributes,
               )
             end
           end

data/lib/datadog/opentelemetry/sdk/trace/span.rb

@@ -164,7 +164,7 @@ module Datadog
           if key == "analytics.event" && value.respond_to?(:casecmp)
             Datadog::Tracing::Analytics.set_sample_rate(
               span,
-              (value.casecmp("true") == 0) ? 1 : 0
+              (value.casecmp("true") == 0) ? 1 : 0,
             )
           end
 

data/lib/datadog/profiling/collectors/code_provenance.rb

@@ -34,7 +34,7 @@ module Datadog
               version: RUBY_VERSION,
               path: standard_library_path,
               extra_paths: [ruby_native_filename],
-            )
+            ),
           )
         end
 
@@ -104,7 +104,7 @@ module Datadog
                 version: spec.version,
                 path: spec.gem_dir,
                 extra_paths: extra_paths,
-              )
+              ),
             )
             recorded_library = true
           end
@@ -146,7 +146,7 @@ module Datadog
         rescue Exception => e # rubocop:disable Lint/RescueException
           Datadog.logger.debug(
             "CodeProvenance#bundler_bin_path failed. " \
-            "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+            "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
           )
           nil
         end

data/lib/datadog/profiling/collectors/cpu_and_wall_time_worker.rb

@@ -55,10 +55,10 @@ module Datadog
         )
           unless dynamic_sampling_rate_enabled
             Datadog.logger.warn(
-              "Profiling dynamic sampling rate disabled. This should only be used for testing, and will increase overhead!"
+              "Profiling dynamic sampling rate disabled. This should only be used for testing, and will increase overhead!",
             )
             Datadog::Core::Telemetry::Logger.error(
-              "Profiling dynamic sampling rate disabled. This should only be used for testing, and will increase overhead!"
+              "Profiling dynamic sampling rate disabled. This should only be used for testing, and will increase overhead!",
             )
           end
 
@@ -110,7 +110,7 @@ module Datadog
               @failure_exception = e
               Datadog.logger.warn(
                 "Profiling was not started as another profiler or gem is already using the SIGPROF signal. " \
-                "Please disable the other profiler to use Datadog profiling."
+                "Please disable the other profiler to use Datadog profiling.",
               )
               on_failure_proc&.call(log_failure: false)
             rescue Exception => e # rubocop:disable Lint/RescueException
@@ -118,7 +118,7 @@ module Datadog
               operation_name = self.class._native_failure_exception_during_operation(self).inspect
               Datadog.logger.warn(
                 "CpuAndWallTimeWorker thread error. " \
-                "Operation: #{operation_name} Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+                "Operation: #{operation_name} Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
               )
               on_failure_proc&.call
               Datadog::Core::Telemetry::Logger.report(e, description: "CpuAndWallTimeWorker thread error: #{operation_name}")

data/lib/datadog/profiling/collectors/idle_sampling_helper.rb

@@ -44,7 +44,7 @@ module Datadog
               @failure_exception = e
               Datadog.logger.warn(
                 "IdleSamplingHelper thread error. " \
-                "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+                "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
               )
               Datadog::Core::Telemetry::Logger.report(e, description: "IdleSamplingHelper thread error")
             end

data/lib/datadog/profiling/collectors/info.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "json"
 require "libdatadog"
 require_relative "../../core/utils/time"
 
@@ -9,11 +10,11 @@ module Datadog
       # Collects information of relevance for profiler. This will get sent alongside
       # the profile and show up in the UI or potentially influence processing in some way.
       #
-      # Information is currently collected and frozen at construction time. A full collector
-      # could be seen as overkill for this case but it allows us to centralize information
-      # gathering and easily support more flexible/dynamic info collection in the future.
+      # Most information is static: collected and frozen at construction time. The exception is
+      # what `collect_dynamic_info` collects, which can change during the process lifetime.
       class Info
-        # @rbs @info: ::Hash[::Symbol, ::Hash[::Symbol, untyped]]
+        # @rbs @static_info: ::Hash[::Symbol, ::Hash[::Symbol, untyped]]
+        # @rbs @info_json_cache: [::Hash[::Symbol, bool], ::String]?
         # @rbs @platform_info: ::Hash[::Symbol, untyped]
         # @rbs @runtime_info: ::Hash[::Symbol, untyped]
         # @rbs @application_info: ::Hash[::Symbol, untyped]
@@ -24,18 +25,48 @@ module Datadog
         def initialize(settings)
           @profiler_info = nil
 
-          @info = {
+          @static_info = {
             platform: collect_platform_info,
             runtime: collect_runtime_info,
             application: collect_application_info(settings),
             profiler: collect_profiler_info(settings),
           }.freeze
+          @info_json_cache = nil
         end
 
-        attr_reader :info #: ::Hash[::Symbol, ::Hash[::Symbol, untyped]]
+        #: () -> ::Hash[::Symbol, ::Hash[::Symbol, untyped]]
+        def info
+          build_info(collect_dynamic_info)
+        end
+
+        # Serializing the same info on every flush would be wasteful, so the latest result is
+        # cached and reused for as long as the dynamic info is unchanged.
+        #: () -> ::String
+        def info_json
+          dynamic_info = collect_dynamic_info
+          cached_dynamic_info, cached_json = @info_json_cache
+          return cached_json if cached_json && cached_dynamic_info == dynamic_info
+
+          json = JSON.generate(build_info(dynamic_info)).freeze
+          @info_json_cache = [dynamic_info, json].freeze #: [::Hash[::Symbol, bool], ::String]
+          json
+        end
 
         private
 
+        #: (::Hash[::Symbol, bool]) -> ::Hash[::Symbol, ::Hash[::Symbol, untyped]]
+        def build_info(dynamic_info)
+          @static_info.merge(runtime: @static_info.fetch(:runtime).merge(dynamic_info))
+        end
+
+        #: () -> ::Hash[::Symbol, bool]
+        def collect_dynamic_info
+          {
+            yjit_enabled: defined?(::RubyVM::YJIT) ? ::RubyVM::YJIT.enabled? : false,
+            zjit_enabled: defined?(::RubyVM::ZJIT) ? ::RubyVM::ZJIT.enabled? : false,
+          }
+        end
+
         # Ruby GC tuning environment variables
         RUBY_GC_TUNING_ENV_VARS = [
           "RUBY_GC_HEAP_FREE_SLOTS",

data/lib/datadog/profiling/collectors/thread_context.rb

@@ -44,7 +44,7 @@ module Datadog
           endpoint_collection_enabled: false,
           otel_context_enabled: false,
           native_filenames_enabled: true,
-          show_classes: false,
+          show_classes: true,
           trigger_global_reset: true,
           **options
         )

data/lib/datadog/profiling/component.rb

@@ -145,14 +145,14 @@ module Datadog
             RubyVersion.is?(">= 3.2", "< 3.2.3")
           logger.warn(
             "Current Ruby version (#{RUBY_VERSION}) has a VM bug where enabling GC profiling would cause " \
-            "crashes (https://bugs.ruby-lang.org/issues/18464). GC profiling has been disabled."
+            "crashes (https://bugs.ruby-lang.org/issues/18464). GC profiling has been disabled.",
           )
           return false
         elsif RubyVersion.is?(">= 3", "< 4")
           logger.debug(
             "Using Ractors may result in GC profiling unexpectedly " \
             "stopping (https://bugs.ruby-lang.org/issues/19112). Note that this stop has no impact in your " \
-            "application stability or performance. This issue is fixed on Ruby 4."
+            "application stability or performance. This issue is fixed on Ruby 4.",
           )
         end
 
@@ -180,7 +180,7 @@ module Datadog
           logger.warn(
             "Allocation profiling is not supported in Ruby versions 3.2.0, 3.2.1 and 3.2.2 and will be forcibly " \
             "disabled. This is due to a VM bug that can lead to crashes (https://bugs.ruby-lang.org/issues/19482). " \
-            "Other Ruby versions do not suffer from this issue."
+            "Other Ruby versions do not suffer from this issue.",
           )
           return false
         end
@@ -195,7 +195,7 @@ module Datadog
           logger.warn(
             "Current Ruby version (#{RUBY_VERSION}) has a VM bug where enabling allocation profiling while using " \
             "Ractors may cause unexpected issues, including crashes (https://bugs.ruby-lang.org/issues/18464). " \
-            "This does not happen if Ractors are not used."
+            "This does not happen if Ractors are not used.",
           )
         # ANNOYANCE - Only with Ractors
         # On all known versions of Ruby 3.x, due to https://bugs.ruby-lang.org/issues/19112, when a ractor gets
@@ -204,7 +204,7 @@ module Datadog
           logger.debug(
             "Using Ractors may result in allocation profiling " \
             "stopping (https://bugs.ruby-lang.org/issues/19112). Note that this stop has no impact in your " \
-            "application stability or performance. This issue is fixed on Ruby 4."
+            "application stability or performance. This issue is fixed on Ruby 4.",
           )
         end
 
@@ -221,7 +221,7 @@ module Datadog
         if RubyVersion.is?("< 3.1")
           logger.warn(
             "Current Ruby version (#{RUBY_VERSION}) cannot support heap profiling due to VM limitations. " \
-            "Please upgrade to Ruby >= 3.1 in order to use this feature. Heap profiling has been disabled."
+            "Please upgrade to Ruby >= 3.1 in order to use this feature. Heap profiling has been disabled.",
           )
           return false
         end
@@ -232,7 +232,7 @@ module Datadog
           logger.warn(
             "Current Ruby version (#{RUBY_VERSION}) cannot support heap profiling due to a VM bug. " \
             "Please upgrade to Ruby >= 3.1.4 or >= 3.2.3 in order to use this feature. " \
-            "Heap profiling has been disabled."
+            "Heap profiling has been disabled.",
           )
           return false
         end
@@ -240,7 +240,7 @@ module Datadog
         unless allocation_profiling_enabled
           logger.warn(
             "Heap profiling was requested but allocation profiling is not enabled. " \
-            "Heap profiling has been disabled."
+            "Heap profiling has been disabled.",
           )
           return false
         end
@@ -262,7 +262,7 @@ module Datadog
         unless [true, false, :auto].include?(setting_value)
           logger.warn(
             "Ignoring invalid value for profiling no_signals_workaround_enabled setting: #{setting_value.inspect}. " \
-            "Valid options are `true`, `false` or (default) `:auto`."
+            "Valid options are `true`, `false` or (default) `:auto`.",
           )
 
           setting_value = :auto
@@ -275,7 +275,7 @@ module Datadog
               "This is not recommended " \
               "in production environments, as due to limitations in Ruby APIs, we suspect it may lead to rare crashes " \
               "Please report any issues you run into to Datadog support or " \
-              "via <https://github.com/datadog/dd-trace-rb/issues/new>!"
+              "via <https://github.com/datadog/dd-trace-rb/issues/new>!",
             )
           else
             logger.warn('Profiling "no signals" workaround disabled via configuration')
@@ -286,7 +286,7 @@ module Datadog
 
         if setting_value == true
           logger.warn(
-            'Profiling "no signals" workaround enabled via configuration. Profiling data will have lower quality.'
+            'Profiling "no signals" workaround enabled via configuration. Profiling data will have lower quality.',
           )
 
           return true
@@ -301,7 +301,7 @@ module Datadog
           logger.warn(
             'Enabling the profiling "no signals" workaround because an incompatible version of the mysql2 gem is ' \
             "installed. Profiling data will have lower quality. " \
-            "To fix this, upgrade the libmysqlclient in your OS image to version 8.0.0 or above."
+            "To fix this, upgrade the libmysqlclient in your OS image to version 8.0.0 or above.",
           )
           return true
         end
@@ -311,7 +311,7 @@ module Datadog
             'Enabling the profiling "no signals" workaround because the rugged gem is installed. ' \
             "This is needed because some operations on this gem are currently incompatible with the normal working mode " \
             "of the profiler, as detailed in <https://github.com/datadog/dd-trace-rb/issues/2721>. " \
-            "Profiling data will have lower quality."
+            "Profiling data will have lower quality.",
           )
           return true
         end
@@ -320,7 +320,7 @@ module Datadog
           logger.warn(
             'Enabling the profiling "no signals" workaround because an incompatible version of the passenger gem is ' \
             "installed. Profiling data will have lower quality." \
-            "To fix this, upgrade the passenger gem to version 6.0.19 or above."
+            "To fix this, upgrade the passenger gem to version 6.0.19 or above.",
           )
           return true
         end
@@ -341,7 +341,7 @@ module Datadog
         return true if settings.profiling.advanced.skip_mysql2_check
 
         logger.debug(
-          "Requiring `mysql2` to check if the `libmysqlclient` version it uses is compatible with profiling"
+          "Requiring `mysql2` to check if the `libmysqlclient` version it uses is compatible with profiling",
         )
 
         begin
@@ -371,14 +371,14 @@ module Datadog
 
           logger.debug(
             "The `mysql2` gem is using #{compatible ? "a compatible" : "an incompatible"} version of " \
-            "the `libmysqlclient` library (#{libmysqlclient_version})"
+            "the `libmysqlclient` library (#{libmysqlclient_version})",
           )
 
           !compatible
         rescue StandardError, LoadError => e
           logger.warn(
             "Failed to probe `mysql2` gem information. " \
-            "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+            "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
           )
 
           true
@@ -404,7 +404,7 @@ module Datadog
         else
           logger.warn(
             "Ignoring invalid value for profiling overhead_target_percentage setting: " \
-            "#{overhead_target_percentage.inspect}. Falling back to default value."
+            "#{overhead_target_percentage.inspect}. Falling back to default value.",
           )
 
           2.0
@@ -416,7 +416,7 @@ module Datadog
           logger.warn(
             "Profiling cpu_sampling_interval_ms is set to #{cpu_sampling_interval_ms}ms, but values above 10ms are " \
             "not supported. Using 10ms instead. To reduce profiler overhead, consider adjusting the " \
-            "overhead_target_percentage setting."
+            "overhead_target_percentage setting.",
           )
           10
         elsif cpu_sampling_interval_ms < 10
@@ -427,33 +427,35 @@ module Datadog
         end
       end
 
-      # To add just a bit more complexity to our detection code, in https://github.com/DataDog/dd-trace-rb/issues/3334
-      # a user reported that our code was incorrectly flagging the mariadb variant of libmysqlclient as being
-      # incompatible. In fact we have no reports of the mariadb variant needing the "no signals" workaround,
-      # so we flag it as compatible when it's in use.
+      # MariaDB's client has diverged from Oracle's and has no reported need for the "no signals" workaround, as
+      # reported in https://github.com/DataDog/dd-trace-rb/issues/3334, so we flag it as compatible when it's in use.
       #
-      # A problem is that there doesn't seem to be an obvious way to query the mysql2 gem on which kind of
-      # libmysqlclient it's using, so we detect it by looking at the version.
+      # There's no way to ask the mysql2 gem which client it was built against, so we infer it from the two versions
+      # that `Mysql2::Client.info` exposes:
       #
-      # The info method for mysql2 with mariadb looks something like this:
-      # `{:id=>30308, :version=>"3.3.8", :header_version=>"11.2.2"}`
+      # * `version` is `mysql_get_client_info()`. For MariaDB that's the version of
+      #   https://github.com/mariadb-corporation/mariadb-connector-c, which is still 3.x.
+      # * `header_version` is a constant baked in when mysql2 was compiled. Up to mysql2 0.5.6 it was the MariaDB
+      #   server version, which is what people usually call "the mariadb version"; 0.5.7 switched it to the
+      #   Connector/C version (https://github.com/brianmario/mysql2/pull/1406) and that broke our detection, as
+      #   reported in https://github.com/DataDog/dd-trace-rb/issues/6536.
       #
-      # * The version seems to come from https://github.com/mariadb-corporation/mariadb-connector-c and the latest
-      # one is 3.x.
-      # * The header_version is what people usually see as the "mariadb version"
+      # So MariaDB shows up as `{version: "3.3.8", header_version: "11.2.2"}` with mysql2 < 0.5.7 and as
+      # `{version: "3.4.9", header_version: "3.4.9"}` from 0.5.7 on. We accept both shapes: a header that is a server
+      # version (10 or newer, a sane range for modern releases per https://endoflife.date/mariadb) or one that is a
+      # Connector/C version (3.x, matching the client version).
       #
-      # As a comparison, for libmysql the info looks like:
-      # * `{:id=>80035, :version=>"8.0.35", :header_version=>"8.0.35"}`
-      #
-      # Thus our detection is version 4 or older, because libmysqlclient 4 is almost 20 years old so it's most probably
-      # not that one + header_version being 10 or newer, since according to https://endoflife.date/mariadb that's a
-      # sane range for modern mariadb releases.
+      # Oracle's client repeats one version in both fields (`{version: "8.0.35", header_version: "8.0.35"}`), and our
+      # caller already considers 8.0.0 and above compatible, so only a client older than 5.0.0 gets this far -- and
+      # no Oracle release has been one since libmysqlclient 4.1, which is from 2004.
       private_class_method def self.looks_like_mariadb?(info, libmysqlclient_version)
         header_version = Gem::Version.new(info[:header_version]) if info[:header_version]
 
-        !!(header_version &&
-          libmysqlclient_version < Gem::Version.new("5.0.0") &&
-          header_version >= Gem::Version.new("10.0.0"))
+        return false unless header_version && libmysqlclient_version < Gem::Version.new("5.0.0")
+        return true if header_version >= Gem::Version.new("10.0.0")
+
+        connector_c_versions = Gem::Requirement.new(">= 3.0.0", "< 4.0.0")
+        connector_c_versions.satisfied_by?(libmysqlclient_version) && connector_c_versions.satisfied_by?(header_version)
       end
 
       private_class_method def self.dir_interruption_workaround_enabled?(settings, no_signals_workaround_enabled)

data/lib/datadog/profiling/exporter.rb

@@ -27,7 +27,7 @@ module Datadog
       attr_reader :last_flush_finish_at #: ::Time?
       attr_reader :created_at #: ::Time
       attr_reader :internal_metadata #: ::Hash[::Symbol, untyped]
-      attr_reader :info_json #: ::String
+      attr_reader :info_collector #: Datadog::Profiling::Collectors::Info
       attr_reader :sequence_tracker #: singleton(Datadog::Profiling::SequenceTracker)
 
       public
@@ -56,9 +56,7 @@ module Datadog
         @last_flush_finish_at = nil
         @created_at = time_provider.now.utc
         @internal_metadata = internal_metadata
-        # NOTE: At the time of this comment collected info does not change over time so we'll hardcode
-        #       it on startup to prevent serializing the same info on every flush.
-        @info_json = JSON.generate(info_collector.info).freeze
+        @info_collector = info_collector
         @sequence_tracker = sequence_tracker
       end
 
@@ -109,9 +107,9 @@ module Datadog
               profile_stats: profile_stats,
               recorder_stats: pprof_recorder.stats,
               gc: gc_stats,
-            }
+            },
           ),
-          info_json: info_json,
+          info_json: info_collector.info_json,
         )
       end
 

data/lib/datadog/profiling/http_transport.rb

@@ -40,7 +40,7 @@ module Datadog
       def export(flush)
         status, result = self.class._native_do_export(
           exporter_configuration,
-          flush
+          flush,
         )
 
         if status == :ok
@@ -50,10 +50,10 @@ module Datadog
           else
             Datadog.logger.warn(
               "Failed to report profiling data (#{config_without_api_key}): " \
-              "server returned unexpected HTTP #{result} status code"
+              "server returned unexpected HTTP #{result} status code",
             )
             Datadog::Core::Telemetry::Logger.error(
-              "Failed to report profiling data: unexpected HTTP #{result} status code"
+              "Failed to report profiling data: unexpected HTTP #{result} status code",
             )
             false
           end

data/lib/datadog/profiling/profiler.rb

@@ -32,7 +32,7 @@ module Datadog
           on_failure_proc: ->(log_failure: true) do
             # @type var log_failure: bool
             component_failed(:worker, log_failure: log_failure)
-          end
+          end,
         )
         scheduler.start(on_failure_proc: proc { component_failed(:scheduler) })
       end
@@ -67,7 +67,7 @@ module Datadog
         if log_failure
           Datadog.logger.warn(
             "Detected issue with profiler (#{failed_component} component), stopping profiling. " \
-            "See previous log messages for details."
+            "See previous log messages for details.",
           )
           Datadog::Core::Telemetry::Logger
             .error("Detected issue with profiler (#{failed_component} component), stopping profiling")

data/lib/datadog/profiling/scheduler.rb

@@ -65,7 +65,7 @@ module Datadog
       rescue Exception => e # rubocop:disable Lint/RescueException
         Datadog.logger.warn(
           "Profiling::Scheduler thread error. " \
-          "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+          "Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
         )
         on_failure_proc&.call
         Datadog::Core::Telemetry::Logger.report(e, description: "Profiling::Scheduler thread error")
@@ -136,7 +136,7 @@ module Datadog
           transport.export(flush)
         rescue => e
           Datadog.logger.warn(
-            "Unable to report profile. Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}"
+            "Unable to report profile. Cause: #{e.class}: #{e.message} Location: #{Array(e.backtrace).first}",
           )
           Datadog::Core::Telemetry::Logger.report(e, description: "Unable to report profile")
         end

data/lib/datadog/profiling/tag_builder.rb

@@ -38,7 +38,7 @@ module Datadog
           "process_id",
           "profiler_version",
           "profile_seq",
-        ]
+        ],
       ).freeze
 
       def self.call(

data/lib/datadog/symbol_database/extractor.rb

@@ -491,7 +491,7 @@ module Datadog
           start_line: UNKNOWN_MIN_LINE,
           end_line: UNKNOWN_MAX_LINE,
           language_specifics: lang,
-          scopes: inner_scopes
+          scopes: inner_scopes,
         )
       end
 
@@ -507,7 +507,7 @@ module Datadog
           source_file: source_file,
           start_line: UNKNOWN_MIN_LINE,
           end_line: UNKNOWN_MAX_LINE,
-          symbols: extract_scope_symbols(mod)
+          symbols: extract_scope_symbols(mod),
         )
       end
 
@@ -527,7 +527,7 @@ module Datadog
           end_line: end_line,
           language_specifics: build_class_language_specifics(klass),
           scopes: method_scopes,
-          symbols: extract_scope_symbols(klass)
+          symbols: extract_scope_symbols(klass),
         )
       end
 
@@ -997,7 +997,7 @@ module Datadog
           end_line: end_line,
           language_specifics: lang,
           scopes: method_scopes + child_scopes,
-          symbols: symbols
+          symbols: symbols,
         )
       end
 
@@ -1023,11 +1023,11 @@ module Datadog
           end_line: end_line,
           targetable_lines: targetable_lines,
           language_specifics: {
-            visibility: klass ? method_visibility(klass, method_name) : "public", # steep:ignore
+            visibility: klass ? method_visibility(klass, method_name) : "public",
             method_type: "instance",
             arity: method.arity,
           },
-          symbols: extract_method_parameters(method)
+          symbols: extract_method_parameters(method),
         )
       rescue Exception => e # standard:disable Lint/RescueException
         Datadog::DI.reraise_if_fatal(e)
@@ -1049,7 +1049,7 @@ module Datadog
             symbols << Symbol.new(
               symbol_type: "STATIC_FIELD",
               name: var_name.to_s,
-              line: UNKNOWN_MIN_LINE
+              line: UNKNOWN_MIN_LINE,
             )
           end
         end
@@ -1065,7 +1065,7 @@ module Datadog
             symbol_type: "STATIC_FIELD",
             name: const_name.to_s,
             line: UNKNOWN_MIN_LINE,
-            type: safe_mod_name(KERNEL_CLASS.bind(const_value).call)
+            type: safe_mod_name(KERNEL_CLASS.bind(const_value).call),
           )
         rescue NameError, LoadError, NoMethodError, TypeError => e # standard:disable Lint/ShadowedException
           # Expected: constant removed/undefined (NameError), autoload failure (LoadError),

data/lib/datadog/symbol_database/uploader.rb

@@ -73,7 +73,7 @@ module Datadog
         @telemetry&.distribution(
           Tracing::Ext::TELEMETRY_METRICS_NAMESPACE,
           "symbol_database.payload_size",
-          compressed_data.bytesize
+          compressed_data.bytesize,
         )
 
         # Symbols for very large applications (>50MB after gzip) are dropped:

data/lib/datadog/tracing/buffer.rb

@@ -56,7 +56,7 @@ module Datadog
         @buffer_spans += trace.length
       rescue => e
         Datadog.logger.debug(
-          "Failed to measure queue accept. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}"
+          "Failed to measure queue accept. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}",
         )
       end
 
@@ -66,7 +66,7 @@ module Datadog
         @buffer_spans -= trace.length
       rescue => e
         Datadog.logger.debug(
-          "Failed to measure queue drop. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}"
+          "Failed to measure queue drop. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}",
         )
       end
 
@@ -91,7 +91,7 @@ module Datadog
         @buffer_spans = 0
       rescue => e
         Datadog.logger.debug(
-          "Failed to measure queue. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}"
+          "Failed to measure queue. Cause: #{e.class}: #{e.message} Source: #{Array(e.backtrace).first}",
         )
       end
     end

data/lib/datadog/tracing/client_ip.rb

@@ -41,7 +41,7 @@ module Datadog
           if headers && configuration.header_name
             return Datadog::Core::Utils::Network.stripped_ip_from_request_headers(
               headers,
-              ip_headers_to_check: Array(configuration.header_name)
+              ip_headers_to_check: Array(configuration.header_name),
             )
           end