instana 1.192.1 → 1.193.5

data/lib/instana/instrumentation/instrumented_request.rb

@@ -1,6 +1,11 @@
 # Note: We really only need "cgi/util" here but Ruby 2.4.1 has an issue:
 # https://bugs.ruby-lang.org/issues/13539
-require "cgi"
+
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2021
+
+require 'cgi'
+require 'rack/request'
 
 module Instana
   class InstrumentedRequest < Rack::Request

data/lib/instana/instrumentation/net-http.rb

@@ -1,61 +1,60 @@
-require 'net/http'
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2016
 
-if defined?(::Net::HTTP) && ::Instana.config[:nethttp][:enabled]
-  Net::HTTP.class_eval {
+require 'net/http'
 
-    def request_with_instana(*args, &block)
-      if !Instana.tracer.tracing? || !started?
-        do_skip = true
-        return request_without_instana(*args, &block)
-      end
+module Instana
+  module Instrumentation
+    module NetHTTPInstrumentation
+      def request(*args, &block)
+        if !Instana.tracer.tracing? || !started?
+          do_skip = true
+          return super(*args, &block)
+        end
 
-      ::Instana.tracer.log_entry(:'net-http')
+        ::Instana.tracer.log_entry(:'net-http')
 
-      # Send out the tracing context with the request
-      request = args[0]
+        # Send out the tracing context with the request
+        request = args[0]
 
-      # Set request headers; encode IDs as hexadecimal strings
-      t_context = ::Instana.tracer.context
-      request['X-Instana-T'] = t_context.trace_id_header
-      request['X-Instana-S'] = t_context.span_id_header
+        # Set request headers; encode IDs as hexadecimal strings
+        t_context = ::Instana.tracer.context
+        request['X-Instana-T'] = t_context.trace_id_header
+        request['X-Instana-S'] = t_context.span_id_header
 
-      # Collect up KV info now in case any exception is raised
-      kv_payload = { :http => {} }
-      kv_payload[:http][:method] = request.method
+        # Collect up KV info now in case any exception is raised
+        kv_payload = { :http => {} }
+        kv_payload[:http][:method] = request.method
 
-      if request.uri
-        kv_payload[:http][:url] = request.uri.to_s
-      else
-        if use_ssl?
-          kv_payload[:http][:url] = "https://#{@address}:#{@port}#{request.path}"
+        if request.uri
+          kv_payload[:http][:url] = request.uri.to_s
         else
-          kv_payload[:http][:url] = "http://#{@address}:#{@port}#{request.path}"
+          if use_ssl?
+            kv_payload[:http][:url] = "https://#{@address}:#{@port}#{request.path}"
+          else
+            kv_payload[:http][:url] = "http://#{@address}:#{@port}#{request.path}"
+          end
         end
-      end
-      
-      kv_payload[:http][:url] = ::Instana.secrets.remove_from_query(kv_payload[:http][:url])
 
-      # The core call
-      response = request_without_instana(*args, &block)
+        kv_payload[:http][:url] = ::Instana.secrets.remove_from_query(kv_payload[:http][:url])
 
-      kv_payload[:http][:status] = response.code
-      if response.code.to_i.between?(500, 511)
-        # Because of the 5xx response, we flag this span as errored but
-        # without a backtrace (no exception)
-        ::Instana.tracer.log_error(nil)
-      end
+        # The core call
+        response = super(*args, &block)
 
-      response
-    rescue => e
-      ::Instana.tracer.log_error(e)
-      raise
-    ensure
-      ::Instana.tracer.log_exit(:'net-http', kv_payload) unless do_skip
-    end
-
-    ::Instana.logger.debug "Instrumenting Net::HTTP"
+        kv_payload[:http][:status] = response.code
+        if response.code.to_i.between?(500, 511)
+          # Because of the 5xx response, we flag this span as errored but
+          # without a backtrace (no exception)
+          ::Instana.tracer.log_error(nil)
+        end
 
-    alias request_without_instana request
-    alias request request_with_instana
-  }
+        response
+      rescue => e
+        ::Instana.tracer.log_error(e)
+        raise
+      ensure
+        ::Instana.tracer.log_exit(:'net-http', kv_payload) unless do_skip
+      end
+    end
+  end
 end

