datadog 2.19.0 → 2.21.0

data/lib/datadog/tracing/contrib/rack/header_tagging.rb

@@ -12,17 +12,17 @@ module Datadog
 
             # Use global DD_TRACE_HEADER_TAGS if integration-level configuration is not provided
             tags = if configuration.using_default?(:headers) && !Datadog.configuration.tracing.using_default?(:header_tags)
-                     Datadog.configuration.tracing.header_tags.request_tags(headers)
-                   else
-                     whitelist = configuration[:headers][:request] || []
-                     whitelist.each_with_object({}) do |header, result|
-                       header_value = headers.get(header)
-                       unless header_value.nil?
-                         header_tag = Tracing::Metadata::Ext::HTTP::RequestHeaders.to_tag(header)
-                         result[header_tag] = header_value
-                       end
-                     end
-                   end
+              Datadog.configuration.tracing.header_tags.request_tags(headers)
+            else
+              whitelist = configuration[:headers][:request] || []
+              whitelist.each_with_object({}) do |header, result|
+                header_value = headers.get(header)
+                unless header_value.nil?
+                  header_tag = Tracing::Metadata::Ext::HTTP::RequestHeaders.to_tag(header)
+                  result[header_tag] = header_value
+                end
+              end
+            end
 
             span.set_tags(tags)
           end
@@ -32,27 +32,27 @@ module Datadog
 
             # Use global DD_TRACE_HEADER_TAGS if integration-level configuration is not provided
             tags = if configuration.using_default?(:headers) && !Datadog.configuration.tracing.using_default?(:header_tags)
-                     Datadog.configuration.tracing.header_tags.response_tags(headers)
-                   else
-                     whitelist = configuration[:headers][:response] || []
-                     whitelist.each_with_object({}) do |header, result|
-                       header_value = headers[header]
-
-                       next if header_value.nil?
-
-                       header_tag = Tracing::Metadata::Ext::HTTP::ResponseHeaders.to_tag(header)
-
-                       # Maintain the value format between Rack 2 and 3
-                       #
-                       # Rack 2.x => { 'foo' => 'bar,baz' }
-                       # Rack 3.x => { 'foo' => ['bar', 'baz'] }
-                       result[header_tag] = if header_value.is_a? Array
-                                              header_value.join(',')
-                                            else
-                                              header_value
-                                            end
-                     end
-                   end
+              Datadog.configuration.tracing.header_tags.response_tags(headers)
+            else
+              whitelist = configuration[:headers][:response] || []
+              whitelist.each_with_object({}) do |header, result|
+                header_value = headers[header]
+
+                next if header_value.nil?
+
+                header_tag = Tracing::Metadata::Ext::HTTP::ResponseHeaders.to_tag(header)
+
+                # Maintain the value format between Rack 2 and 3
+                #
+                # Rack 2.x => { 'foo' => 'bar,baz' }
+                # Rack 3.x => { 'foo' => ['bar', 'baz'] }
+                result[header_tag] = if header_value.is_a? Array
+                  header_value.join(',')
+                else
+                  header_value
+                end
+              end
+            end
 
             span.set_tags(tags)
           end

data/lib/datadog/tracing/contrib/rack/integration.rb

@@ -19,7 +19,7 @@ module Datadog
           register_as :rack, auto_patch: false
 
           def self.version
-            Gem.loaded_specs['rack'] && Gem.loaded_specs['rack'].version
+            Gem.loaded_specs['rack']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/rack/middlewares.rb

@@ -47,7 +47,7 @@ module Datadog
             end
 
             TraceProxyMiddleware.call(env, configuration) do
-              trace_options = { type: Tracing::Metadata::Ext::HTTP::TYPE_INBOUND }
+              trace_options = {type: Tracing::Metadata::Ext::HTTP::TYPE_INBOUND}
               trace_options[:service] = configuration[:service_name] if configuration[:service_name]
 
               # start a new request span and attach it to the current Rack environment;
@@ -80,7 +80,7 @@ module Datadog
               # catch exceptions that may be raised in the middleware chain
               # Note: if a middleware catches an Exception without re raising,
               # the Exception cannot be recorded here.
-              request_span.set_error(e) unless request_span.nil?
+              request_span&.set_error(e)
               raise e
             ensure
               env[Ext::RACK_ENV_REQUEST_SPAN] = previous_request_span if previous_request_span
