appoptics_apm 4.7.0 → 4.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dbff2f2eda2d2c65cf7dbac060957eb6505b3eba743f11dc415f6b06c67217cb
-  data.tar.gz: 6008d6151ab8eb3b29aa465ebb131eef38139d159587e35d5d68dd07de716104
+  metadata.gz: d0ccc84ee6065cc578c0b3dd768825b56a9cd17a1661e4df703621cec10e03c7
+  data.tar.gz: f981f8b2dd4e858b52dc71ad8055c931544a3428a35ca13f30575447fb4a67d9
 SHA512:
-  metadata.gz: 1247aa650e5ff98074d479248c01ff11c8b18f0c63dac47987f79b75efe7518987f43af8ce9adf63cf50df7b9f8d9024deb59da1b9d4b3bd9be56c9d70252114
-  data.tar.gz: 7196764529dfde97d4ca10871d9506a21e411454e5acdd0218945241de2acb22b1ece9e38876ed054d586ed695b276d602ebe116b1f08ff2cf5a98c9537fd5de
+  metadata.gz: 8636c892f94298fb8851641566ca7ccef1f72888c4f37242898993e68278f09904b231c75271a2448cef3530ab7477b36cab000644ab24e605eee9ba0f377c6e
+  data.tar.gz: 96b455b4744604a8d52ce794c070bca3b95543a0e820454d86e3a55ab49a9f22232b9386cecf4bdd1d97585f5e4fdb4e8d796af9ee4cf5138a67736ee0cefedd

data/.travis.yml

@@ -14,7 +14,6 @@ rvm:
   - 2.6.2
   - 2.5.3
   - 2.4.5
-  - 2.3.8
   - ruby-head
 #  - jruby-9.0.5.0
 

data/.yardopts

@@ -1,5 +1,6 @@
 --no-private
 --readme yardoc_frontpage.md
+lib/appoptics_apm/sdk/logging.rb
 lib/appoptics_apm/sdk/tracing.rb
 lib/appoptics_apm/sdk/custom_metrics.rb
 lib/appoptics_apm/sdk/current_trace.rb

data/README.md