data/lib/instana/instrumentation/rack.rb

@@ -1,3 +1,7 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2016
+
+require 'rack'
 require 'instana/instrumentation/instrumented_request'
 
 module Instana

data/lib/instana/instrumentation/redis.rb

@@ -1,11 +1,14 @@
-if defined?(::Redis) && ::Instana.config[:redis][:enabled]
-  ::Redis::Client.class_eval do
-    def call_with_instana(*args, &block)
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2017
+
+module Instana
+  module RedisInstrumentation
+    def call(*args, &block)
       kv_payload = { redis: {} }
       dnt_spans = [:redis, :'resque-client', :'sidekiq-client']
 
-      if !Instana.tracer.tracing? || dnt_spans.include?(::Instana.tracer.current_span.name)
-        return call_without_instana(*args, &block)
+      if !Instana.tracer.tracing? || dnt_spans.include?(::Instana.tracer.current_span.name) || !Instana.config[:redis][:enabled]
+        return super(*args, &block)
       end
 
       begin
@@ -19,7 +22,7 @@ if defined?(::Redis) && ::Instana.config[:redis][:enabled]
           nil
         end
 
-        call_without_instana(*args, &block)
+        super(*args, &block)
       rescue => e
         ::Instana.tracer.log_info({ redis: {error: true} })
         ::Instana.tracer.log_error(e)
@@ -29,17 +32,12 @@ if defined?(::Redis) && ::Instana.config[:redis][:enabled]
       end
     end
 
-    ::Instana.logger.debug "Instrumenting Redis"
-
-    alias call_without_instana call
-    alias call call_with_instana
-
-    def call_pipeline_with_instana(*args, &block)
+    def call_pipeline(*args, &block)
       kv_payload = { redis: {} }
       dnt_spans = [:redis, :'resque-client', :'sidekiq-client']
 
-      if !Instana.tracer.tracing? || dnt_spans.include?(::Instana.tracer.current_span.name)
-        return call_pipeline_without_instana(*args, &block)
+      if !Instana.tracer.tracing? || dnt_spans.include?(::Instana.tracer.current_span.name) || !Instana.config[:redis][:enabled]
+        return super(*args, &block)
       end
 
       begin
@@ -54,7 +52,7 @@ if defined?(::Redis) && ::Instana.config[:redis][:enabled]
           nil
         end
 
-        call_pipeline_without_instana(*args, &block)
+        super(*args, &block)
       rescue => e
         ::Instana.tracer.log_info({ redis: {error: true} })
         ::Instana.tracer.log_error(e)
@@ -63,8 +61,5 @@ if defined?(::Redis) && ::Instana.config[:redis][:enabled]
         ::Instana.tracer.log_exit(:redis, kv_payload)
       end
     end
-
-    alias call_pipeline_without_instana call_pipeline
-    alias call_pipeline call_pipeline_with_instana
   end
 end

data/lib/instana/instrumentation/resque.rb

@@ -1,13 +1,13 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2018
+
 require 'socket'
 
 module Instana
   module Instrumentation
     module ResqueClient
-      def self.included(klass)
+      def self.prepended(klass)
         klass.send :extend, ::Resque
-        ::Instana::Util.method_alias(klass, :enqueue)
-        ::Instana::Util.method_alias(klass, :enqueue_to)
-        ::Instana::Util.method_alias(klass, :dequeue)
       end
 
       def collect_kvs(op, klass, args)
@@ -23,50 +23,46 @@ module Instana
         { :'resque-client' => kvs }
       end
 
-      def enqueue_with_instana(klass, *args)
+      def enqueue(klass, *args)
         if Instana.tracer.tracing?
           kvs = collect_kvs(:enqueue, klass, args)
 
           Instana.tracer.trace(:'resque-client', kvs) do
-            enqueue_without_instana(klass, *args)
+            super(klass, *args)
           end
         else
-          enqueue_without_instana(klass, *args)
+          super(klass, *args)
         end
       end
 
-      def enqueue_to_with_instana(queue, klass, *args)
+      def enqueue_to(queue, klass, *args)
         if Instana.tracer.tracing? && !Instana.tracer.tracing_span?(:'resque-client')
           kvs = collect_kvs(:enqueue_to, klass, args)
           kvs[:Queue] = queue.to_s if queue
 
           Instana.tracer.trace(:'resque-client', kvs) do
-            enqueue_to_without_instana(queue, klass, *args)
+            super(queue, klass, *args)
           end
         else
-          enqueue_to_without_instana(queue, klass, *args)
+          super(queue, klass, *args)
         end
       end
 
-      def dequeue_with_instana(klass, *args)
+      def dequeue(klass, *args)
         if Instana.tracer.tracing?
           kvs = collect_kvs(:dequeue, klass, args)
 
           Instana.tracer.trace(:'resque-client', kvs) do
-            dequeue_without_instana(klass, *args)
+            super(klass, *args)
           end
         else
-          dequeue_without_instana(klass, *args)
+          super(klass, *args)
         end
       end
     end
 
     module ResqueWorker
-      def self.included(klass)
-        ::Instana::Util.method_alias(klass, :perform)
-      end
-
-      def perform_with_instana(job)
+      def perform(job)
         kvs = {}
         kvs[:'resque-worker'] = {}
 
@@ -81,17 +77,13 @@ module Instana
         end
 
         Instana.tracer.start_or_continue_trace(:'resque-worker', kvs) do
-          perform_without_instana(job)
+          super(job)
         end
       end
     end
 
     module ResqueJob
-      def self.included(klass)
-        ::Instana::Util.method_alias(klass, :fail)
-      end
-
-      def fail_with_instana(exception)
+      def fail(exception)
         if Instana.tracer.tracing?
           ::Instana.tracer.log_info(:'resque-worker' => { :error => "#{exception.class}: #{exception}"})
           ::Instana.tracer.log_error(exception)
@@ -99,33 +91,8 @@ module Instana
       rescue Exception => e
         ::Instana.logger.debug { "#{__method__}:#{File.basename(__FILE__)}:#{__LINE__}: #{e.message}" } if Instana::Config[:verbose]
       ensure
-        fail_without_instana(exception)
+        super(exception)
       end
     end
   end
 end
-
-if defined?(::Resque) && RUBY_VERSION >= '1.9.3'
-
-  if ::Instana.config[:'resque-client'][:enabled]
-    ::Instana.logger.debug 'Instrumenting Resque Client'
-    ::Instana::Util.send_include(::Resque,         ::Instana::Instrumentation::ResqueClient)
-  end
-
-  if ::Instana.config[:'resque-worker'][:enabled]
-    ::Instana.logger.debug 'Instrumenting Resque Worker'
-
-    ::Instana::Util.send_include(::Resque::Worker, ::Instana::Instrumentation::ResqueWorker)
-    ::Instana::Util.send_include(::Resque::Job,    ::Instana::Instrumentation::ResqueJob)
-
-    ::Resque.before_fork do |job|
-      ::Instana.agent.before_resque_fork
-    end
-    ::Resque.after_fork do |job|
-      ::Instana.agent.after_resque_fork
-    end
-
-    # Set this so we assure that any remaining collected traces are reported at_exit
-    ENV['RUN_AT_EXIT_HOOKS'] = "1"
-  end
-end

data/lib/instana/instrumentation/rest-client.rb