@@ -123,7 +123,7 @@ module Datadog
             # 4. Fallback with verb + status, eq `GET 200`
             request_span.resource ||=
               if configuration[:middleware_names] && env['RESPONSE_MIDDLEWARE']
-                "#{env['RESPONSE_MIDDLEWARE']}##{original_request_method}"
+                "#{env["RESPONSE_MIDDLEWARE"]}##{original_request_method}"
               elsif trace.resource_override?
                 trace.resource
               else
@@ -240,16 +240,18 @@ module Datadog
             Datadog.configuration.tracing[:rack]
           end
 
+          # rubocop:disable Metrics/AbcSize
+          # rubocop:disable Metrics/MethodLength
           def parse_url(env, original_env)
             request_obj = ::Rack::Request.new(env)
 
             # scheme, host, and port
             base_url = if request_obj.respond_to?(:base_url)
-                         request_obj.base_url
-                       else
-                         # Compatibility for older Rack versions
-                         request_obj.url.chomp(request_obj.fullpath)
-                       end
+              request_obj.base_url
+            else
+              # Compatibility for older Rack versions
+              request_obj.url.chomp(request_obj.fullpath)
+            end
 
             # https://github.com/rack/rack/blob/main/SPEC.rdoc
             #
@@ -269,18 +271,20 @@ module Datadog
             # prepended to PATH_INFO to reflect the correct user visible path.
             request_uri = env['REQUEST_URI'].to_s
             fullpath = if request_uri.empty?
-                         query_string = original_env['QUERY_STRING'].to_s
-                         path = original_env['SCRIPT_NAME'].to_s + original_env['PATH_INFO'].to_s
-
-                         query_string.empty? ? path : "#{path}?#{query_string}"
-                       else
-                         # normally REQUEST_URI starts at the path, but it
-                         # might contain the full URL in some cases (e.g WEBrick)
-                         request_uri.delete_prefix(base_url)
-                       end
+              query_string = original_env['QUERY_STRING'].to_s
+              path = original_env['SCRIPT_NAME'].to_s + original_env['PATH_INFO'].to_s
+
+              query_string.empty? ? path : "#{path}?#{query_string}"
+            else
+              # normally REQUEST_URI starts at the path, but it
+              # might contain the full URL in some cases (e.g WEBrick)
+              request_uri.delete_prefix(base_url)
+            end
 
             base_url + fullpath
           end
+          # rubocop:enable Metrics/AbcSize
+          # rubocop:enable Metrics/MethodLength
 
           def parse_user_agent_header(headers)
             headers.get(Tracing::Metadata::Ext::HTTP::HEADER_USER_AGENT)

data/lib/datadog/tracing/contrib/rack/patcher.rb

@@ -45,7 +45,7 @@ module Datadog
           end
 
           def retain_middleware_name(middleware)
-            return unless middleware && middleware.respond_to?(:call)
+            return unless middleware&.respond_to?(:call)
 
             middleware.singleton_class.class_eval do
               alias_method :__call, :call

data/lib/datadog/tracing/contrib/rack/request_queue.rb

@@ -24,6 +24,7 @@ module Datadog
             # nginx header is seconds in the format "t=1512379167.574"
             # apache header is microseconds in the format "t=1570633834463123"
             # heroku header is milliseconds in the format "1570634024294"
+            # @see https://github.com/heroku/vegur/blob/65d168f757e0ddb448f41cfb9e4b0281c747378d/README.md?plain=1#L383-L384
             time_string = header.to_s.delete('^0-9')
             return if time_string.nil?
 
@@ -34,8 +35,8 @@ module Datadog
             # return the request_start only if it's lesser than
             # current time, to avoid significant clock skew
             request_start = Time.at(time_value)
-            request_start.utc > now ? nil : request_start
-          rescue StandardError => e
+            (request_start.utc > now) ? nil : request_start
+          rescue => e
             # in case of an Exception we don't create a
             # `request.queuing` span
             Datadog.logger.debug("[rack] unable to parse request queue headers: #{e}")

data/lib/datadog/tracing/contrib/rack/trace_proxy_middleware.rb

@@ -43,7 +43,13 @@ module Datadog
             # excluding the time spent processing the request itself
             queue_span.finish
 
-            yield.tap { request_span.finish }
+            yield
+          ensure
+            # Ensure that the spans are finished even if an exception is raised.
+            # **This is very important** to prevent the trace from leaking between requests,
+            # especially because `queue_span` is normally a root span.
+            queue_span&.finish
+            request_span&.finish
           end
         end
       end

data/lib/datadog/tracing/contrib/rails/configuration/settings.rb

@@ -13,7 +13,7 @@ module Datadog
           # @public_api
           class Settings < Contrib::Configuration::Settings
             def initialize(options = {})
-              super(options)
+              super
 
               # NOTE: Eager load these
               #       Rails integration is responsible for orchestrating other integrations.

data/lib/datadog/tracing/contrib/rails/ext.rb

@@ -12,7 +12,8 @@ module Datadog
           # @!visibility private
           ENV_ANALYTICS_ENABLED = 'DD_TRACE_RAILS_ANALYTICS_ENABLED'
           ENV_ANALYTICS_SAMPLE_RATE = 'DD_TRACE_RAILS_ANALYTICS_SAMPLE_RATE'
-          ENV_DISABLE = 'DISABLE_DATADOG_RAILS'
+          # Note: Alias (previous name DISABLE_DATADOG_RAILS) defined in supported-configurations.json
+          ENV_DISABLE = 'DD_DISABLE_DATADOG_RAILS'
 
           SPAN_RUNNER_FILE = 'rails.runner.file'
           SPAN_RUNNER_INLINE = 'rails.runner.inline'

data/lib/datadog/tracing/contrib/rails/integration.rb

@@ -23,7 +23,7 @@ module Datadog
           register_as :rails, auto_patch: false
 
           def self.version
-            Gem.loaded_specs['railties'] && Gem.loaded_specs['railties'].version
+            Gem.loaded_specs['railties']&.version
           end
 
           def self.loaded?
@@ -35,7 +35,7 @@ module Datadog
           end
 
           def self.patchable?
-            super && !ENV.key?(Ext::ENV_DISABLE)
+            super && !DATADOG_ENV.key?(Ext::ENV_DISABLE)
           end
 
           def new_configuration

data/lib/datadog/tracing/contrib/rails/log_injection.rb

@@ -16,7 +16,7 @@ module Datadog
 
             app_config.log_tags ||= [] # Can be nil, we initialized it if so
             app_config.log_tags << proc { Tracing.log_correlation if Datadog.configuration.tracing.log_injection }
-          rescue StandardError => e
+          rescue => e
             Datadog.logger.warn(
               "Unable to add Datadog Trace context to ActiveSupport::TaggedLogging: #{e.class.name} #{e.message}"
             )

data/lib/datadog/tracing/contrib/rails/middlewares.rb

@@ -34,7 +34,7 @@ module Datadog
               # Some exception gets handled by Rails middleware before it can be set on Rack middleware
               # The rack span is the root span of the request and should make sure it has the full exception
               # set on it.
-              env[Contrib::Rack::Ext::RACK_ENV_REQUEST_SPAN].set_error(e) if env[Contrib::Rack::Ext::RACK_ENV_REQUEST_SPAN]
+              env[Contrib::Rack::Ext::RACK_ENV_REQUEST_SPAN]&.set_error(e)
             end
             raise e
           end

data/lib/datadog/tracing/contrib/rails/runner.rb

@@ -69,14 +69,15 @@ module Datadog
             end
           end
 
-          def self.prepend(base)
-            base.const_set(:Kernel, Kernel)
+          def self.prepended(base)
+            base.const_set(:Kernel, InstrumentedKernel)
           end
 
           # Instruments the `Kernel.load` method, but only for the
           # `Rails::Command::RunnerCommand` class.
-          module Kernel
-            def self.load(file, wrap = true)
+          module InstrumentedKernel
+            def self.load(*args)
+              file = args[0]
               name = Ext::SPAN_RUNNER_FILE
               resource = file
               operation = Ext::TAG_OPERATION_FILE

data/lib/datadog/tracing/contrib/rake/instrumentation.rb

@@ -65,7 +65,7 @@ module Datadog
               span.set_tag(Tracing::Metadata::Ext::TAG_OPERATION, Ext::TAG_OPERATION_INVOKE)
               span.set_tag(Ext::TAG_TASK_ARG_NAMES, arg_names)
               span.set_tag(Ext::TAG_INVOKE_ARGS, quantize_args(args)) unless args.nil?
-            rescue StandardError => e
+            rescue => e
               Datadog.logger.debug("Error while tracing Rake invoke: #{e.class.name} #{e.message}")
             end
 
