rack-redic 0.2.2 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a9190ad054eec78d0314a378fb9617f826c05ab0
-  data.tar.gz: 330593cae1a58e77bbcbfbbb516c984b5e1bb7cf
+  metadata.gz: 060c56a771cd80ea2e61cb4abbf91406e8766daa
+  data.tar.gz: 4f77b38762b713d241ea93a5daf9a70241f4e1f0
 SHA512:
-  metadata.gz: 34b0c681b2aedc62288d23cd78accec23ff69c69c6c30aa8cbe1f5daeda69f294f5eb76d87c3a878b15dae7e011085a5ac56d64403b4b87351236b7c6143b1cd
-  data.tar.gz: 35f31a99b9a287fc994a8953bde3617401a3313730ee52d5e7db5ff57a0642b011f9bdd99e89b0f0def166b20574d70d55df1f48e52e13ab53f7158ba7e207a1
+  metadata.gz: 21d1c9430203b82f90daabe32fcbd30050468b2f59c693ec454cbbcf2ff62398a580c86486cb60b7631467e0e7b74dca44e67bc317e20b3458e51b39b6cfface
+  data.tar.gz: c63d41ef4a89ae67783e12d70a0cd27c82a1a3803af6b3b7127575a398f25a8fe413baac3b365e93c36d64011c12457831f87a53d7df2f18f88e89cc176637b5

data/lib/rack/session/redic.rb

@@ -28,6 +28,8 @@ module Rack
     # Any other options will get passed to Rack::Session::Abstract::Persisted.
     #
     class Redic < Abstract::Persisted
+      REDIS_URL = 'REDIS_URL'.freeze
+
       def initialize(app, options = {})
         super
 
@@ -35,7 +37,7 @@ module Rack
         @storage = Storage.new(
           options[:expire_after],
           options.delete(:marshaller) { Marshal },
-          options.delete(:url) { ENV.fetch('REDIS_URL') }
+          options.delete(:url) { ENV.fetch(REDIS_URL) }
         )
       end
 
@@ -48,7 +50,7 @@ module Rack
       end
 
       # Find the session (or generate a blank one).
-      def find_session(req, sid)
+      def find_session(_req, sid)
         @mutex.synchronize do
           unless sid and session = @storage.get(sid)
             sid, session = generate_sid, {}
@@ -60,7 +62,7 @@ module Rack
       end
 
       # Write the session.
-      def write_session(req, session_id, new_session, options)
+      def write_session(_req, session_id, new_session, _options)
         @mutex.synchronize do
           @storage.set(session_id, new_session)
 
@@ -69,7 +71,7 @@ module Rack
       end
 
       # Kill the session.
-      def delete_session(req, session_id, options)
+      def delete_session(_req, session_id, options)
         @mutex.synchronize do
           @storage.delete(session_id)
           generate_sid unless options[:drop]
@@ -80,10 +82,16 @@ module Rack
 
       # A wrapper around Redic to simplify calls.
       class Storage
-        DELETE = 'DEL'
-        EXISTS = 'EXISTS'
-        GET = 'GET'
-        SET = 'SET'
+        # Redis commands.
+        DELETE = 'DEL'.freeze
+        EX = 'EX'.freeze
+        EXISTS = 'EXISTS'.freeze
+        GET = 'GET'.freeze
+        SET = 'SET'.freeze
+
+        # Assorted.
+        PACK = 'm'.freeze
+        ZERO = 0
 
         def initialize(expires, marshaller, url)
           @expires = expires
@@ -92,7 +100,7 @@ module Rack
         end
 
         def exists?(id)
-          @storage.call(EXISTS, id) != 0
+          @storage.call(EXISTS, id) != ZERO
         end
 
         def get(id)
@@ -101,7 +109,7 @@ module Rack
 
         def set(id, object)
           arguments = [SET, id, serialize(object)]
-          arguments = arguments + ['EX', @expires] if @expires
+          arguments = arguments + [EX, @expires] if @expires
 
           @storage.call(*arguments)
         end
@@ -114,13 +122,12 @@ module Rack
 
         # Should always return a string.
         def serialize(object)
-          [Zlib::Deflate.deflate(@marshaller.dump(object))].pack('m')
+          [Zlib::Deflate.deflate(@marshaller.dump(object))].pack(PACK)
         end
 
         # Should always return the session object.
         def deserialize(string)
-          return unless string
-          @marshaller.load(Zlib::Inflate.inflate(string.unpack('m').first))
+          @marshaller.load(Zlib::Inflate.inflate(string.unpack(PACK).first))
         end
       end
     end

data/rack-redic.gemspec

@@ -2,7 +2,7 @@
 # frozen_string_literal: true
 Gem::Specification.new do |spec|
   spec.name    = 'rack-redic'
-  spec.version = '0.2.2'
+  spec.version = '0.3.0'
   spec.authors = ['Evan Lecklider']
   spec.email   = ['evan@lecklider.com']
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rack-redic
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Evan Lecklider
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-19 00:00:00.000000000 Z
+date: 2016-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack