gitlab-labkit 0.14.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b2b166e9a540a6314a2c5e863e7f109f6c3763f2649aa459dfec41a5018e57f
-  data.tar.gz: 7dc9c77305e855d930420dca4e59cba0e18bd990fae24a7a44e871a7e03e84ed
+  metadata.gz: 235f0b00e0d08d7ee809ae43736e0ce5c900c0c75d402028bfb1247df868f9ac
+  data.tar.gz: 759c4d7bd91c75aeb7bc2b737f55558c6426b32940a399cca982efebb7b1740f
 SHA512:
-  metadata.gz: 81819b7e4fd168c805e286f9315dd5a757423367938619021dcb7e81d5ad41a6f057224bfc046d96a132354688b33b0b4ed2c3e2196dfa09725176bb32b6ca3e
-  data.tar.gz: 35d67adc6f7561609f617df26af9a3942e77cd1b66000523584308bd160c88062dc0fcd1ed6e0d9662f1af6a5c6436c80fe80b53d5b6b4c86ba1e880b24f5a61
+  metadata.gz: 42c83bbcdcb9c588a703400bd5d1830e426680e9d433a223026566d1417a7eb8000dec99530e2e9887935686938e6e2128c8953f44540ead950447c3a3d2b564
+  data.tar.gz: 9cd740e3bd04db453cf812b1efc71a6d5b516c875cfce4cfb0837660390c8457b86679c775f2d23b1dd373b59183255268340d710215d1f1bac5864fdf03c2f6

data/.gitlab-ci.yml

@@ -1,3 +1,6 @@
+default:
+  image: ruby:2.7
+
 workflow:
   rules:
     # For merge requests, create a pipeline.
@@ -22,9 +25,11 @@ test:2.6:
   image: ruby:2.6
   <<: *test_definition
 
-test:2.5:
-  image: ruby:2.5
-  <<: *test_definition
+static-analysis:
+  before_script:
+    - bundle install
+  script:
+    - rake verify
 
 deploy:
   stage: deploy

data/.rubocop.yml

@@ -1,8 +1,11 @@
-AllCops:
-  TargetRubyVersion: 2.4
+inherit_from: .rubocop_todo.yml
+
+inherit_gem:
+  gitlab-styles:
+    - rubocop-default.yml
 
-require:
-  - rubocop-rspec
+AllCops:
+  TargetRubyVersion: 2.7
 
 Style/HashSyntax:
   EnforcedStyle: no_mixed_keys
@@ -28,11 +31,11 @@ Style/StringLiterals:
 Style/StringLiteralsInInterpolation:
   EnforcedStyle: double_quotes
 
-Style/UseLambda:
+Style/Lambda:
   Enabled: false
 
-Layout/MultilineMethodCallIndentation:
-  Enabled: No
+Layout/LineLength:
+  Enabled: false
 
 Layout/SpaceInLambdaLiteral:
   Enabled: No
@@ -43,69 +46,6 @@ Layout/SpaceInsideBlockBraces:
 Layout/FirstParameterIndentation:
   Enabled: No
 
-Metrics/AbcSize:
-  Enabled: true
-  Max: 54.28
-
-Metrics/BlockLength:
-  Enabled: false
-
-Metrics/BlockNesting:
-  Enabled: true
-  Max: 4
-
-Metrics/ClassLength:
-  Enabled: false
-
-Metrics/CyclomaticComplexity:
-  Enabled: true
-  Max: 13
-
-Metrics/LineLength:
-  Enabled: false
-
-Metrics/MethodLength:
-  Enabled: false
-
-Metrics/ModuleLength:
-  Enabled: false
-
-Metrics/ParameterLists:
-  Enabled: true
-  Max: 8
-
-Metrics/PerceivedComplexity:
-  Enabled: true
-  Max: 14
-
-RSpec/AnyInstance:
-  Enabled: false
-
-RSpec/BeEql:
-  Enabled: true
-
-RSpec/BeforeAfterAll:
-  Enabled: false
-
-RSpec/DescribeClass:
-  Enabled: false
-
-RSpec/DescribeMethod:
-  Enabled: false
-
-RSpec/DescribeSymbol:
-  Enabled: true
-
-RSpec/DescribedClass:
-  Enabled: true
-
-RSpec/EmptyExampleGroup:
-  Enabled: true
-
-RSpec/ExampleLength:
-  Enabled: false
-  Max: 5
-
 RSpec/ExampleWording:
   Enabled: false
   CustomTransform:
@@ -114,63 +54,9 @@ RSpec/ExampleWording:
     not: does not
   IgnoredWords: []
 
-RSpec/ExpectActual:
-  Enabled: true
-
-RSpec/ExpectOutput:
+Lint/HashCompareByIdentity: # (new in 0.93)
   Enabled: true
-
-RSpec/FilePath:
+Lint/RedundantSafeNavigation: # (new in 0.93)
   Enabled: true
-  IgnoreMethods: true
-
-RSpec/Focus:
-  Enabled: true
-
-RSpec/HookArgument:
-  Enabled: true
-  EnforcedStyle: implicit
-
-RSpec/ImplicitExpect:
-  Enabled: true
-  EnforcedStyle: is_expected
-
-RSpec/InstanceVariable:
-  Enabled: false
-
-RSpec/LeadingSubject:
-  Enabled: false
-
-RSpec/LetSetup:
-  Enabled: false
-
-RSpec/MessageChain:
-  Enabled: false
-
-RSpec/MessageSpies:
-  Enabled: false
-
-RSpec/MultipleDescribes:
-  Enabled: false
-
-RSpec/MultipleExpectations:
-  Enabled: false
-
-RSpec/NamedSubject:
-  Enabled: false
-
-RSpec/NestedGroups:
-  Enabled: false
-
-RSpec/NotToNot:
-  EnforcedStyle: not_to
+Style/ClassEqualityComparison: # (new in 0.93)
   Enabled: true
-
-RSpec/RepeatedDescription:
-  Enabled: false
-
-RSpec/SubjectStub:
-  Enabled: false
-
-RSpec/VerifiedDoubles:
-  Enabled: false

data/.rubocop_todo.yml

@@ -0,0 +1,238 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2021-04-18 00:00:00 UTC using RuboCop version 0.93.1.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 51
+# Cop supports --auto-correct.
+CodeReuse/ActiveRecord:
+  Exclude:
+    - 'spec/labkit/correlation/grpc/client_interceptor_spec.rb'
+    - 'spec/labkit/correlation/grpc/server_interceptor_spec.rb'
+    - 'spec/labkit/middleware/rack_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/context/server_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/server_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/tracing/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/tracing/server_spec.rb'
+    - 'spec/labkit/tracing/factory_spec.rb'
+    - 'spec/labkit/tracing/grpc/client_interceptor_spec.rb'
+    - 'spec/labkit/tracing/grpc/server_interceptor_spec.rb'
+    - 'spec/labkit/tracing_spec.rb'
+    - 'spec/support/tracing/shared_examples.rb'
+
+# Offense count: 1
+# Configuration parameters: Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'gitlab-labkit.gemspec'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle.
+# SupportedHashRocketStyles: key, separator, table
+# SupportedColonStyles: key, separator, table
+# SupportedLastArgumentHashStyles: always_inspect, always_ignore, ignore_implicit, ignore_explicit
+Layout/HashAlignment:
+  Exclude:
+    - 'spec/labkit/context_spec.rb'
+
+# Offense count: 1
+Lint/BinaryOperatorWithIdenticalOperands:
+  Exclude:
+    - 'spec/labkit/tracing/redis/redis_interceptor_helper_spec.rb'
+
+# Offense count: 4
+Lint/ConstantDefinitionInBlock:
+  Exclude:
+    - 'spec/labkit/middleware/sidekiq/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/context/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/context/server_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/server_spec.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: AllowedMethods.
+# AllowedMethods: instance_of?, kind_of?, is_a?, eql?, respond_to?, equal?
+Lint/RedundantSafeNavigation:
+  Exclude:
+    - 'lib/labkit/tracing.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredName.
+Naming/RescuedExceptionsVariableName:
+  Exclude:
+    - 'lib/labkit/logging/grpc/server_interceptor.rb'
+
+# Offense count: 2
+Performance/MethodObjectAsBlock:
+  Exclude:
+    - 'lib/labkit/context.rb'
+
+# Offense count: 1
+# Configuration parameters: Prefixes.
+# Prefixes: when, with, without
+RSpec/ContextWording:
+  Exclude:
+    - 'spec/labkit/middleware/sidekiq/server_spec.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: SkipBlocks, EnforcedStyle.
+# SupportedStyles: described_class, explicit
+RSpec/DescribedClass:
+  Exclude:
+    - 'spec/labkit/tracing_spec.rb'
+
+# Offense count: 2
+# Configuration parameters: CustomTransform, IgnoreMethods, SpecSuffixOnly.
+RSpec/FilePath:
+  Exclude:
+    - 'spec/labkit/httpclient_publisher_spec.rb'
+    - 'spec/logging/sanitizer_spec.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: single_line_only, single_statement_only, disallow
+RSpec/ImplicitSubject:
+  Exclude:
+    - 'spec/labkit/correlation/correlation_id_spec.rb'
+
+# Offense count: 4
+RSpec/LeakyConstantDeclaration:
+  Exclude:
+    - 'spec/labkit/middleware/sidekiq/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/context/client_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/context/server_spec.rb'
+    - 'spec/labkit/middleware/sidekiq/server_spec.rb'
+
+# Offense count: 2
+# Cop supports --auto-correct.
+RSpec/LetBeforeExamples:
+  Exclude:
+    - 'spec/labkit/tracing_spec.rb'
+
+# Offense count: 18
+# Configuration parameters: AllowSubject.
+RSpec/MultipleMemoizedHelpers:
+  Max: 7
+
+# Offense count: 15
+# Cop supports --auto-correct.
+RSpec/ScatteredLet:
+  Exclude:
+    - 'spec/labkit/excon_publisher_spec.rb'
+    - 'spec/labkit/httpclient_publisher_spec.rb'
+    - 'spec/labkit/net_http_publisher_spec.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Rails/NegateInclude:
+  Exclude:
+    - 'spec/labkit/correlation/grpc/client_interceptor_spec.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: Include.
+# Include: **/Rakefile, **/*.rake
+Rails/RakeEnvironment:
+  Exclude:
+    - 'lib/capistrano/tasks/**/*.rake'
+    - 'Rakefile'
+
+# Offense count: 1
+# Configuration parameters: AllowedChars.
+Style/AsciiComments:
+  Exclude:
+    - 'spec/support/grpc_service/test_service_impl.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/CaseLikeIf:
+  Exclude:
+    - 'lib/labkit/excon_publisher.rb'
+
+# Offense count: 23
+# Cop supports --auto-correct.
+Style/ExplicitBlockArgument:
+  Exclude:
+    - 'lib/labkit/correlation/grpc/server_interceptor.rb'
+    - 'lib/labkit/logging/grpc/server_interceptor.rb'
+    - 'lib/labkit/middleware/sidekiq/client.rb'
+    - 'lib/labkit/middleware/sidekiq/context/server.rb'
+    - 'lib/labkit/middleware/sidekiq/server.rb'
+    - 'lib/labkit/middleware/sidekiq/tracing/server.rb'
+    - 'lib/labkit/tracing/grpc/client_interceptor.rb'
+    - 'lib/labkit/tracing/grpc/server_interceptor.rb'
+    - 'lib/labkit/tracing/redis/redis_interceptor_helper.rb'
+
+# Offense count: 8
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: always, always_true, never
+Style/FrozenStringLiteralComment:
+  Exclude:
+    - 'spec/labkit/tracing/rails/active_record/sql_instrumenter_spec.rb'
+    - 'spec/labkit/tracing/rails/active_support/cache_delete_instrumenter_spec.rb'
+    - 'spec/labkit/tracing/rails/active_support/cache_fetch_hit_instrumenter_spec.rb'
+    - 'spec/labkit/tracing/rails/active_support/cache_generate_instrumenter_spec.rb'
+    - 'spec/labkit/tracing/rails/active_support/cache_read_instrumenter_spec.rb'
+    - 'spec/labkit/tracing/rails/active_support/cache_write_instrumenter_spec.rb'
+    - 'spec/support/grpc_service/test_pb.rb'
+    - 'tools/update-changelog.rb'
+
+# Offense count: 2
+# Cop supports --auto-correct.
+Style/GlobalStdStream:
+  Exclude:
+    - 'lib/labkit/tracing/jaeger_factory.rb'
+
+# Offense count: 3
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, UseHashRocketsWithSymbolValues, PreferHashRocketsForNonAlnumEndingSymbols.
+# SupportedStyles: ruby19, hash_rockets, no_mixed_keys, ruby19_no_mixed_keys
+Style/HashSyntax:
+  Exclude:
+    - 'spec/labkit/excon_publisher_spec.rb'
+
+# Offense count: 16
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredDelimiters.
+Style/PercentLiteralDelimiters:
+  Exclude:
+    - 'spec/labkit/tracing/redis/redis_interceptor_helper_spec.rb'
+
+# Offense count: 2
+# Cop supports --auto-correct.
+Style/RedundantBegin:
+  Exclude:
+    - 'Rakefile'
+    - 'spec/labkit/net_http_publisher_spec.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/RedundantRegexpEscape:
+  Exclude:
+    - 'lib/labkit/logging/sanitizer.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+Style/StringConcatenation:
+  Exclude:
+    - 'lib/labkit/tracing/redis/redis_interceptor_helper.rb'
+
+# Offense count: 28
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyleForMultiline.
+# SupportedStylesForMultiline: comma, consistent_comma, no_comma
+Style/TrailingCommaInArrayLiteral:
+  Exclude:
+    - 'spec/labkit/excon_publisher_spec.rb'
+    - 'spec/labkit/httpclient_publisher_spec.rb'
+    - 'spec/labkit/net_http_publisher_spec.rb'

data/Rakefile

@@ -9,7 +9,9 @@ begin
 end
 
 require "rubocop/rake_task"
-RuboCop::RakeTask.new
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.options = %w[--parallel]
+end
 
 desc "Alias for `rake rufo:run`"
 task :format => ["rufo:run"]

data/gitlab-labkit.gemspec

@@ -16,20 +16,21 @@ Gem::Specification.new do |spec|
 
   spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(spec|tools)/}) }
   spec.require_paths = ["lib"]
-  spec.required_ruby_version = ">= 2.4.0"
+  spec.required_ruby_version = ">= 2.6.0"
 
   # Please maintain alphabetical order for dependencies
   spec.add_runtime_dependency "actionpack", ">= 5.0.0", "< 7.0.0"
   spec.add_runtime_dependency "activesupport", ">= 5.0.0", "< 7.0.0"
-  spec.add_runtime_dependency "gitlab-pg_query", "~> 1.3"
   spec.add_runtime_dependency "grpc", "~> 1.19" # Be sure to update the "grpc-tools" dev_depenency too
   spec.add_runtime_dependency "jaeger-client", "~> 1.1"
   spec.add_runtime_dependency "opentracing", "~> 0.4"
+  spec.add_runtime_dependency "pg_query", "~> 2.0"
   spec.add_runtime_dependency "redis", ">3.0.0", "<5.0.0"
 
   # Please maintain alphabetical order for dev dependencies
   spec.add_development_dependency "excon", "~> 0.78.1"
   spec.add_development_dependency "faraday", "~> 1.2.0"
+  spec.add_development_dependency "gitlab-styles", "~> 6.2.0"
   spec.add_development_dependency "grpc-tools", "~> 1.19"
   spec.add_development_dependency "httparty", "~> 0.17.3"
   spec.add_development_dependency "httpclient", "~> 2.8.3"
@@ -37,10 +38,8 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rack", "~> 2.0"
   spec.add_development_dependency "rake", "~> 12.3"
   spec.add_development_dependency "rest-client", "~> 2.1.0"
-  spec.add_development_dependency "rspec", "~> 3.8.0"
+  spec.add_development_dependency "rspec", "~> 3.10.0"
   spec.add_development_dependency "rspec-parameterized", "~> 0.4"
-  spec.add_development_dependency "rubocop", "~> 0.65.0"
-  spec.add_development_dependency "rubocop-rspec", "~> 1.22.1"
   spec.add_development_dependency "rufo", "0.9.0"
   spec.add_development_dependency "sidekiq", "~> 5.2.7"
 end

data/lib/gitlab-labkit.rb

@@ -1,8 +1,6 @@
 # rubocop:disable Naming/FileName
 # frozen_string_literal: true
 
-require "active_support/all"
-
 # LabKit is a module for handling cross-project
 # infrastructural concerns, partcularly related to
 # observability.

data/lib/labkit/context.rb

@@ -1,5 +1,8 @@
 # frozen_string_literal: true
 
+require "active_support/core_ext/module/delegation"
+require "active_support/core_ext/string/starts_ends_with"
+
 module Labkit
   # A context can be used to provide structured information on what resources
   # GitLab is working on within a service.