@@ -74,7 +74,7 @@ module Datadog
               span.set_tag(Tracing::Metadata::Ext::TAG_COMPONENT, Ext::TAG_COMPONENT)
               span.set_tag(Tracing::Metadata::Ext::TAG_OPERATION, Ext::TAG_OPERATION_EXECUTE)
               span.set_tag(Ext::TAG_EXECUTE_ARGS, quantize_args(args.to_hash)) unless args.nil?
-            rescue StandardError => e
+            rescue => e
               Datadog.logger.debug("Error while tracing Rake execute: #{e.class.name} #{e.message}")
             end
 
@@ -84,12 +84,12 @@ module Datadog
             end
 
             def enabled?
-              Datadog.configuration.tracing.enabled && \
+              Datadog.configuration.tracing.enabled &&
                 configuration[:enabled] == true
             end
 
             def span_options
-              { service: configuration[:service_name] }
+              {service: configuration[:service_name]}
             end
 
             def configuration

data/lib/datadog/tracing/contrib/rake/integration.rb

@@ -18,7 +18,7 @@ module Datadog
           register_as :rake
 
           def self.version
-            Gem.loaded_specs['rake'] && Gem.loaded_specs['rake'].version
+            Gem.loaded_specs['rake']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/redis/configuration/resolver.rb

@@ -19,13 +19,13 @@ module Datadog
             protected
 
             def parse_matcher(matcher)
-              matcher = { url: matcher } if matcher.is_a?(String)
+              matcher = {url: matcher} if matcher.is_a?(String)
 
               normalize(connection_resolver.resolve(matcher))
             end
 
             def normalize(hash)
-              return { url: hash[:url] } if hash[:scheme] == UNIX_SCHEME
+              return {url: hash[:url]} if hash[:scheme] == UNIX_SCHEME
 
               # Connexion strings are always converted to host, port, db and scheme
               # but the host, port, db and scheme will generate the :url only after

data/lib/datadog/tracing/contrib/redis/ext.rb

@@ -24,10 +24,11 @@ module Datadog
           TAG_OPERATION_COMMAND = 'command'
           TAG_SYSTEM = 'redis'
           TAG_DATABASE_INDEX = 'db.redis.database_index'
-          PEER_SERVICE_SOURCES = Array[
+          PEER_SERVICE_SOURCES = [
             Tracing::Metadata::Ext::TAG_PEER_HOSTNAME,
             Tracing::Metadata::Ext::NET::TAG_DESTINATION_NAME,
-            Tracing::Metadata::Ext::NET::TAG_TARGET_HOST,].freeze
+            Tracing::Metadata::Ext::NET::TAG_TARGET_HOST,
+          ].freeze
         end
       end
     end

data/lib/datadog/tracing/contrib/redis/integration.rb

@@ -31,11 +31,11 @@ module Datadog
           end
 
           def self.redis_version
-            Gem.loaded_specs['redis'] && Gem.loaded_specs['redis'].version
+            Gem.loaded_specs['redis']&.version
           end
 
           def self.redis_client_version
-            Gem.loaded_specs['redis-client'] && Gem.loaded_specs['redis-client'].version
+            Gem.loaded_specs['redis-client']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/redis/patcher.rb

@@ -30,7 +30,7 @@ module Datadog
                 # For `redis-rb` 3.x
                 return client if respond_to?(:client)
 
-                Datadog.logger.warn 'Fail to apply configuration on redis client instance with '  \
+                Datadog.logger.warn 'Fail to apply configuration on redis client instance with ' \
                                                         '`Datadog.configure_onto(redis)`.'
 
                 # Null object instead of raising error
@@ -49,8 +49,8 @@ module Datadog
             module InstanceMethods
               def datadog_pin=(_pin)
                 Datadog.logger.warn \
-                  '`Datadog.configure_onto(redis)` is not supported on Redis 5+. To instrument '\
-                  "a redis instance with custom configuration, please initialize it with:\n"\
+                  '`Datadog.configure_onto(redis)` is not supported on Redis 5+. To instrument ' \
+                  "a redis instance with custom configuration, please initialize it with:\n" \
                   "  `Redis.new(..., custom: { datadog: { service_name: 'my-service' } })`.\n\n" \
                   'See: https://github.com/DataDog/dd-trace-rb/blob/master/docs/GettingStarted.md#redis'
               end
@@ -61,7 +61,7 @@ module Datadog
 
           def default_tags
             [].tap do |tags|
-              tags << "target_redis_version:#{Integration.redis_version}"               if Integration.redis_version
+              tags << "target_redis_version:#{Integration.redis_version}" if Integration.redis_version
               tags << "target_redis_client_version:#{Integration.redis_client_version}" if Integration.redis_client_version
             end
           end

data/lib/datadog/tracing/contrib/redis/quantize.rb

@@ -43,7 +43,7 @@ module Datadog
             return 'AUTH ?' if auth_command?(command_args)
 
             verb, *args = command_args.map { |x| format_arg(x) }
-            Core::Utils.truncate("#{verb.upcase} #{args.join(' ')}", CMD_MAX_LEN, TOO_LONG_MARK)
+            Core::Utils.truncate("#{verb.upcase} #{args.join(" ")}", CMD_MAX_LEN, TOO_LONG_MARK)
           end
 
           def get_verb(command_args)

data/lib/datadog/tracing/contrib/redis/tags.rb

@@ -46,7 +46,7 @@ module Datadog
               span.set_tag Ext::TAG_RAW_COMMAND, raw_command
 
               Contrib::SpanAttributeSchema.set_peer_service!(span, Ext::PEER_SERVICE_SOURCES)
-            rescue StandardError => e
+            rescue => e
               Datadog.logger.error(e.message)
               Datadog::Core::Telemetry::Logger.report(e)
             end

data/lib/datadog/tracing/contrib/redis/trace_middleware.rb

@@ -62,10 +62,10 @@ module Datadog
             # Quantizes a multi-command Redis pipeline execution
             def get_pipeline_commands(commands, command_args)
               list = if command_args
-                       commands.map { |c| Contrib::Redis::Quantize.format_command_args(c) }
-                     else
-                       commands.map { |c| Contrib::Redis::Quantize.get_verb(c) }
-                     end
+                commands.map { |c| Contrib::Redis::Quantize.format_command_args(c) }
+              else
+                commands.map { |c| Contrib::Redis::Quantize.get_verb(c) }
+              end
 
               list.empty? ? '(none)' : list.join("\n")
             end

data/lib/datadog/tracing/contrib/registry.rb

@@ -35,7 +35,7 @@ module Datadog
         def [](name)
           @mutex.synchronize do
             entry = @data[name]
-            entry.klass if entry
+            entry&.klass
           end
         end
 

data/lib/datadog/tracing/contrib/resque/integration.rb

@@ -18,7 +18,7 @@ module Datadog
           register_as :resque, auto_patch: true
 
           def self.version
-            Gem.loaded_specs['resque'] && Gem.loaded_specs['resque'].version
+            Gem.loaded_specs['resque']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/resque/resque_job.rb

@@ -76,7 +76,7 @@ module Datadog
           end
 
           def span_options
-            { service: datadog_configuration[:service_name], on_error: datadog_configuration[:on_error] }
+            {service: datadog_configuration[:service_name], on_error: datadog_configuration[:on_error]}
           end
 
           def datadog_configuration

data/lib/datadog/tracing/contrib/rest_client/ext.rb

@@ -17,10 +17,11 @@ module Datadog
           SPAN_REQUEST = 'rest_client.request'
           TAG_COMPONENT = 'rest_client'
           TAG_OPERATION_REQUEST = 'request'
-          PEER_SERVICE_SOURCES = Array[
+          PEER_SERVICE_SOURCES = [
             Tracing::Metadata::Ext::TAG_PEER_HOSTNAME,
             Tracing::Metadata::Ext::NET::TAG_DESTINATION_NAME,
-            Tracing::Metadata::Ext::NET::TAG_TARGET_HOST,].freeze
+            Tracing::Metadata::Ext::NET::TAG_TARGET_HOST,
+          ].freeze
         end
       end
     end

data/lib/datadog/tracing/contrib/rest_client/integration.rb

@@ -21,7 +21,7 @@ module Datadog
           end
 
           def self.version
-            Gem.loaded_specs['rest-client'] && Gem.loaded_specs['rest-client'].version
+            Gem.loaded_specs['rest-client']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/rest_client/request_patch.rb

@@ -22,7 +22,7 @@ module Datadog
             def execute(&block)
               uri = URI.parse(url)
 
-              return super(&block) unless Tracing.enabled?
+              return super unless Tracing.enabled?
 
               datadog_trace_request(uri) do |_span, trace|
                 if Tracing::Distributed::PropagationPolicy.enabled?(
@@ -106,11 +106,11 @@ module Datadog
               # rubocop:disable Lint/RescueException
             rescue Exception => e
               # rubocop:enable Lint/RescueException
-              span.set_error(e) if span
+              span&.set_error(e)
 
               raise e
             ensure
-              span.finish if span
+              span&.finish
             end
 
             private

data/lib/datadog/tracing/contrib/roda/instrumentation.rb

@@ -46,7 +46,7 @@ module Datadog
               ensure
                 begin
                   response = yield
-                rescue StandardError
+                rescue
                   # The status code is unknown to Roda and decided by the upstream web runner.
                   # In this case, spans default to status code 500 rather than a blank status code.
                   default_error_status = '500'

data/lib/datadog/tracing/contrib/roda/integration.rb

@@ -20,7 +20,7 @@ module Datadog
           register_as :roda
 
           def self.version
-            Gem.loaded_specs['roda'] && Gem.loaded_specs['roda'].version
+            Gem.loaded_specs['roda']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/semantic_logger/instrumentation.rb

@@ -25,7 +25,7 @@ module Datadog
               # if the user already has conflicting log_tags
               # we want them to clobber ours, because we should allow them to override if needed.
               log.named_tags = correlation.to_h.merge(original_named_tags)
-              super(log, message, progname, &block)
+              super
             end
           end
         end

data/lib/datadog/tracing/contrib/semantic_logger/integration.rb

@@ -21,7 +21,7 @@ module Datadog
           register_as :semantic_logger
 
           def self.version
-            Gem.loaded_specs['semantic_logger'] && Gem.loaded_specs['semantic_logger'].version
+            Gem.loaded_specs['semantic_logger']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/sequel/database.rb

@@ -23,7 +23,7 @@ module Datadog
               response = nil
 
               Tracing.trace(Ext::SPAN_QUERY) do |span|
-                span.service =  Datadog.configuration_for(self, :service_name) \
+                span.service = Datadog.configuration_for(self, :service_name) \
                                 || Datadog.configuration.tracing[:sequel][:service_name] \
                                 || Contrib::SpanAttributeSchema.fetch_service_name(
                                   '',
@@ -46,10 +46,10 @@ module Datadog
 
             def parse_opts(sql, opts)
               db_opts = if ::Sequel::VERSION < '3.41.0' && self.class.to_s !~ /Dataset$/
-                          @opts
-                        elsif instance_variable_defined?(:@pool) && @pool
-                          @pool.db.opts
-                        end
+                @opts
+              elsif instance_variable_defined?(:@pool) && @pool
+                @pool.db.opts
+              end
               sql = sql.is_a?(::Sequel::SQL::Expression) ? literal(sql) : sql.to_s
 
               Utils.parse_opts(sql, opts, db_opts)

data/lib/datadog/tracing/contrib/sequel/dataset.rb

@@ -40,7 +40,7 @@ module Datadog
               response = nil
 
               Tracing.trace(Ext::SPAN_QUERY) do |span|
-                span.service =  Datadog.configuration_for(db, :service_name) \
+                span.service = Datadog.configuration_for(db, :service_name) \
                                 || Datadog.configuration.tracing[:sequel][:service_name] \
                                 || Contrib::SpanAttributeSchema.fetch_service_name(
                                   '',

data/lib/datadog/tracing/contrib/sequel/integration.rb

@@ -18,7 +18,7 @@ module Datadog
           register_as :sequel, auto_patch: false
 
           def self.version
-            Gem.loaded_specs['sequel'] && Gem.loaded_specs['sequel'].version
+            Gem.loaded_specs['sequel']&.version
           end
 
           def self.loaded?

data/lib/datadog/tracing/contrib/sequel/utils.rb

@@ -34,7 +34,7 @@ module Datadog
 
             def parse_opts(sql, opts, db_opts, dataset = nil)
               # Prepared statements don't provide their sql query in the +sql+ parameter.
-              if !sql.is_a?(String) && (dataset && dataset.respond_to?(:prepared_sql) &&
+              if !sql.is_a?(String) && (dataset&.respond_to?(:prepared_sql) &&
                 (resolved_sql = dataset.prepared_sql))
                 # The dataset contains the resolved SQL query and prepared statement name.
                 prepared_name = dataset.prepared_statement_name

data/lib/datadog/tracing/contrib/shoryuken/integration.rb

@@ -19,7 +19,7 @@ module Datadog
           register_as :shoryuken
 
           def self.version
-            Gem.loaded_specs['shoryuken'] && Gem.loaded_specs['shoryuken'].version
+            Gem.loaded_specs['shoryuken']&.version
           end
 
           def self.loaded?