🧪 RuboCop-LTS for Ruby 2.2+

Author: pboling

.github/dependabot.yml

@@ -0,0 +1,2 @@
+ignore:
+  - dependency-name: "rubocop-lts"

.rubocop.yml

@@ -0,0 +1,2 @@
+inherit_gem:
+  rubocop-lts: config/rubygem_rspec.yml

.rubocop_gradual.lock

@@ -0,0 +1,39 @@
+{
+  "lib/omniauth/strategies/jwt.rb:543932255": [
+    [60, 9, 76, "Lint/RescueException: Avoid rescuing the `Exception` class. Perhaps you meant to rescue `StandardError`?", 967033479]
+  ],
+  "omniauth-jwt2.gemspec:998952283": [
+    [18, 16, 16, "Packaging/GemspecGit: Avoid using git to produce lists of files. Downstreams often need to build your package in an environment that does not have git (on purpose). Use some pure Ruby alternative, like `Dir` or `Dir.glob`.", 1973161220]
+  ],
+  "spec/lib/omniauth/strategies/jwt_spec.rb:2698313308": [
+    [3, 1, 34, "RSpec/FilePath: Spec path should end with `omni_auth/strategies/jwt*_spec.rb`.", 1935033905],
+    [3, 1, 34, "RSpec/SpecFilePathFormat: Spec path should end with `omni_auth/strategies/jwt*_spec.rb`.", 1935033905],
+    [12, 13, 25, "RSpec/DescribedClass: Use `described_class` instead of `OmniAuth::Strategies::JWT`.", 2234488924],
+    [19, 11, 15, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 391893083],
+    [20, 5, 42, "RSpec/MultipleExpectations: Example has too many expectations [2/1].", 4106660663],
+    [29, 11, 16, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 1409468707],
+    [51, 7, 51, "RSpec/MultipleExpectations: Example has too many expectations [2/1].", 4149552871],
+    [51, 7, 531, "RSpec/ExampleLength: Example has too many lines. [6/5]", 2143440997],
+    [91, 3, 3512, "RSpec/MultipleMemoizedHelpers: Example group has too many memoized helpers [10/5]", 2363831099],
+    [110, 7, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [111, 7, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [116, 9, 6, "RSpec/ExpectInHook: Do not use `expect` in `before` hook", 1179768986],
+    [116, 9, 20, "RSpec/StubbedMock: Prefer `allow` over `expect` when configuring a response.", 2951559342],
+    [116, 33, 7, "RSpec/MessageSpies: Prefer `have_received` for setting message expectations. Setup `rack_request` as a spy using `allow` or `instance_spy`.", 1384559950],
+    [130, 5, 56, "Performance/RedundantMerge: Use `algos[OpenSSL::PKey::EC] = %w[ES256 ES384 ES512]` instead of `algos.merge!(OpenSSL::PKey::EC => %w[ES256 ES384 ES512])`.", 2983772293],
+    [133, 9, 846, "RSpec/MultipleMemoizedHelpers: Example group has too many memoized helpers [10/5]", 3520352246],
+    [152, 22, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [160, 5, 310, "RSpec/MultipleMemoizedHelpers: Example group has too many memoized helpers [10/5]", 3501674141],
+    [160, 13, 28, "RSpec/ContextWording: Context description should match /^when\\b/, /^with\\b/, or /^without\\b/.", 1877551307],
+    [170, 18, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [174, 5, 515, "RSpec/MultipleMemoizedHelpers: Example group has too many memoized helpers [10/5]", 1246671601],
+    [185, 9, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [189, 18, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [193, 5, 537, "RSpec/MultipleMemoizedHelpers: Example group has too many memoized helpers [10/5]", 3770030886],
+    [205, 9, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441],
+    [209, 18, 7, "RSpec/NamedSubject: Name your test subject if you need to reference it explicitly.", 1892732441]
+  ],
+  "spec/support/hash.rb:812296649": [
+    [2, 3, 110, "Style/ClassMethodsDefinitions: Use `class << self` to define a class method.", 3570181400]
+  ]
+}

Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in omniauth-jwt.gemspec
 gemspec

Guardfile

@@ -3,6 +3,6 @@
 
 guard :rspec do
   watch(%r{^spec/.+_spec\.rb$})
-  watch(%r{^lib/(.+)\.rb$})     { |m| "spec/lib/#{m[1]}_spec.rb" }
-  watch('spec/spec_helper.rb')  { "spec" }
-end
+  watch(%r{^lib/(.+)\.rb$}) { |m| "spec/lib/#{m[1]}_spec.rb" }
+  watch("spec/spec_helper.rb") { "spec" }
+end

README.md

@@ -30,7 +30,7 @@ Or install it yourself as:
 You use OmniAuth::JWT just like you do any other OmniAuth strategy:
 
 ```ruby
-use OmniAuth::JWT, 'SHAREDSECRET', auth_url: 'http://example.com/login'
+use OmniAuth::JWT, "SHAREDSECRET", auth_url: "http://example.com/login"
 ```
 
 The first parameter is the shared secret that will be used by the external authenticator to verify
@@ -48,30 +48,30 @@ in. Other available options are:
 * **valid_within:** integer of how many seconds of time skew you will allow. Defaults to `nil`. If this
   is set, the `iat` claim becomes required and must be within the specified number of seconds of the
   current time. This helps to prevent replay attacks.
-  
+
 ### Authentication Process
 
 When you authenticate through `omniauth-jwt` you can send users to `/auth/jwt` and it will redirect
 them to the URL specified in the `auth_url` option. From there, the provider must generate a JWT
 and send it to the `/auth/jwt/callback` URL as a "jwt" parameter:
 
     /auth/jwt/callback?jwt=ENCODEDJWTGOESHERE
-    
+
 An example of how to do that in Sinatra:
 
 ```ruby
-require 'jwt'
+require "jwt"
 
-get '/login/sso/other-app' do
+get "/login/sso/other-app" do
   # assuming the user is already logged in and this is available as current_user
   claims = {
     id: current_user.id,
     name: current_user.name,
     email: current_user.email,
-    iat: Time.now.to_i
+    iat: Time.now.to_i,
   }
-  
-  payload = JWT.encode(claims, ENV['SSO_SECRET'])
+
+  payload = JWT.encode(claims, ENV["SSO_SECRET"])
   redirect "http://other-app.com/auth/jwt/callback?jwt=#{payload}"
 end
 ```

Rakefile

@@ -13,4 +13,11 @@ rescue LoadError
   # NOOP
 end
 
+begin
+  require "rubocop/lts"
+  Rubocop::Lts.install_tasks
+rescue LoadError
+  # NOOP
+end
+
 task default: :spec

gemfiles/contexts/debug.gemfile

@@ -1,5 +1,5 @@
 # Ancient rubies do not have String#casecmp?
-debugging = ENV['CI'].nil? && ENV.fetch('DEBUG', 'false')
+debugging = ENV["CI"].nil? && ENV.fetch("DEBUG", "false")
 
 if debugging && debugging[/true/i]
   gem "byebug"

gemfiles/contexts/style.gemfile

@@ -1,4 +1,5 @@
 # Style
-gem "rubocop-lts", "~> 12.1", ">= 12.1.1"     # ruby 2.7 - Lint Support for Ruby 2.4+
+gem "rubocop-lts", "~> 8.1", ">= 8.1.1"       # ruby 2.7 - Lint Support for Ruby 2.2+
 gem "rubocop-packaging", "~> 0.5", ">= 0.5.2" # ruby 2.6
 gem "rubocop-rspec", "~> 2.25"                # ruby 2.7
+gem "rspec-block_is_expected", "~> 1.0", ">= 1.0.5" # ruby 1.8.7

gemfiles/contexts/testing.gemfile

@@ -1,7 +1,8 @@
 # Testing
-gem "rack", "~> 3.0", ">= 3.0.8"              # ruby 2.4
-gem "rack-session", "~> 2.0"                  # ruby 2.4
+gem "ed25519", "~> 1.3"                       # ruby 2.4
 gem "json", "~> 2.6", ">= 2.6.3"              # ruby 2.3
 gem "openssl", ">= 2.0"                       # ruby 2.3, v3.0 is >= 2.6, v3.2 is >= 2.7
 gem "openssl-signature_algorithm", "~> 1.3"   # ruby 2.4
-gem "ed25519", "~> 1.3"                       # ruby 2.4
+gem "rack", "~> 3.0", ">= 3.0.8"              # ruby 2.4
+gem "rack-session", "~> 2.0"                  # ruby 2.4
+gem "rspec-block_is_expected", "~> 1.0", ">= 1.0.5" # ruby 1.8.7