@@ -23,7 +26,7 @@ module Labkit
     RAW_KEYS = [CORRELATION_ID_KEY].freeze
     HEADER_PREFIX = "X-Gitlab-"
     KNOWN_KEYS = %w[user project root_namespace subscription_plan caller_id
-                    remote_ip related_class feature_category].freeze
+                    remote_ip related_class feature_category client_id].freeze
 
     class << self
       def with_context(attributes = {})
@@ -106,6 +109,12 @@ module Labkit
       end
     end
 
+    def get_attribute(attribute)
+      raw = call_or_value(data[log_key(attribute)])
+
+      call_or_value(raw)
+    end
+
     protected
 
     def assign_attributes(attributes)
@@ -130,9 +139,13 @@ module Labkit
 
     attr_reader :data
 
+    def call_or_value(value)
+      value.respond_to?(:call) ? value.call : value
+    end
+
     def expand_data
       data.transform_values do |value|
-        value = value.respond_to?(:call) ? value.call : value
+        value = call_or_value(value)
 
         value.presence
       end.compact

data/lib/labkit/correlation/grpc/client_interceptor.rb

@@ -2,9 +2,8 @@
 
 # Disable the UnusedMethodArgument linter, since we need to declare the kwargs
 # in the methods, but we don't actually use them.
-# rubocop:disable Lint/UnusedMethodArgument
-
 require "grpc"
+require "singleton"
 
 module Labkit
   module Correlation
@@ -48,5 +47,3 @@ module Labkit
     end
   end
 end
-
-# rubocop:enable Lint/UnusedMethodArgument

data/lib/labkit/correlation/grpc/server_interceptor.rb

@@ -2,7 +2,6 @@
 
 # Disable the UnusedMethodArgument linter, since we need to declare the kwargs
 # in the methods, but we don't actually use them.
-# rubocop:disable Lint/UnusedMethodArgument
 
 require "grpc"
 
@@ -51,5 +50,3 @@ module Labkit
     end
   end
 end
-
-# rubocop:enable Lint/UnusedMethodArgument

data/lib/labkit/excon_publisher.rb

@@ -92,17 +92,19 @@ module Labkit
     def start_payload(datum)
       payload = {
         method: datum[:method].to_s.upcase,
-        host: datum[:host],
-        path: datum[:path],
-        port: datum[:port],
-        scheme: datum[:scheme],
-        query: datum[:query],
+        host: nil_or_string(datum[:host]),
+        path: nil_or_string(datum[:path]),
+        port: nil_or_int(datum[:port]),
+        scheme: nil_or_string(datum[:scheme]),
+        query: generate_query_string(datum[:query]),
         start_time: ::Labkit::System.monotonic_time,
       }
+
       unless datum[:proxy].nil?
         payload[:proxy_host] = datum[:proxy][:host]
         payload[:proxy_port] = datum[:proxy][:port]
       end
+
       payload
     end
 
@@ -126,5 +128,23 @@ module Labkit
     def remove_connection_payload(datum)
       connection_payload.delete(datum[:connection])
     end
+
+    def nil_or_string(str)
+      str&.to_s
+    end
+
+    def nil_or_int(int)
+      int&.to_i
+    rescue StandardError
+      nil
+    end
+
+    def generate_query_string(query)
+      if query.is_a?(Hash)
+        query.to_query
+      else
+        nil_or_string(query)
+      end
+    end
   end
 end

data/lib/labkit/logging/grpc/server_interceptor.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "active_support/time_with_zone"
+require "active_support/values/time_zone"
 require "grpc"
 require "json"
 
@@ -77,6 +79,11 @@ module Labkit
             time: Time.now.utc.strftime("%Y-%m-%dT%H:%M:%S.%LZ"),
           )
 
+          if ex
+            message["exception"] = ex.message
+            message["exception_backtrace"] = ex.backtrace[0..5] if ex.backtrace
+          end
+
           @log_file.puts(JSON.dump(message))
         end
       end

data/lib/labkit/logging/sanitizer.rb

@@ -19,9 +19,7 @@ module Labkit
 
       def self.sanitize_field(content)
         content = content.gsub(URL_REGEXP) { |url| mask_url(url) }
-        content = content.gsub(SCP_URL_REGEXP) { |scp_url| mask_scp_url(scp_url) }
-
-        content
+        content.gsub(SCP_URL_REGEXP) { |scp_url| mask_scp_url(scp_url) }
       end
 
       def self.sanitize_sql(sql)

data/lib/labkit/middleware/sidekiq/context/server.rb

