jwt_keeper 6.0.0 → 6.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 135ae5f5451cdcd997de5a5208642833ef5d043038b020fc96b67415c243b8b7
-  data.tar.gz: f940045097cb8cbd2d281bd1b887598f7c2f0f7bf25e598b834978e61a9337f8
+  metadata.gz: d6ad212dc01831b9bae31adab98ba2edbe1fd803923e32eb9817e2b96515c706
+  data.tar.gz: c6ef8deb150ffeed569d4da108ac745a907c8bfc065c5564cd49b6aff978f00e
 SHA512:
-  metadata.gz: dfa8fe29790a114e9f180822f32015fb5881cc268b249b757bec44df0ed419b203e33c27f439ed019cbdeecdbc6aa34512930c4210b5c000a77adb5da6dee07c
-  data.tar.gz: 881df471eaa098d00614578f551d2e67414cdee8260c45e7ad90a2cdca71fd2af52b3c88ebd8d9fa61bedaf9863dceaee7281a6f5f92f56b238111bf4b382527
+  metadata.gz: 87f73d56ceb956a78337d4df685ac8fb86b6eda03046ba137f92f689eb10b5485103f9ea003ccdb7643149b4e74722d674571ec698e8eed40eb0a2d3b1b2feb2
+  data.tar.gz: 7695c3ce514f9e150d492766b90e7b76e04cb5c496e422aa265d6f255a311e72d986319f169fc0bb5919a42fbe4cb364da700bfeb42fb947bfdb9a6841c5c280

data/jwt_keeper.gemspec

@@ -28,8 +28,9 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rspec', '~> 3.8'
   spec.add_development_dependency 'fuubar'
   spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'redis-client'
+  spec.add_development_dependency 'redis'
 
-  spec.add_dependency 'redis'
   spec.add_dependency 'rails'
   spec.add_dependency 'activesupport'
   spec.add_dependency 'jwt', '>= 1.5'

data/lib/generators/templates/jwt_keeper.rb

@@ -27,10 +27,8 @@ JWTKeeper.configure do |config|
   # config.audience         = 'example.com'
 
   # the location of redis config file
-  # config.redis_connection = Redis.new(connection_options)
-  # config.redis_connection = ConnectionPool.new(size: ENV.fetch('RAILS_MAX_THREADS', 5)) do
-  #   Redis.new(url: ENV['REDISCLOUD_URL'] || 'redis://localhost:6379/')
-  # end
+  # config.redis_connection = RedisClient.new(connection_options)
+  # config.redis_connection = RedisClient.config(connection_options).new_pool(size: ENV.fetch('RAILS_MAX_THREADS', 5))
 
   # A unique idenfitier for the token version.
   # config.version          = 1

data/lib/jwt_keeper/datastore.rb

@@ -27,27 +27,40 @@ module JWTKeeper
 
       # @!visibility private
       def set_with_expiry(jti, seconds, type)
-        redis = JWTKeeper.configuration.redis_connection
-
-        if redis.is_a?(Redis)
-          redis.setex(jti, seconds, type)
-        elsif defined?(ConnectionPool) && redis.is_a?(ConnectionPool)
-          redis.with { |conn| conn.setex(jti, seconds, type) }
-        else
-          throw 'Bad Redis Connection'
+        with_redis do |redis|
+          if redis.respond_to?(:call) # For RedisClient
+            redis.call('SETEX', jti, seconds, type)
+          elsif redis.respond_to?(:setex) # For Redis
+            redis.setex(jti, seconds, type)
+          else
+            throw 'Bad Redis Connection'
+          end
         end
       end
 
       # @!visibility private
       def get(jti)
+        with_redis do |redis|
+          if redis.respond_to?(:call) # For RedisClient
+            redis.call('GET', jti)
+          elsif redis.respond_to?(:get) # For Redis
+            redis.get(jti)
+          else
+            throw 'Bad Redis Connection'
+          end
+        end
+      end
+
+      # @!visibility private
+      def with_redis
         redis = JWTKeeper.configuration.redis_connection
 
-        if redis.is_a?(Redis)
-          redis.get(jti)
-        elsif defined?(ConnectionPool) && redis.is_a?(ConnectionPool)
-          redis.with { |conn| conn.get(jti) }
+        if redis.respond_to?(:with)
+          redis.with do |conn|
+            yield conn
+          end
         else
-          throw 'Bad Redis Connection'
+          yield(redis)
         end
       end
     end

data/lib/jwt_keeper/version.rb

@@ -1,4 +1,4 @@
 # Gem Version
 module JWTKeeper
-  VERSION = '6.0.0'.freeze
+  VERSION = '6.1.0'.freeze
 end

data/spec/lib/jwt_keeper/datastore_spec.rb

@@ -1,70 +1,107 @@
 RSpec.describe JWTKeeper::Datastore do
-  include_context 'initialize config'
   let(:jti) { SecureRandom.uuid }
 
-  describe '.rotate' do
-    before { described_class.rotate(jti, 30) }
+  shared_examples 'Datastore Specs' do
+    describe '.rotate' do
+      before { described_class.rotate(jti, 30) }
 
-    it 'stores a token_id with a soft expiry' do
-      expect(described_class.send(:get, jti)).to eq 'soft'
+      it 'stores a token_id with a soft expiry' do
+        expect(described_class.send(:get, jti)).to eq 'soft'
+      end
     end
-  end
 
-  describe '.pending?' do
-    context 'with a missing token' do
-      it 'returns false' do
-        expect(described_class.pending?(jti)).to be false
+    describe '.pending?' do
+      context 'with a missing token' do
+        it 'returns false' do
+          expect(described_class.pending?(jti)).to be false
+        end
       end
-    end
 
-    context 'with a revoked token' do
-      before { described_class.revoke(jti, 30) }
+      context 'with a revoked token' do
+        before { described_class.revoke(jti, 30) }
 
-      it 'returns false' do
-        expect(described_class.pending?(jti)).to be false
+        it 'returns false' do
+          expect(described_class.pending?(jti)).to be false
+        end
       end
-    end
 
-    context 'with a pending token' do
-      before { described_class.rotate(jti, 30) }
+      context 'with a pending token' do
+        before { described_class.rotate(jti, 30) }
 
-      it 'returns true' do
-        expect(described_class.pending?(jti)).to be true
+        it 'returns true' do
+          expect(described_class.pending?(jti)).to be true
+        end
       end
     end
-  end
 
-  describe '.revoke' do
-    before do
-      described_class.revoke(jti, 30)
-    end
+    describe '.revoke' do
+      before do
+        described_class.revoke(jti, 30)
+      end
 
-    it 'stores a token_id with a hard expiry' do
-      expect(described_class.send(:get, jti)).to eq 'hard'
+      it 'stores a token_id with a hard expiry' do
+        expect(described_class.send(:get, jti)).to eq 'hard'
+      end
     end
-  end
 
-  describe '.revoked?' do
-    context 'with a missing token' do
-      it 'returns false' do
-        expect(described_class.revoked?(jti)).to be false
+    describe '.revoked?' do
+      context 'with a missing token' do
+        it 'returns false' do
+          expect(described_class.revoked?(jti)).to be false
+        end
+      end
+
+      context 'with a revoked token' do
+        before { described_class.revoke(jti, 30) }
+
+        it 'returns true' do
+          expect(described_class.revoked?(jti)).to be true
+        end
       end
-    end
 
-    context 'with a revoked token' do
-      before { described_class.revoke(jti, 30) }
+      context 'with a pending token' do
+        before { described_class.rotate(jti, 30) }
 
-      it 'returns true' do
-        expect(described_class.revoked?(jti)).to be true
+        it 'returns false' do
+          expect(described_class.revoked?(jti)).to be false
+        end
       end
     end
+  end
 
-    context 'with a pending token' do
-      before { described_class.rotate(jti, 30) }
+  context 'with Redis' do
+    include_context 'initialize config'
+
+    let(:redis_connection) { Redis.new(url: ENV['REDIS_URL']) }
+
+    include_examples 'Datastore Specs'
+  end
+
+  context 'with Redis Connection Pool' do
+    include_context 'initialize config'
 
-      it 'returns false' do
-        expect(described_class.revoked?(jti)).to be false
+    let(:redis_connection) do
+      ConnectionPool.new(size: ENV.fetch('RAILS_MAX_THREADS', 5)) do
+        Redis.new(url: ENV['REDISCLOUD_URL'] || 'redis://localhost:6379/')
       end
     end
+
+    include_examples 'Datastore Specs'
+  end
+
+  context 'with RedisClient' do
+    include_context 'initialize config'
+
+    let(:redis_connection) { RedisClient.new(url: ENV['REDIS_URL']) }
+
+    include_examples 'Datastore Specs'
+  end
+
+  context 'with RedisClient Pool' do
+    include_context 'initialize config'
+
+    let(:redis_connection) { RedisClient.config(url: ENV['REDIS_URL']).new_pool(size: ENV.fetch('RAILS_MAX_THREADS', 5)) }
+
+    include_examples 'Datastore Specs'
   end
 end

data/spec/spec_helper.rb

@@ -39,6 +39,7 @@ RSpec.configure do |config|
 end
 
 RSpec.shared_context 'initialize config' do
+  let(:redis_connection) { RedisClient.new(url: ENV['REDIS_URL']) }
   let(:config) do
     {
       algorithm:        'HS256',
@@ -46,12 +47,13 @@ RSpec.shared_context 'initialize config' do
       expiry:           24.hours,
       issuer:           'api.example.com',
       audience:         'example.com',
-      redis_connection: Redis.new(url: ENV['REDIS_URL']),
+      redis_connection: redis_connection,
       version:          nil,
       cookie_lock:      false
     }
   end
 
+
   before(:each) do
     JWTKeeper.configure(JWTKeeper::Configuration.new(config))
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jwt_keeper
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 6.1.0
 platform: ruby
 authors:
 - David Rivera
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-05 00:00:00.000000000 Z
+date: 2023-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -137,6 +137,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: redis-client
+  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'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -144,7 +158,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-  type: :runtime
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
@@ -249,7 +263,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.4.1
 signing_key:
 specification_version: 4
 summary: JWT for Rails made easy