bc-lightstep-ruby 2.1.1 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d8f1fc7a5f74205acf72c3fa8cbb6c138c5a08ad836a1e16a5dfa18eac508e61
-  data.tar.gz: e83b6493f8e1f0960b5c5d4c160ad6908c0f18f12be6faf39b71196ba4d6ccfb
+  metadata.gz: f8438758949aeee5ef6d8908fdb428a466d768b9548c50ec11d91ebfe6d4a51d
+  data.tar.gz: 1afbe2aca9eed46303b8a3a40802ead2fad62ed937638213e74f257149a3c3b2
 SHA512:
-  metadata.gz: a5e1791d8d2d2557eb01ec82cfe25cda4bff66637f325dceba731c44d2669f954afcdcf73f68e6dd05561e407ebc4db1e9b01af493f5652081efd43c588c79b6
-  data.tar.gz: 0353ccabf22f3e023ce98d41957f2ee8209c5669b9417cc5f750b3f6ebafa32a7265b7e69ebf74c1de6c9f82494cfe215d2aadea52c27dfcc428b83b57f60a19
+  metadata.gz: ddfe877c18f72a78694074fa5d71ca8cf405638fcb4dc171417f5371fb8ea3a0d3048933e7ecf1bee1b1b785cd93e965485cf16df31ab224dea8b261797f0496
+  data.tar.gz: 60a2238aa127269ba871c21d0a4a6749ba3bcbc1752bc658fec9f1b055cc77e6b6bc3125eadd997a4b436788a90d0b909a959927b1a83718348cfa33ce7981c0

data/CHANGELOG.md

@@ -2,6 +2,11 @@ Changelog for the bc-lightstep-ruby gem.
 
 ### Pending Release
 
+### 2.2.0
+
+- Only set error tag in exceptions if not already set
+- Update to Rubocop 1.0
+
 ### 2.1.1
 
 - Bump lightstep gem to 0.16.0

data/bc-lightstep-ruby.gemspec

@@ -39,7 +39,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '>= 12.0'
   spec.add_development_dependency 'rspec', '~> 3.8'
   spec.add_development_dependency 'rspec_junit_formatter', '~> 0.4'
-  spec.add_development_dependency 'rubocop', '~> 0.79'
+  spec.add_development_dependency 'rubocop', '~> 1.0'
   spec.add_development_dependency 'simplecov', '~> 0.15'
   spec.add_development_dependency 'pry', '>= 0.12'
 

data/lib/bigcommerce/lightstep.rb

@@ -42,6 +42,12 @@ module Bigcommerce
   module Lightstep
     extend Configuration
 
+    ##
+    # Start the global tracer and configure LightStep
+    #
+    # @param [String] component_name
+    # @param [::Bigcommerce::Lightstep::TransportFactory] transport_factory
+    #
     def self.start(component_name: nil, transport_factory: nil)
       component_name ||= ::Bigcommerce::Lightstep.component_name
       transport_factory ||= ::Bigcommerce::Lightstep::TransportFactory.new

data/lib/bigcommerce/lightstep/configuration.rb

@@ -98,7 +98,7 @@ module Bigcommerce
           send("#{k}=".to_sym, v)
         end
 
-        default_logger = ::Logger.new(STDOUT)
+        default_logger = ::Logger.new($stdout)
         default_logger.level = ::Logger::INFO
         self.logger = defined?(Rails) ? Rails.logger : default_logger
 

data/lib/bigcommerce/lightstep/interceptors/env.rb

@@ -43,6 +43,7 @@ module Bigcommerce
         # @param [Array<Symbol>] presets Specify presets that automatically setup keys
         #
         def initialize(keys: nil, env: nil, presets: [])
+          super()
           @keys = keys || {}
           @presets = presets || []
           @env = env || ENV

data/lib/bigcommerce/lightstep/middleware/faraday.rb

@@ -30,7 +30,7 @@ module Bigcommerce
         OT_TAG_SAMPLED = 'ot-tracer-sampled'
 
         def initialize(app, service_name = nil)
-          @app = app
+          super(app)
           @service_name = (service_name || 'external').to_s
         end
 

data/lib/bigcommerce/lightstep/transport.rb

@@ -34,6 +34,22 @@ module Bigcommerce
       LIGHTSTEP_PORT = 443
       REPORTS_API_ENDPOINT = '/api/v0/reports'
 
+      DEFAULT_OPEN_TIMEOUT = 20
+      DEFAULT_READ_TIMEOUT = 20
+      DEFAULT_KEEPALIVE_TIMEOUT = 2
+      DEFAULT_SSL_PORT = 443
+
+      ##
+      # Verbosity levels for the transport
+      #
+      class Verbosity
+        FATAL = 0
+        ERROR = 1
+        WARN = 2
+        INFO = 3
+        DEBUG = 4
+      end
+
       # Initialize the transport
       # @param host [String] host of the domain to the endpoind to push data
       # @param port [Numeric] port on which to connect
@@ -44,31 +60,34 @@ module Bigcommerce
       # @return [Transport]
       def initialize(
         access_token:,
-        host: LIGHTSTEP_HOST,
-        port: LIGHTSTEP_PORT,
-        verbose: 0,
-        encryption: ENCRYPTION_TLS,
+        host: nil,
+        port: nil,
+        verbose: nil,
+        encryption: nil,
         ssl_verify_peer: true,
-        open_timeout: 2,
-        read_timeout: 2,
+        ssl_port: nil,
+        open_timeout: nil,
+        read_timeout: nil,
         continue_timeout: nil,
-        keep_alive_timeout: 2,
+        keep_alive_timeout: nil,
         logger: nil
       )
-        @host = host
-        @port = port
-        @verbose = verbose
-        @encryption = encryption
+        @host = host || LIGHTSTEP_HOST
+        @port = port || LIGHTSTEP_PORT
+        @verbose = verbose || Verbosity::FATAL
+        @encryption = encryption || ENCRYPTION_TLS
         @ssl_verify_peer = ssl_verify_peer
-        @open_timeout = open_timeout.to_i
-        @read_timeout = read_timeout.to_i
+        @ssl_port = (ssl_port || DEFAULT_SSL_PORT).to_i
+        @open_timeout = (open_timeout || DEFAULT_OPEN_TIMEOUT).to_i
+        @read_timeout = (read_timeout || DEFAULT_READ_TIMEOUT).to_i
         @continue_timeout = continue_timeout
-        @keep_alive_timeout = keep_alive_timeout.to_i
+        @keep_alive_timeout = (keep_alive_timeout || DEFAULT_KEEPALIVE_TIMEOUT).to_i
         @access_token = access_token.to_s
 
-        default_logger = ::Logger.new(STDOUT)
+        default_logger = ::Logger.new($stdout)
         default_logger.level = ::Logger::INFO
         @logger = logger || default_logger
+        super()
       end
 
       ##
@@ -78,12 +97,12 @@ module Bigcommerce
       # @return [NilClass]
       #
       def report(report)
-        @logger.info report if @verbose >= 3
+        @logger.info report if @verbose >= Verbosity::INFO
 
         req = build_request(report)
         res = connection.request(req)
 
-        @logger.info res.to_s if @verbose >= 3
+        @logger.info res.to_s if @verbose >= Verbosity::INFO
 
         nil
       end
@@ -109,7 +128,7 @@ module Bigcommerce
       def connection
         unless @connection
           @connection = ::Net::HTTP.new(@host, @port)
-          if @port == 443
+          if @port == @ssl_port
             @connection.use_ssl = @encryption == ENCRYPTION_TLS
             @connection.verify_mode = ::OpenSSL::SSL::VERIFY_NONE unless @ssl_verify_peer
           end

data/lib/bigcommerce/lightstep/version.rb

@@ -17,6 +17,6 @@
 #
 module Bigcommerce
   module Lightstep
-    VERSION = '2.1.1'
+    VERSION = '2.2.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bc-lightstep-ruby
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0
 platform: ruby
 authors:
 - Shaun McCormick
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-08 00:00:00.000000000 Z
+date: 2020-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -100,14 +100,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.79'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.79'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement