gruf-lightstep 1.2.0 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 03171299f85c9b42debe5369c906caf4c48fd658c593e5d352cbde7d4b2cacfe
-  data.tar.gz: 4d877406f1f62a6830dd5716beae9b3856f7788771e62036bf86a949f613fb27
+  metadata.gz: 287470dd4cdd7a7190852a7971c019cd9ec7e856914a870ca7501630ea4ff51d
+  data.tar.gz: 8b34d6ab2b31a2a5cf2f2a6b5836a539cc6001825c659951c006cc2fc24c50ec
 SHA512:
-  metadata.gz: f4d188c2a0a91c11f5d9aa6ffd47617116171a78c766d40612a54e515f2182925f9e451ee77eeb99db0ffed12f2d7e35f5f70adf87f3bbb6d894b8c80b55e080
-  data.tar.gz: af7280d6b9c409234d199e450e3cfd656f9ed86b5a3bb02fe39e5dbbe244f2b888026a56635a91fa70f3f16189a4bc8058247a19dd813d512e480991246fef12
+  metadata.gz: 1f6682eb881f2ee3e439cf399c3b2d0f458a4e1e7c60dedcfe29fcdd403588a398557c7ffe2e1f8566d8d062fd0f6e5965ef573069b8818b1652b87309d08ed4
+  data.tar.gz: f5d5ae22dc7210d37e863f6dbe60e9611718990017f2b1c82d0edb9ab826fc95c533ae99f6b3a5e021d0a9ea07c14a8eea1cabafe89741aca37b9134e149becd

data/CHANGELOG.md

@@ -1,56 +1,80 @@
-Changelog for the gruf-zipkin gem.
+Changelog for the gruf-lightstep gem.
 
-h3. Pending Release
+### Pending Release
 
-h3. 1.2.0
+### 1.5.1
+
+- Remove unnecessarily strict dev dependencies
+
+### 1.5.0
+
+- Add official support for Ruby 3 
+
+### 1.4.0
+
+- Ensure `error` span tag is always set when an exception is raised 
+- Move from whitelist -> allowlist in server interceptor
+
+### 1.3.0
+
+- Bump Ruby requirement to 2.6+
+- Bump bc-lightstep-ruby dependency to 2.0+
+- Explicitly declare gruf requirement in gemspec
+- Explicitly declare development deps for gem
+
+###  1.2.1
+
+- Ensure boundary span has `span.kind` set
+
+### 1.2.0
 
 - Add `frozen_string_literal: true` to all files
 - Deprecate ruby 2.2 support
 
-h3. 1.1.3
+### 1.1.3
 
 - Bump bc-lightstep-ruby to 1.2.0
 
-h3. 1.1.2
+### 1.1.2
 
 - First OSS release
 - Explicitly require bc-lightstep-ruby dependency
-- Add option to whitelist request params to lightstep as span tags
+- Add option to allowlist request params to lightstep as span tags
 
-h3. 1.1.1
+### 1.1.1
 
 - Bump bc-lightstep-ruby to 1.1.5
 - Bump gruf minimum version to 2.4
 
-h3. 1.1.0
+### 1.1.0
 
 - Exclude client validation errors from being "errors"
 
-h3. 1.0.3
+### 1.0.3
 
 - Ensure gRPC requests are always the root span
 
-h3. 1.0.2
+### 1.0.2
 
 - Add ignore_methods as an option to server interceptor
 - Standardize span tags according to BC spec
 
-h3. 1.0.0
+### 1.0.0
 
 - Support for gruf 2.0.0
 
-h3. 0.10.3
+### 0.10.3
 
 - Update to 0.10.3, add rubocop+bundler-audit
 
-h3. 0.10.2
+### 0.10.2
 
 - Updated license to MIT
 
-h3. 0.10.1
+### 0.10.1
 
 - Handle case when tracer is not yet initialized before trace executes
 
-h3. 0.10.0
+### 0.10.0
 
 - Initial public release

data/README.md

@@ -1,6 +1,6 @@
 # gruf-lightstep - LightStep tracing for gruf
 
-[![CircleCI](https://circleci.com/gh/bigcommerce/gruf-lightstep/tree/master.svg?style=svg)](https://circleci.com/gh/bigcommerce/gruf-lightstep/tree/master) [![Gem Version](https://badge.fury.io/rb/gruf-lightstep.svg)](https://badge.fury.io/rb/gruf-lightstep) [![Inline docs](http://inch-ci.org/github/bigcommerce/gruf-lightstep.svg?branch=master)](http://inch-ci.org/github/bigcommerce/gruf-lightstep)
+[![CircleCI](https://circleci.com/gh/bigcommerce/gruf-lightstep/tree/main.svg?style=svg)](https://circleci.com/gh/bigcommerce/gruf-lightstep/tree/main) [![Gem Version](https://badge.fury.io/rb/gruf-lightstep.svg)](https://badge.fury.io/rb/gruf-lightstep) [![Inline docs](http://inch-ci.org/github/bigcommerce/gruf-lightstep.svg?branch=main)](http://inch-ci.org/github/bigcommerce/gruf-lightstep)
 
 Adds LightStep tracing support for [gruf](https://github.com/bigcommerce/gruf) 2.0.0+.
 
@@ -14,18 +14,24 @@ Then in an initializer or before use, after loading gruf:
 
 ```ruby
 require 'gruf/lightstep'
-
-Gruf::Lightstep.configure do |c|
-  c.component_name = 'myapp'
-  c.access_token = 'abcdefg'
-  c.host = 'my.lightstep.service.io'
-  c.port = 8080
-  c.verbosity = 1
-end
-Gruf::Lightstep.start
 ```
 
-Then after, in your gruf initializer:
+### Configuration
+
+You can use the following ENV vars to configure lightstep:
+
+| Name | Description |
+| ---- | ---- |
+|LIGHTSTEP_ACCESS_TOKEN|LightStep access token, if used in your satellites|
+|LIGHTSTEP_COMPONENT_NAME|The name of your service| 
+|LIGHTSTEP_HOST|The host of your lightstep satellites|
+|LIGHTSTEP_PORT|The port of your lightstep satellites|
+
+See more configuration here: https://github.com/bigcommerce/bc-lightstep-ruby#environment-config
+
+### Server Interceptors
+
+In your gruf initializer:
 
 ```ruby
 Gruf.configure do |c|
@@ -37,11 +43,11 @@ It comes with a few more options as well:
 
 | Option | Description | Default |
 | ------ | ----------- | ------- |
-| whitelist | An array of parameter key names to log to lightstep. E.g. `[uuid kind]` | `[]` |
+| allowlist | An array of parameter key names to log to lightstep. E.g. `[uuid kind]` | `[]` |
 | ignore_methods | An array of method names to ignore from logging. E.g. `['namespace.health.check']` | `[]` |
 
-It's important to maintain a safe whitelist should you decide to log parameters; gruf does no
-parameter sanitization on its own. We also recommend do not whitelist parameters that may contain
+It's important to maintain a safe allowlist should you decide to log parameters; gruf does no
+parameter sanitization on its own. We also recommend do not allowlist parameters that may contain
 very large values (such as binary or json data).
 
 ### Client Interceptors

data/gruf-lightstep.gemspec

@@ -15,7 +15,7 @@
 # COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
 # OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #
-$:.push File.expand_path("../lib", __FILE__)
+$LOAD_PATH.unshift(File.expand_path('lib', __dir__))
 require 'gruf/lightstep/version'
 
 Gem::Specification.new do |spec|
@@ -24,7 +24,7 @@ Gem::Specification.new do |spec|
   spec.authors       = ['Shaun McCormick']
   spec.email         = ['shaun.mccormick@bigcommerce.com']
 
-  spec.summary       = %q{Plugin for lightstep tracing for gruf}
+  spec.summary       = 'Plugin for lightstep tracing for gruf'
   spec.description   = spec.summary
   spec.homepage      = 'https://github.com/bigcommerce/gruf-lightstep'
   spec.license       = 'MIT'
@@ -32,10 +32,16 @@ Gem::Specification.new do |spec|
   spec.files         = Dir['README.md', 'CHANGELOG.md', 'CODE_OF_CONDUCT.md', 'lib/**/*', 'gruf-lightstep.gemspec']
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'bundler', '~> 1.11'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'rspec', '~> 3.7'
-  spec.add_development_dependency 'pry'
+  spec.required_ruby_version = '>= 2.6'
 
-  spec.add_runtime_dependency 'bc-lightstep-ruby', '~> 1.2'
+  spec.add_development_dependency 'bundler-audit', '>= 0.6'
+  spec.add_development_dependency 'pry', '>= 0.13'
+  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.82'
+  spec.add_development_dependency 'simplecov', '>= 0.15'
+
+  spec.add_runtime_dependency 'bc-lightstep-ruby', '~> 2.2'
+  spec.add_runtime_dependency 'gruf', '>= 2.4', '< 3'
 end

data/lib/gruf/lightstep/client_interceptor.rb

@@ -44,7 +44,7 @@ module Gruf
         return unless tracer
 
         span = tracer.active_span
-        return unless span&.is_a?(::LightStep::Span)
+        return unless span.is_a?(::LightStep::Span)
 
         span
       end

data/lib/gruf/lightstep/method.rb

@@ -35,7 +35,7 @@ module Gruf
       end
 
       ##
-      # @return [Gruf::Zipkin::Headers]
+      # @return [Gruf::Lightstep::Headers]
       #
       def headers
         @headers ||= Gruf::Lightstep::Headers.new(@active_call)

data/lib/gruf/lightstep/server_interceptor.rb

@@ -21,32 +21,32 @@ module Gruf
     # Intercepts inbound calls to provide LightStep tracing
     #
     class ServerInterceptor < Gruf::Interceptors::ServerInterceptor
+      DEFAULT_ERROR_CLASSES = %w[GRPC::Unknown GRPC::Internal GRPC::DataLoss GRPC::FailedPrecondition GRPC::Unavailable GRPC::DeadlineExceeded GRPC::Cancelled].freeze
+
       ##
       # Handle the gruf around hook and trace sampled requests
       #
       def call(&_block)
-        return yield if options.fetch(:ignore_methods, []).include?(request.method_name)
+        return yield if ignore_methods.include?(request.method_name)
 
         result = nil
-
-        whitelist = options.fetch(:whitelist, []).map(&:to_s).map(&:to_sym)
         params = request_message_params
-
         tracer = ::Bigcommerce::Lightstep::Tracer.instance
         tracer.clear_active_span! # because we're always starting from the top on a gRPC boundary
         tracer.start_span(request.method_name, context: request_method.headers.to_h) do |span|
           span.set_tag('grpc.method', request.method_key)
           span.set_tag('grpc.request_class', request.request_class)
           span.set_tag('grpc.service', request.service_key)
+          span.set_tag('span.kind', 'server')
 
-          whitelist.each do |param|
+          allowlist.each do |param|
             span.set_tag(param.to_s, params[param]) if params.key?(param)
           end
 
           begin
             result = yield
           rescue StandardError => e
-            span.set_tag('error', true) if error?(e)
+            span.set_tag('error', error?(e))
             span.set_tag('grpc.error', true)
             span.set_tag('grpc.error_code', code_for(e))
             span.set_tag('grpc.error_class', e.class)
@@ -58,6 +58,19 @@ module Gruf
 
       private
 
+      ##
+      # @return [Array<String>]
+      def ignore_methods
+        @ignore_methods ||= options.fetch(:ignore_methods, nil) || []
+      end
+
+      ##
+      # @return [Array<Symbol>]
+      #
+      def allowlist
+        @allowlist ||= (options.fetch(:allowlist, nil) || []).map(&:to_s).map(&:to_sym)
+      end
+
       ##
       # @param [StandardError]
       # @return [Number] that maps to one of the GRCP::Core::StatusCodes or Gruf::Lightstep.default_error_code
@@ -93,7 +106,7 @@ module Gruf
       # @return [Array]
       #
       def error_classes
-        options.fetch(:error_classes, %w[GRPC::Unknown GRPC::Internal GRPC::DataLoss GRPC::FailedPrecondition GRPC::Unavailable GRPC::DeadlineExceeded GRPC::Cancelled])
+        @error_classes ||= (options.fetch(:error_classes, nil) || DEFAULT_ERROR_CLASSES)
       end
     end
   end

data/lib/gruf/lightstep/version.rb

@@ -17,6 +17,6 @@
 #
 module Gruf
   module Lightstep
-    VERSION = '1.2.0'
+    VERSION = '1.5.1'
   end
 end

metadata

@@ -1,85 +1,147 @@
 --- !ruby/object:Gem::Specification
 name: gruf-lightstep
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.5.1
 platform: ruby
 authors:
 - Shaun McCormick
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-05-01 00:00:00.000000000 Z
+date: 2021-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: bundler-audit
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '0.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.11'
+        version: '0.13'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '3.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.7'
+        version: '3.8'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rspec_junit_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.4'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.82'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.82'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.15'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.15'
 - !ruby/object:Gem::Dependency
   name: bc-lightstep-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '2.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '2.2'
+- !ruby/object:Gem::Dependency
+  name: gruf
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.4'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '2.4'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 description: Plugin for lightstep tracing for gruf
 email:
 - shaun.mccormick@bigcommerce.com
@@ -102,7 +164,7 @@ homepage: https://github.com/bigcommerce/gruf-lightstep
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -110,16 +172,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.7
-signing_key: 
+rubygems_version: 3.2.16
+signing_key:
 specification_version: 4
 summary: Plugin for lightstep tracing for gruf
 test_files: []