@@ -2,7 +2,7 @@
 
 The appoptics_apm gem provides [AppOptics APM](https://www.appoptics.com/) performance instrumentation for Ruby.
 
-![Ruby AppOpticsAPM](https://docs.appoptics.com/_images/ruby_trace.png)
+![Ruby AppOpticsAPM](https://docs.appoptics.com/_images/ruby_trace_smaller.png)
 
 It has the ability to report performance metrics on an array of libraries, databases and frameworks such as Rails, 
 Mongo, Memcache, ActiveRecord, Cassandra, Rack, Resque 

data/Rakefile

@@ -108,6 +108,7 @@ task :fetch_ext_deps do
   # oboe and bson header files
   FileUtils.mkdir_p(File.join(ext_src_dir, 'bson'))
   %w(oboe.h oboe.hpp oboe_debug.h oboe.i bson/bson.h bson/platform_hacks.h).each do |filename|
+  # %w(oboe.h oboe_debug.h bson/bson.h bson/platform_hacks.h).each do |filename|
     remote_file = File.join(oboe_s3_dir, 'include', filename)
     local_file = File.join(ext_src_dir, filename)
 

data/ext/oboe_metal/src/VERSION

@@ -1 +1 @@
-4.1.0
+5.0.0

data/lib/appoptics_apm.rb

@@ -21,22 +21,23 @@ begin
       require 'joboe_metal'
     elsif RUBY_PLATFORM =~ /linux/
       require_relative './oboe_metal.so'
+      require 'appoptics_apm/oboe_init_options'
       require 'oboe_metal.rb'  # sets AppOpticsAPM.loaded = true if successful
     else
-      $stderr.puts '==================================================================='
-      $stderr.puts "AppOptics warning: Platform #{RUBY_PLATFORM} not yet supported."
-      $stderr.puts 'see: https://docs.appoptics.com/kb/apm_tracing/supported_platforms/'
-      $stderr.puts 'Tracing disabled.'
-      $stderr.puts 'Contact support@appoptics.com if this is unexpected.'
-      $stderr.puts '==================================================================='
+      AppOpticsAPM.logger.warn '==================================================================='
+      AppOpticsAPM.logger.warn "AppOptics warning: Platform #{RUBY_PLATFORM} not yet supported."
+      AppOpticsAPM.logger.warn 'see: https://docs.appoptics.com/kb/apm_tracing/supported_platforms/'
+      AppOpticsAPM.logger.warn 'Tracing disabled.'
+      AppOpticsAPM.logger.warn 'Contact support@appoptics.com if this is unexpected.'
+      AppOpticsAPM.logger.warn '==================================================================='
     end
   rescue LoadError => e
     unless ENV['RAILS_GROUP'] == 'assets' or ENV['IGNORE_APPOPTICS_WARNING']
-      $stderr.puts '=============================================================='
-      $stderr.puts 'Missing AppOpticsAPM libraries.  Tracing disabled.'
-      $stderr.puts "Error: #{e.message}"
-      $stderr.puts 'See: https://docs.appoptics.com/kb/apm_tracing/ruby/'
-      $stderr.puts '=============================================================='
+      AppOpticsAPM.logger.error '=============================================================='
+      AppOpticsAPM.logger.error 'Missing AppOpticsAPM libraries.  Tracing disabled.'
+      AppOpticsAPM.logger.error "Error: #{e.message}"
+      AppOpticsAPM.logger.error 'See: https://docs.appoptics.com/kb/apm_tracing/ruby/'
+      AppOpticsAPM.logger.error '=============================================================='
     end
   end
 
@@ -46,8 +47,6 @@ begin
   require 'appoptics_apm/method_profiling'
 
   if AppOpticsAPM.loaded
-    # tracing mode is configured via config file but can only be set once we have oboe_metal loaded
-    AppOpticsAPM.set_tracing_mode(AppOpticsAPM::Config[:tracing_mode].to_sym)
     require 'appoptics_apm/instrumentation'
     require 'appoptics_apm/support/transaction_metrics'
 
@@ -57,10 +56,11 @@ begin
     require 'appoptics_apm/frameworks/padrino'
     require 'appoptics_apm/frameworks/grape'
   else
-    $stderr.puts '=============================================================='
-    $stderr.puts 'AppOpticsAPM not loaded. Tracing disabled.'
-    $stderr.puts 'Service Key may be wrong or missing.'
-    $stderr.puts '=============================================================='
+    AppOpticsAPM.logger.warn '=============================================================='
+    AppOpticsAPM.logger.warn 'AppOpticsAPM not loaded. Tracing disabled.'
+    AppOpticsAPM.logger.warn 'There may be a problem with the service key or other settings.'
+    AppOpticsAPM.logger.warn 'Please check previous log messages.'
+    AppOpticsAPM.logger.warn '=============================================================='
     require 'appoptics_apm/noop/context'
     require 'appoptics_apm/noop/metadata'
   end

data/lib/appoptics_apm/base.rb

@@ -21,13 +21,10 @@ SAMPLE_SOURCE_MASK = 0b1111000000000000000000000000
 # ZERO_SAMPLE_RATE_MASK   = 0b1111000000000000000000000000
 # ZERO_SAMPLE_SOURCE_MASK = 0b0000111111111111111111111111
 
-APPOPTICS_STR_BLANK = ''.freeze
+# APPOPTICS_STR_BLANK = ''.freeze
 APPOPTICS_STR_LAYER = 'Layer'.freeze
 APPOPTICS_STR_LABEL = 'Label'.freeze
 
-# Used in tests to store local trace data
-TRACE_FILE = '/tmp/appoptics_traces.bson'.freeze
-
 ##
 # This module is the base module for the various implementations of AppOpticsAPM reporting.
 # Current variations as of 2014-09-10 are a c-extension, JRuby (using AppOpticsAPM Java

data/lib/appoptics_apm/config.rb

@@ -67,19 +67,22 @@ module AppOpticsAPM
         ].join(' ')
       end
       load(config_files[0])
-      check_env_vars
+      # sets AppOpticsAPM::Config[:debug_level], AppOpticsAPM.logger.level
+      set_log_level
+
+      # the verbose setting is only relevant for ruby, ENV['APPOPTICS_GEM_VERBOSE'] overrides
+      if ENV.key?('APPOPTICS_GEM_VERBOSE')
+        AppOpticsAPM::Config[:verbose] = ENV['APPOPTICS_GEM_VERBOSE'].downcase == 'true'
+      end
     end
 
-    # There are 4 variables that can be set in the config file or as env vars.
-    # Oboe will override vars passed in if it finds an environment variable
-    # :debug_level and :verbose need special consideration, because they are used in Ruby
-    def self.check_env_vars
+    def self.set_log_level
       unless (-1..6).include?(AppOpticsAPM::Config[:debug_level])
         AppOpticsAPM::Config[:debug_level] = 3
       end
 
-      # let's find and use the  equivalent debug level for ruby
-      debug_level = ENV['APPOPTICS_DEBUG_LEVEL'] ? ENV['APPOPTICS_DEBUG_LEVEL'].to_i : AppOpticsAPM::Config[:debug_level]
+      # let's find and use the equivalent debug level for ruby
+      debug_level = (ENV['APPOPTICS_DEBUG_LEVEL'] || AppOpticsAPM::Config[:debug_level] || 3).to_i
       if debug_level < 0
         # there should be no logging if APPOPTICS_DEBUG_LEVEL == -1
         # In Ruby level 5 is UNKNOWN and it can log, but level 6 is quiet
@@ -87,11 +90,7 @@ module AppOpticsAPM
       else
         AppOpticsAPM.logger.level = [4 - debug_level, 0].max
       end
-
-      # the verbose setting is only relevant for ruby, ENV['APPOPTICS_GEM_VERBOSE'] overrides
-      if ENV.key?('APPOPTICS_GEM_VERBOSE')
-        AppOpticsAPM::Config[:verbose] = ENV['APPOPTICS_GEM_VERBOSE'].downcase == 'true'
-      end
+      AppOpticsAPM::Config[:debug_level] = debug_level
     end
 
     ##
@@ -144,9 +143,6 @@ module AppOpticsAPM
       # no guarantee of completeness in the user's config file
       load(File.join(File.dirname(File.dirname(__FILE__)),
                     'rails/generators/appoptics_apm/templates/appoptics_apm_initializer.rb'))
-
-      # to make sure we include the service_key if it is set as an ENV var
-      check_env_vars
     end
     # rubocop:enable Metrics/AbcSize
 
@@ -243,11 +239,12 @@ module AppOpticsAPM
           @@config[i][:log_args] = value
         end
 
-      # Update liboboe if updating :tracing_mode
       elsif key == :tracing_mode
-        AppOpticsAPM.set_tracing_mode(value.to_sym) if AppOpticsAPM.loaded
+      #   CAN'T DO THIS ANYMORE, ALL TRACING COMMUNICATION TO OBOE
+      #   IS NOW HANDLED BY TransactionSettings
+      #   AppOpticsAPM.set_tracing_mode(value.to_sym) if AppOpticsAPM.loaded
 
-        # Make sure that the mode is stored as a symbol
+      # Make sure that the mode is stored as a symbol
         @@config[key.to_sym] = value.to_sym
       end
     end

data/lib/appoptics_apm/inst/sidekiq-worker.rb

@@ -35,11 +35,12 @@ module AppOpticsAPM
       # args: 0: worker, 1: msg, 2: queue
       report_kvs = collect_kvs(args)
 
+      # TODO remove completely once it is determined that this works without
       # Something is happening across Celluloid threads where liboboe settings
       # are being lost.  So we re-set the tracing mode to assure
       # we sample as desired.  Setting the tracing mode will re-update
       # the liboboe settings.
-      AppOpticsAPM.set_tracing_mode(AppOpticsAPM::Config[:tracing_mode].to_sym)
+      # AppOpticsAPM.set_tracing_mode(AppOpticsAPM::Config[:tracing_mode].to_sym)
 
       # Continue the trace from the enqueue side?
       if args[1].is_a?(Hash) && AppOpticsAPM::XTrace.valid?(args[1]['SourceTrace'])

data/lib/appoptics_apm/oboe_init_options.rb

@@ -0,0 +1,180 @@
+# Copyright (c) 2019 SolarWinds, LLC.
+# All rights reserved.
+
+require 'singleton'
+
+module AppOpticsAPM
+
+  class OboeInitOptions
+    include Singleton
+
+    attr_reader :reporter, :host, :service_name  # exposing these mainly for testing
+
+    # TODO decide if these globals are useful when testing
+    # OBOE_HOSTNAME_ALIAS = 0
+    # OBOE_DEBUG_LEVEL = 1
+    # OBOE_LOGFILE = 2
+    #
+    # OBOE_MAX_TRANSACTIONS = 3
+    # OBOE_FLUSH_MAX_WAIT_TIME = 4
+    # OBOE_EVENTS_FLUSH_INTERVAL = 5
+    # OBOE_EVENTS_FLUSH_BATCH_SIZE = 6
+    #
+    # OBOE_REPORTER = 7
+    # OBOE_COLLECTOR = 8
+    # OBOE_SERVICE_KEY = 9
+    # OBOE_TRUSTEDPATH = 10
+    #
+    # OBOE_BUFSIZE = 11
+    # OBOE_TRACE_METRICS = 12
+    # OBOE_HISTOGRAM_PRECISION = 13
+    # OBOE_TOKEN_BUCKET_CAPACITY = 14
+    # OBOE_TOKEN_BUCKET_RATE = 15
+    # OBOE_FILE_SINGLE = 16
+
+    def initialize
+      # optional hostname alias
+      @hostname_alias = ENV['APPOPTICS_HOSTNAME_ALIAS'] || AppOpticsAPM::Config[:hostname_alias] || ''
+      # level at which log messages will be written to log file (0-6)
+      @debug_level = (ENV['APPOPTICS_DEBUG_LEVEL'] || AppOpticsAPM::Config[:debug_level] || 3).to_i
+      # file name including path for log file
+      # TODO eventually find better way to combine ruby and oboe logs
+      @log_file_path = ENV['APPOPTICS_LOGFILE'] || ''
+      # maximum number of transaction names to track
+      @max_transactions = (ENV['APPOPTICS_MAX_TRANSACTIONS'] || -1).to_i
+      # maximum wait time for flushing data before terminating in milli seconds
+      @max_flush_wait_time = (ENV['APPOPTICS_FLUSH_MAX_WAIT_TIME'] || -1).to_i
+      # events flush timeout in seconds (threshold for batching messages before sending off)
+      @events_flush_interval = (ENV['APPOPTICS_EVENTS_FLUSH_INTERVAL'] || -1).to_i
+      # events flush batch size in KB (threshold for batching messages before sending off)
+      @event_flush_batch_size = (ENV['APPOPTICS_EVENTS_FLUSH_BATCH_SIZE'] || -1).to_i
+
+      # the reporter to be used (ssl, upd, file, null)
+      # collector endpoint (reporter=ssl), udp address (reporter=udp), or file path (reporter=file)
+      @reporter, @host = reporter_and_host
+
+      # the service key
+      @service_key = read_and_validate_service_key
+      # path to the SSL certificate (only for ssl)
+      @trusted_path = ENV['APPOPTICS_TRUSTEDPATH'] || ''
+      # size of the message buffer
+      @buffer_size = (ENV['APPOPTICS_BUFSIZE'] || -1).to_i
+      # flag indicating if trace metrics reporting should be enabled (default) or disabled
+      @trace_metrics = (ENV['APPOPTICS_TRACE_METRICS'] || -1).to_i
+      # the histogram precision (only for ssl)
+      @histogram_precision = (ENV['APPOPTICS_HISTOGRAM_PRECISION'] || -1).to_i
+      # custom token bucket capacity
+      @token_bucket_capacity = (ENV['APPOPTICS_TOKEN_BUCKET_CAPACITY'] || -1).to_i
+      # custom token bucket rate
+      @token_bucket_rate = (ENV['APPOPTICS_TOKEN_BUCKET_RATE'] || -1).to_i
+      # use single files in file reporter for each event
+      @file_single = (ENV['APPOPTICS_REPORTER_FILE_SINGLE'].to_s.downcase == 'true') ? 1 : 0
+    end
+
+    def re_init # for testing with changed ENV vars
+      initialize
+    end
+
+    def array_for_oboe
+      [
+        @hostname_alias,
+        @debug_level,
+        @log_file_path,
+        @max_transactions,
+        @max_flush_wait_time,
+        @events_flush_interval,
+        @event_flush_batch_size,
+
+        @reporter,
+        @host,
+        @service_key,
+        @trusted_path,
+        @buffer_size,
+        @trace_metrics,
+        @histogram_precision,
+        @token_bucket_capacity,
+        @token_bucket_rate,
+        @file_single
+      ]
+    end
+
+    def service_key_ok?
+      return !@service_key.empty? || @reporter != 'ssl'
+    end
+
+    private
+
+    def reporter_and_host
+
+      reporter = ENV['APPOPTICS_REPORTER'] || 'ssl'
+      # override with 'file', e.g. when running tests
+      reporter = 'file' if ENV.key?('APPOPTICS_GEM_TEST')
+
+      host = ''
+      case reporter
+      when 'ssl', 'file'
+        host = ENV['APPOPTICS_COLLECTOR'] || ''
+      when 'udp'
+        host = ENV['APPOPTICS_COLLECTOR'] ||
+                "#{AppOpticsAPM::Config[:reporter_host]}:#{AppOpticsAPM::Config[:reporter_port]}"
+        # TODO decide what to do
+        # ____ AppOpticsAPM::Config[:reporter_host] and
+        # ____ AppOpticsAPM::Config[:reporter_port] were moved here from
+        # ____ oboe_metal.rb and are not documented anywhere
+      when 'null'
+        host = ''
+      end
+
+      [reporter, host]
+    end
+
+    def read_and_validate_service_key
+      return '' unless @reporter == 'ssl'
+
+      service_key = ENV['APPOPTICS_SERVICE_KEY'] || AppOpticsAPM::Config[:service_key]
+      unless service_key
+        AppOpticsAPM.logger.error "[appoptics_apm/oboe_options] APPOPTICS_SERVICE_KEY not configured."
+        return ''
+      end
+
+      match = service_key.match( /([^:]+)(:{0,1})(.*)/ )
+      token = match[1]
+      service_name = match[3]
+
+      return '' unless validate_token(token)
+      return '' unless validate_transform_service_name(service_name)
+
+      return "#{token}:#{service_name}"
+    end
+
+    def validate_token(token)
+      if (token !~ /^[0-9a-fA-F]{64}|[0-9a-zA-Z_\-]{71}$/)
+        masked = "#{token[0..3]}...#{token[-4..-1]}"
+        AppOpticsAPM.logger.error "[appoptics_apm/oboe_options] APPOPTICS_SERVICE_KEY problem. API Token in wrong format. Masked token: #{masked}"
+        return false
+      end
+
+      true
+    end
+
+    def validate_transform_service_name(service_name)
+      if service_name.empty?
+        AppOpticsAPM.logger.error "[appoptics_apm/oboe_options] APPOPTICS_SERVICE_KEY problem. Service Name is missing"
+        return false
+      end
+
+      name = service_name.dup
+      name.downcase!
+      name.gsub!(/[^a-z0-9.:_-]/, '')
+      name = name[0..254]
+
+      if name != service_name
+        AppOpticsAPM.logger.warn "[appoptics_apm/oboe_options] APPOPTICS_SERVICE_KEY problem. Service Name transformed from #{service_name} to #{name}"
+        service_name = name
+      end
+      @service_name = service_name # instance variable used in testing
+      true
+    end
+  end
+end
+

data/lib/appoptics_apm/support/transaction_settings.rb

@@ -17,7 +17,7 @@ AO_TRACING_DECISIONS_QUEUE_FULL = 5
 
 module AppOpticsAPM
   ##
-  # This module helps with setting up the filters and applying them
+  # This module helps with setting up the transaction filters and applying them
   #
   class TransactionSettings
 
@@ -40,8 +40,9 @@ module AppOpticsAPM
         return
       end
 
-      if AppOpticsAPM.tracing_disabled? && !tracing_enabled?(url) ||
+      if tracing_mode_disabled? && !tracing_enabled?(url) ||
         tracing_disabled?(url)
+
         tracing_mode = AO_TRACING_DISABLED
       end
 
@@ -65,6 +66,14 @@ module AppOpticsAPM
     end
 
     private
+
+    ##
+    # check the config setting for :tracing_mode
+    def tracing_mode_disabled?
+      AppOpticsAPM::Config[:tracing_mode] &&
+        [:disabled, :never].include?(AppOpticsAPM::Config[:tracing_mode])
+    end
+
     ##
     # tracing_enabled?
     #

data/lib/appoptics_apm/version.rb

@@ -6,9 +6,9 @@ module AppOpticsAPM
   # The current version of the gem.  Used mainly by
   # appoptics_apm.gemspec during gem build process
   module Version
-    MAJOR = 4
-    MINOR = 7
-    PATCH = 0
+    MAJOR = 4 # breaking,
+    MINOR = 8 # feature,
+    PATCH = 0 # fix => BFF
 
     STRING = [MAJOR, MINOR, PATCH].compact.join('.')
   end

data/lib/oboe_metal.rb

@@ -18,25 +18,12 @@ module AppOpticsAPM
       # Start the AppOpticsAPM Reporter
       #
       def start
+        AppOpticsAPM.loaded = false unless AppOpticsAPM::OboeInitOptions.instance.service_key_ok?
         return unless AppOpticsAPM.loaded
 
         begin
-          options = []
-
-          ENV['APPOPTICS_REPORTER'] = 'file' if ENV.key?('APPOPTICS_GEM_TEST')
-
-          case ENV['APPOPTICS_REPORTER']
-          when 'file'
-            ENV['APPOPTICS_REPORTER_FILE'] = TRACE_FILE
-          when 'udp'
-            ENV['APPOPTICS_REPORTER_UDP'] = "#{AppOpticsAPM::Config[:reporter_host]}:#{AppOpticsAPM::Config[:reporter_port]}"
-          else # default is ssl, service_key is mandatory
-            return unless valid_service_key?
-            # Oboe will override these settings if there are env settings for them
-            options << AppOpticsAPM::Config[:service_key].to_s
-            options << AppOpticsAPM::Config[:hostname_alias].to_s
-            options << AppOpticsAPM::Config[:debug_level] unless AppOpticsAPM::Config[:debug_level].nil?
-          end
+          options = AppOpticsAPM::OboeInitOptions.instance.array_for_oboe # creates an array with the options in the right order
+
           AppOpticsAPM.reporter = Oboe_metal::Reporter.new(*options)
 
           # Only report __Init from here if we are not instrumenting a framework.
@@ -52,19 +39,6 @@ module AppOpticsAPM
       end
       alias :restart :start
 
-      def valid_service_key?
-        service_key = (ENV['APPOPTICS_SERVICE_KEY'] || AppOpticsAPM::Config[:service_key]).to_s
-        if service_key == ''
-          AppOpticsAPM.logger.warn '[appoptics_apm/warn] APPOPTICS_SERVICE_KEY not set. Cannot submit data.'
-          AppOpticsAPM.loaded = false
-          return false
-        elsif service_key !~ /^[0-9a-fA-F]{64}:[-.:_?\\\/\w ]{1,255}$/
-          AppOpticsAPM.logger.warn '[appoptics_apm/warn] APPOPTICS_SERVICE_KEY problem. No service name or api token in wrong format. Cannot submit data.'
-          AppOpticsAPM.loaded = false
-          return false
-        end
-        true
-      end
       ##
       # sendReport
       #
@@ -89,7 +63,7 @@ module AppOpticsAPM
       # Truncates the trace output file to zero
       #
       def clear_all_traces
-        File.truncate(TRACE_FILE, 0)
+        File.truncate(AppOpticsAPM::OboeInitOptions.instance.host, 0)
       end
 
       ##
@@ -98,14 +72,14 @@ module AppOpticsAPM
       # Retrieves all traces written to the trace file
       #
       def get_all_traces
-        io = File.open(TRACE_FILE, 'r')
+        io = File.open(AppOpticsAPM::OboeInitOptions.instance.host, 'r')
         contents = io.readlines(nil)
+        io.close
 
         return contents if contents.empty?
 
         traces = []
 
-        #
         # We use Gem.loaded_spec because older versions of the bson
         # gem didn't even have a version embedded in the gem.  If the
         # gem isn't in the bundle, it should rightfully error out
@@ -140,49 +114,49 @@ module AppOpticsAPM
   end
 
   class << self
-    def sample?(opts = {})
-      # Return false if no-op mode
-      return false unless AppOpticsAPM.loaded
-
-      # Assure defaults since SWIG enforces Strings
-      xtrace  = opts[:xtrace]     ? opts[:xtrace].to_s.strip       : APPOPTICS_STR_BLANK
-
-      # the first arg has changed to be the service name, blank means to use the default (from the service key)
-      rv = AppOpticsAPM::Context.sampleRequest(APPOPTICS_STR_BLANK, xtrace)
-
-      if rv == 0
-        AppOpticsAPM.sample_rate = -1
-        AppOpticsAPM.sample_source = -1
-        false
-      else
-        # liboboe version > 1.3.1 returning a bit masked integer with SampleRate and
-        # source embedded
-        AppOpticsAPM.sample_rate = (rv & SAMPLE_RATE_MASK)
-        AppOpticsAPM.sample_source = (rv & SAMPLE_SOURCE_MASK) >> 24
-        true
-      end
-    rescue StandardError => e
-      AppOpticsAPM.logger.debug "[oboe/error] sample? error: #{e.inspect}"
-      false
-    end
-
-    def set_tracing_mode(mode)
-      return unless AppOpticsAPM.loaded
-
-      value = mode.to_sym
-
-      case value
-      when :disabled, :never
-        AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_DISABLED)
-
-      when :enabled, :always
-        AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_ENABLED)
-
-      else
-        AppOpticsAPM.logger.fatal "[oboe/error] Invalid tracing mode set: #{mode}"
-        AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_DISABLED)
-      end
-    end
+    # def sample?(opts = {})
+    #   # Return false if no-op mode
+    #   return false unless AppOpticsAPM.loaded
+    #
+    #   # Assure defaults since SWIG enforces Strings
+    #   xtrace  = opts[:xtrace]     ? opts[:xtrace].to_s.strip       : APPOPTICS_STR_BLANK
+    #
+    #   # the first arg has changed to be the service name, blank means to use the default (from the service key)
+    #   rv = AppOpticsAPM::Context.sampleRequest(APPOPTICS_STR_BLANK, xtrace)
+    #
+    #   if rv == 0
+    #     AppOpticsAPM.sample_rate = -1
+    #     AppOpticsAPM.sample_source = -1
+    #     false
+    #   else
+    #     # liboboe version > 1.3.1 returning a bit masked integer with SampleRate and
+    #     # source embedded
+    #     AppOpticsAPM.sample_rate = (rv & SAMPLE_RATE_MASK)
+    #     AppOpticsAPM.sample_source = (rv & SAMPLE_SOURCE_MASK) >> 24
+    #     true
+    #   end
+    # rescue StandardError => e
+    #   AppOpticsAPM.logger.debug "[oboe/error] sample? error: #{e.inspect}"
+    #   false
+    # end
+
+    # def set_tracing_mode(mode)
+    #   return unless AppOpticsAPM.loaded
+    #
+    #   value = mode.to_sym
+    #
+    #   case value
+    #   when :disabled, :never
+    #     AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_DISABLED)
+    #
+    #   when :enabled, :always
+    #     AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_ENABLED)
+    #
+    #   else
+    #     AppOpticsAPM.logger.fatal "[oboe/error] Invalid tracing mode set: #{mode}"
+    #     AppOpticsAPM::Context.setTracingMode(APPOPTICS_TRACE_DISABLED)
+    #   end
+    # end
 
     def set_sample_rate(rate)
       return unless AppOpticsAPM.loaded

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: appoptics_apm
 version: !ruby/object:Gem::Version
-  version: 4.7.0
+  version: 4.8.0
 platform: ruby
 authors:
 - Maia Engeli
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-23 00:00:00.000000000 Z
+date: 2019-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -208,6 +208,7 @@ files:
 - lib/appoptics_apm/noop/README.md
 - lib/appoptics_apm/noop/context.rb
 - lib/appoptics_apm/noop/metadata.rb
+- lib/appoptics_apm/oboe_init_options.rb
 - lib/appoptics_apm/ruby.rb
 - lib/appoptics_apm/sdk/current_trace.rb
 - lib/appoptics_apm/sdk/custom_metrics.rb