@@ -8,7 +8,8 @@ module Labkit
         # reinstantiate a context in which the job will run.
         class Server
           def call(_worker_class, job, _queue)
-            data = job.merge(Labkit::Context.log_key(:caller_id) => job["class"])
+            worker_name = (job["wrapped"].presence || job["class"]).to_s
+            data = job.merge(Labkit::Context.log_key(:caller_id) => worker_name)
 
             Labkit::Context.with_context(data) do |_context|
               yield

data/lib/labkit/tracing.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "active_support/all"
-
 module Labkit
   # Tracing provides distributed tracing functionality
   module Tracing

data/lib/labkit/tracing/abstract_instrumenter.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "opentracing"
-require "active_support/all"
 
 module Labkit
   module Tracing

data/lib/labkit/tracing/external_http/request_instrumenter.rb

@@ -21,10 +21,12 @@ module Labkit
             "path" => payload[:path],
             "scheme" => payload[:scheme],
           }
+
           unless payload[:proxy_host].nil?
             tags["proxy_host"] = payload[:proxy_host]
             tags["proxy_port"] = payload[:proxy_port]
           end
+
           tags
         end
       end

data/lib/labkit/tracing/grpc/client_interceptor.rb

@@ -2,9 +2,9 @@
 
 # Disable the UnusedMethodArgument linter, since we need to declare the kwargs
 # in the methods, but we don't actually use them.
-# rubocop:disable Lint/UnusedMethodArgument
 require "opentracing"
 require "grpc"
+require "singleton"
 
 module Labkit
   module Tracing
@@ -45,5 +45,3 @@ module Labkit
     end
   end
 end
-
-# rubocop:enable Lint/UnusedMethodArgument

data/lib/labkit/tracing/grpc/server_interceptor.rb

@@ -2,7 +2,6 @@
 
 # Disable the UnusedMethodArgument linter, since we need to declare the kwargs
 # in the methods, but we don't actually use them.
-# rubocop:disable Lint/UnusedMethodArgument
 
 require "opentracing"
 require "grpc"
@@ -60,5 +59,3 @@ module Labkit
     end
   end
 end
-
-# rubocop:enable Lint/UnusedMethodArgument

data/lib/labkit/tracing/rack_middleware.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require "opentracing"
-require "active_support/all"
 require "action_dispatch"
 
 module Labkit

data/lib/labkit/tracing/tracing_common.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-require "active_support/all"
+require "active_support/concern"
+require "active_support/notifications"
 
 module Labkit
   module Tracing

data/lib/labkit/tracing/tracing_utils.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "active_support/core_ext/string/starts_ends_with"
 require "opentracing"
 
 module Labkit

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-labkit
 version: !ruby/object:Gem::Version
-  version: 0.14.0
+  version: 0.17.1
 platform: ruby
 authors:
 - Andrew Newdigate
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-13 00:00:00.000000000 Z
+date: 2021-04-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -51,61 +51,61 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 7.0.0
 - !ruby/object:Gem::Dependency
-  name: gitlab-pg_query
+  name: grpc
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.19'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.19'
 - !ruby/object:Gem::Dependency
-  name: grpc
+  name: jaeger-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.19'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.19'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
-  name: jaeger-client
+  name: opentracing
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '0.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '0.4'
 - !ruby/object:Gem::Dependency
-  name: opentracing
+  name: pg_query
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -154,6 +154,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.2.0
+- !ruby/object:Gem::Dependency
+  name: gitlab-styles
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.2.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.2.0
 - !ruby/object:Gem::Dependency
   name: grpc-tools
   requirement: !ruby/object:Gem::Requirement
@@ -258,14 +272,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.8.0
+        version: 3.10.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.8.0
+        version: 3.10.0
 - !ruby/object:Gem::Dependency
   name: rspec-parameterized
   requirement: !ruby/object:Gem::Requirement
@@ -280,34 +294,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.4'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.65.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.65.0
-- !ruby/object:Gem::Dependency
-  name: rubocop-rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.22.1
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 1.22.1
 - !ruby/object:Gem::Dependency
   name: rufo
   requirement: !ruby/object:Gem::Requirement
@@ -348,6 +334,7 @@ files:
 - ".gitlab/CODEOWNERS"
 - ".rspec"
 - ".rubocop.yml"
+- ".rubocop_todo.yml"
 - ".ruby-version"
 - ".rufo"
 - Gemfile
@@ -428,14 +415,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.4.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Instrumentation for GitLab