sidekiq-logstash 1.2.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a99b4b463ed0d2801d2a32447a29767572a979675d6fca72781fbf0c1971fd9
-  data.tar.gz: dfbfffd67d75c8e0aff617ff8da1a219b264e72e84431632131ec346d4d0dcbb
+  metadata.gz: f1986944d83eef688f7d35ab06cf9dfdb613c51a41cb8f33143ef85fc972be25
+  data.tar.gz: d6480ae4134cb68185ac9c6334c36e6ed9d92e3c0684b760a7064be6f6cdf909
 SHA512:
-  metadata.gz: 38a2a810c8af2eb4afda4f2a340ee39680c4c768d6a9714d897950d23630c189063bf84001f60c97a57ca94c5f0ab03a2464fd9ea8d50803321ac3cfdae86d8d
-  data.tar.gz: 7c4338444e79978f20322d8491e8f4b1b04ff95c643f89fd8d51b92b8379e5b39265190f0b0b0c201ccc6081e89317eec8cedbc9f9fdbe72f505a23e8d705433
+  metadata.gz: b9f8ade62f6abdb3fda0e2b33d617545b127f8b3f7ffa24501669ba76d05c8c9b3ccf461fd33360896421b9f2becebc914b82bc4681f9382ec2a52b373533844
+  data.tar.gz: 9ecfa6afc056cb0b0281eb971bd3e74ceefc71efc9d493b91fa0f901b1f5d2490429d47155301377e1b0d2fece87765821949f27b617f28805383f1b1111aab8

data/.github/workflows/{ci.yaml → ci.yml}

@@ -16,9 +16,9 @@ jobs:
 
     steps:
       - uses: actions/checkout@v2
-      - uses: ruby/setup-ruby@v1
+      - uses: actions/setup-ruby@v1
         with:
-          ruby-version: 2.7
+          ruby-version: 2.7.x
       - name: Install dependencies
         run: bundle install
       - name: Run rubocop
@@ -28,14 +28,13 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby: [2.5, 2.6, 2.7]
+        ruby: ['2.5.x', '2.6.x', '2.7.x']
 
     steps:
       - uses: actions/checkout@v2
-      - uses: ruby/setup-ruby@v1
+      - uses: actions/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby }}
-          bundler-cache: true
       - name: Install dependencies
         run: bundle install
       - name: Run tests

data/.github/workflows/publish.yml

@@ -10,12 +10,12 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@master
 
       - name: Set up Ruby 2.7
-        uses: ruby/setup-ruby@v1
+        uses: actions/setup-ruby@v1
         with:
-          ruby-version: 2.7
+          ruby-version: 2.7.x
 
       - name: Publish to RubyGems
         run: |
@@ -26,4 +26,4 @@ jobs:
           gem build sidekiq-logstash.gemspec
           gem push sidekiq-logstash-*.gem
         env:
-          GEM_HOST_API_KEY: ${{ secrets.RUBYGEMS_AUTH_TOKEN }}
+          GEM_HOST_API_KEY: ${{ secrets.RUBYGEMS_AUTH_TOKEN }}

data/.rubocop.yml

@@ -6,17 +6,18 @@ require:
 AllCops:
   DisplayCopNames: true
   DisplayStyleGuide: true
-  TargetRubyVersion: 2.3
+  NewCops: enable
+  TargetRubyVersion: 2.5
 
 Metrics/BlockLength:
   Exclude:
     - spec/**/*.rb
 
-Metrics/LineLength:
+Layout/LineLength:
   Max: 120
   Exclude:
     - spec/**/*.rb
 
 Style/Documentation:
   Exclude:
-    - 'spec/**/*'
+    - 'spec/**/*'

data/.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2019-04-20 13:23:53 +0200 using RuboCop version 0.67.2.
+# on 2020-08-16 16:36:05 UTC using RuboCop version 0.89.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
@@ -11,31 +11,27 @@ Lint/DuplicateMethods:
   Exclude:
     - 'lib/sidekiq/logstash/configuration.rb'
 
-# Offense count: 4
+# Offense count: 3
+# Configuration parameters: IgnoredMethods.
 Metrics/AbcSize:
-  Max: 31
+  Max: 36
 
-# Offense count: 1
+# Offense count: 2
+# Configuration parameters: IgnoredMethods.
 Metrics/CyclomaticComplexity:
-  Max: 13
+  Max: 18
 
 # Offense count: 6
-# Configuration parameters: CountComments, ExcludedMethods.
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods.
 Metrics/MethodLength:
   Max: 28
 
 # Offense count: 1
+# Configuration parameters: IgnoredMethods.
 Metrics/PerceivedComplexity:
-  Max: 14
+  Max: 19
 
-# Offense count: 7
-Style/Documentation:
+# Offense count: 1
+Style/OptionalBooleanParameter:
   Exclude:
-    - 'spec/**/*'
-    - 'lib/sidekiq/logging/argument_filter.rb'
-    - 'lib/sidekiq/logging/logstash_formatter.rb'
-    - 'lib/sidekiq/logging/shared.rb'
-    - 'lib/sidekiq/logstash.rb'
-    - 'lib/sidekiq/logstash/configuration.rb'
-    - 'lib/sidekiq/logstash_job_logger.rb'
-    - 'lib/sidekiq/middleware/server/logstash_logging.rb'
+    - 'spec/workers/spec_worker.rb'

data/.ruby-version

@@ -1 +1 @@
-2.6.6
+2.7.1

data/README.md

@@ -1,7 +1,7 @@
 # Sidekiq::Logstash
 
-[![Gem Version](https://badge.fury.io/rb/sidekiq-logstash.svg)](https://badge.fury.io/rb/sidekiq-logstash) [<img src="https://travis-ci.org/iMacTia/sidekiq-logstash.svg?branch=master" alt="version" />](https://travis-ci.org/iMacTia/sidekiq-logstash)
-
+[![Gem Version](https://badge.fury.io/rb/sidekiq-logstash.svg)](https://badge.fury.io/rb/sidekiq-logstash)
+[![GitHub Actions CI](https://github.com/iMacTia/sidekiq-logstash/workflows/CI/badge.svg)](https://github.com/iMacTia/sidekiq-logstash/actions?query=workflow%3ACI)
 
 Sidekiq::Logstash turns your [Sidekiq](https://github.com/mperham/sidekiq) log into an organised, aggregated, JSON-syntax log ready to be sent to a logstash server.
 
@@ -30,10 +30,11 @@ Sidekiq::Logstash turns your [Sidekiq](https://github.com/mperham/sidekiq) log i
 
 ## Installation
 
-Add this line to your application's Gemfile:
+Add one of the following lines to your application's Gemfile:
 
 ```ruby
-gem 'sidekiq-logstash'
+gem 'sidekiq-logstash', '~> 2.0' # Sidekiq 6
+gem 'sidekiq-logstash', '< 2' # Sidekiq 5 or older
 ```
 
 And then execute:

data/bin/test_console

@@ -4,7 +4,7 @@
 require 'bundler/setup'
 require 'sidekiq/logstash'
 require_relative '../spec/spec_helper'
-FactoryGirl.find_definitions
+FactoryBot.find_definitions
 
 require 'irb'
 IRB.start

data/lib/sidekiq/logging/argument_filter.rb

@@ -14,7 +14,7 @@ module Sidekiq
       end
 
       # Filters argument by using the filters provided upon initialization.
-      # @param args [Array] the list of arguments.
+      # @param args [Hash] the list of arguments in a hash with key :args.
       def filter(args)
         compiled_filter.call(args)
       end

data/lib/sidekiq/logging/shared.rb

@@ -15,8 +15,8 @@ module Sidekiq
       rescue StandardError => e
         begin
           Sidekiq.logger.warn log_job_exception(job, started_at, e)
-        rescue StandardError => ex
-          log_standard_error(job, e, ex)
+        rescue StandardError => e
+          log_standard_error(job, e, e)
         end
         raise
       end

data/lib/sidekiq/logstash.rb

@@ -2,7 +2,6 @@
 
 require 'sidekiq/logstash/configuration'
 require 'sidekiq/logstash/version'
-require 'sidekiq/middleware/server/logstash_logging'
 require 'sidekiq/logging/logstash_formatter'
 require 'sidekiq/logging/argument_filter'
 require 'sidekiq/logstash_job_logger'
@@ -27,15 +26,7 @@ module Sidekiq
         config.error_handlers.delete_if { |h| h.is_a?(Sidekiq::ExceptionHandler::Logger) }
 
         # Add logstash support
-        # The logging server middleware was removed in Sidekiq 5.0.0, see: https://github.com/mperham/sidekiq/blob/master/Changes.md
-        if Sidekiq::Middleware::Server.const_defined?(:Logging)
-          config.server_middleware do |chain|
-            chain.add Sidekiq::Middleware::Server::LogstashLogging
-            chain.remove Sidekiq::Middleware::Server::Logging
-          end
-        else
-          Sidekiq.options[:job_logger] = Sidekiq::LogstashJobLogger
-        end
+        config.options[:job_logger] = Sidekiq::LogstashJobLogger
 
         # Set custom formatter for Sidekiq logger
         config.logger.formatter = Sidekiq::Logging::LogstashFormatter.new

data/lib/sidekiq/logstash/version.rb

@@ -2,6 +2,6 @@
 
 module Sidekiq
   module Logstash
-    VERSION = '1.2.1'
+    VERSION = '2.0.0'
   end
 end

data/lib/sidekiq/logstash_job_logger.rb

@@ -1,10 +1,11 @@
 # frozen_string_literal: true
 
+require 'sidekiq/job_logger'
 require 'sidekiq/logging/shared'
 
 module Sidekiq
-  # Class used to replace Sidekiq 5 job logger.
-  class LogstashJobLogger
+  # Class used to replace Sidekiq default job logger.
+  class LogstashJobLogger < ::Sidekiq::JobLogger
     include Sidekiq::Logging::Shared
 
     def call(job, _queue, &block)

data/sidekiq-logstash.gemspec

@@ -21,15 +21,14 @@ Gem::Specification.new do |spec|
   spec.bindir = 'exe'
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
+  spec.required_ruby_version = '>= 2.5.0'
 
   spec.add_dependency 'logstash-event', '~> 1.2'
-  spec.add_runtime_dependency 'sidekiq', '>= 3.0', '<6'
-  spec.required_ruby_version = '>= 2.3.0'
+  spec.add_runtime_dependency 'sidekiq', '>= 6.0', '<7'
 
-  spec.add_development_dependency 'factory_girl', '~> 4.0'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'factory_bot', '~> 6.1'
+  spec.add_development_dependency 'rake', '>= 10'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rspec-json_expectations', '~> 2.1.0'
-  spec.add_development_dependency 'rubocop', '~> 0.67.2'
-  spec.add_development_dependency 'rubocop-performance'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.5'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq-logstash
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 2.0.0
 platform: ruby
 authors:
 - Mattia Giuffrida
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-06-21 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logstash-event
@@ -30,48 +30,48 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6'
+        version: '7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6'
+        version: '7'
 - !ruby/object:Gem::Dependency
-  name: factory_girl
+  name: factory_bot
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '6.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '6.1'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -101,44 +101,30 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 2.1.0
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.67.2
+        version: '1.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.67.2
-- !ruby/object:Gem::Dependency
-  name: rubocop-performance
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.5'
 description: 'Sidekiq::Logstash turns your Sidekiq log into an organised, aggregated,
   JSON-syntax log ready to be sent to a logstash server.
 
-  '
+'
 email:
 - giuffrida.mattia@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/workflows/ci.yaml"
+- ".github/workflows/ci.yml"
 - ".github/workflows/publish.yml"
 - ".gitignore"
 - ".rspec"
@@ -152,9 +138,6 @@ files:
 - bin/console
 - bin/setup
 - bin/test_console
-- gemfiles/sidekiq3.gemfile
-- gemfiles/sidekiq4.gemfile
-- gemfiles/sidekiq5.gemfile
 - lib/sidekiq/logging/argument_filter.rb
 - lib/sidekiq/logging/logstash_formatter.rb
 - lib/sidekiq/logging/shared.rb
@@ -162,7 +145,6 @@ files:
 - lib/sidekiq/logstash/configuration.rb
 - lib/sidekiq/logstash/version.rb
 - lib/sidekiq/logstash_job_logger.rb
-- lib/sidekiq/middleware/server/logstash_logging.rb
 - sidekiq-logstash.gemspec
 homepage: https://github.com/iMacTia/sidekiq-logstash
 licenses:
@@ -176,14 +158,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Logstash plugin for Sidekiq

data/gemfiles/sidekiq3.gemfile

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-source 'https://rubygems.org'
-
-gem 'sidekiq', '~> 3.0'
-
-gemspec path: '../'

data/gemfiles/sidekiq4.gemfile

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-source 'https://rubygems.org'
-
-gem 'sidekiq', '~> 4.0'
-
-gemspec path: '../'

data/gemfiles/sidekiq5.gemfile

@@ -1,7 +0,0 @@
-# frozen_string_literal: true
-
-source 'https://rubygems.org'
-
-gem 'sidekiq', '~> 5.0'
-
-gemspec path: '../'

data/lib/sidekiq/middleware/server/logstash_logging.rb

@@ -1,18 +0,0 @@
-# frozen_string_literal: true
-
-require 'sidekiq/logging/shared'
-
-module Sidekiq
-  module Middleware
-    module Server
-      # Class used to replace Sidekiq 4 job logger.
-      class LogstashLogging
-        include Sidekiq::Logging::Shared
-
-        def call(_, job, _, &block)
-          log_job(job, &block)
-        end
-      end
-    end
-  end
-end