@@ -1,34 +1,22 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2016
+
 module Instana
   module Instrumentation
     module RestClientRequest
-      def self.included(klass)
-        if klass.method_defined?(:execute)
-          klass.class_eval do
-            alias execute_without_instana execute
-            alias execute execute_with_instana
-          end
-        end
-      end
-
-      def execute_with_instana & block
+      def execute(&block)
         # Since RestClient uses net/http under the covers, we just
         # provide span visibility here.  HTTP related KVs are reported
         # in the Net::HTTP instrumentation
         ::Instana.tracer.log_entry(:'rest-client')
 
-        execute_without_instana(&block)
+        super(&block)
       rescue => e
         ::Instana.tracer.log_error(e)
         raise
       ensure
         ::Instana.tracer.log_exit(:'rest-client')
       end
-
     end
   end
 end
-
-if defined?(::RestClient::Request) && ::Instana.config[:'rest-client'][:enabled]
-  ::Instana.logger.debug "Instrumenting RestClient"
-  ::RestClient::Request.send(:include, ::Instana::Instrumentation::RestClientRequest)
-end

data/lib/instana/instrumentation/sidekiq-client.rb

@@ -1,3 +1,6 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2017
+
 module Instana
   module Instrumentation
     class SidekiqClient
@@ -34,12 +37,3 @@ module Instana
     end
   end
 end
-
-if defined?(::Sidekiq) && ::Instana.config[:'sidekiq-client'][:enabled]
-  ::Sidekiq.configure_client do |cfg|
-    cfg.client_middleware do |chain|
-      ::Instana.logger.debug "Instrumenting Sidekiq client"
-      chain.add ::Instana::Instrumentation::SidekiqClient
-    end
-  end
-end

data/lib/instana/instrumentation/sidekiq-worker.rb

@@ -1,3 +1,6 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2017
+
 module Instana
   module Instrumentation
     class SidekiqWorker
@@ -43,12 +46,3 @@ module Instana
     end
   end
 end
-
-if defined?(::Sidekiq) && ::Instana.config[:'sidekiq-worker'][:enabled]
-  ::Sidekiq.configure_server do |cfg|
-    cfg.server_middleware do |chain|
-      ::Instana.logger.debug "Instrumenting Sidekiq worker"
-      chain.add ::Instana::Instrumentation::SidekiqWorker
-    end
-  end
-end

data/lib/instana/opentracing/carrier.rb

@@ -1,3 +1,6 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2017
+
 module OpenTracing
   class Carrier < Hash
   end

data/lib/instana/opentracing/tracer.rb

@@ -1,3 +1,6 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2017
+
 module OpenTracing
   class << self
     # Text format for #inject and #extract

data/lib/instana/rack.rb

@@ -7,4 +7,8 @@
 # require 'instana/rack'
 # config.middleware.use ::Instana::Rack
 #
+
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2016
+
 require 'instana/instrumentation/rack'

data/lib/instana/secrets.rb

@@ -1,26 +1,29 @@
+# (c) Copyright IBM Corp. 2021
+# (c) Copyright Instana Inc. 2021
+
 require 'uri'
 require 'cgi'
 
 module Instana
-  class Secrets    
+  class Secrets
     def remove_from_query(str, secret_values = Instana.agent.secret_values)
       return str unless secret_values
-      
+
       url = URI(str)
-      params = CGI.parse(url.query)
-      
+      params = CGI.parse(url.query || '')
+
       redacted = params.map do |k, v|
         needs_redaction = secret_values['list']
           .any? { |t| matcher(secret_values['matcher']).(t,k) }
         [k, needs_redaction ? '<redacted>' : v]
       end
-      
+
       url.query = URI.encode_www_form(redacted)
       CGI.unescape(url.to_s)
     end
-    
+
     private
-    
+
     def matcher(name)
       case name
       when 'equals-ignore-case'