rack-redic 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 060c56a771cd80ea2e61cb4abbf91406e8766daa
-  data.tar.gz: 4f77b38762b713d241ea93a5daf9a70241f4e1f0
+  metadata.gz: 3d4afdb628407b7b0dac6cf05ac9ea5d33497bae
+  data.tar.gz: 747344ab157680920425f77776de3b7150d2cd9c
 SHA512:
-  metadata.gz: 21d1c9430203b82f90daabe32fcbd30050468b2f59c693ec454cbbcf2ff62398a580c86486cb60b7631467e0e7b74dca44e67bc317e20b3458e51b39b6cfface
-  data.tar.gz: c63d41ef4a89ae67783e12d70a0cd27c82a1a3803af6b3b7127575a398f25a8fe413baac3b365e93c36d64011c12457831f87a53d7df2f18f88e89cc176637b5
+  metadata.gz: 7435b1f651cc9952aff451613fc32c6698ef856e3e579b6035d1133d538ba749cbd9ca37fe8cc70f99f68420a1d71b9b9e5d0e16fd6b6d056c3620d8ce331e1b
+  data.tar.gz: aae16f6bce3a267b657b4551eb6e3d2b895635319dc9e3b6c21f58f82a778c64b197bd86dd1b51cb5fa29d4e2e5c9a46e560676018f08ac12009a7d072cc0f59

data/Rakefile

@@ -1,3 +1,3 @@
 require 'bundler/gem_tasks'
 
-task :default => :spec
+task default: :spec

data/lib/rack/session/redic.rb

@@ -18,8 +18,8 @@ module Rack
     #   required is that this class respond to the  `load` and `dump` methods,
     #   returning the session hash and a string respectively.
     # - :url - Addtionally, you may pass in the URL for your Redis server. The
-    #   default URL is fetched from the ENV as 'REDIS_URL' in keeping with Heroku
-    #   and others' practices.
+    #   default URL is fetched from the ENV as 'REDIS_URL' in keeping with
+    #   Heroku and others' practices.
     # - :expire_after - Finally, expiration will be passed to the Redis server
     #   via the 'EX' option on 'SET'. Expiration should be in seconds, just like
     #   Rack's default handling of the :expire_after option. This option will
@@ -52,10 +52,8 @@ module Rack
       # Find the session (or generate a blank one).
       def find_session(_req, sid)
         @mutex.synchronize do
-          unless sid and session = @storage.get(sid)
-            sid, session = generate_sid, {}
-            @storage.set(sid, session)
-          end
+          sid ||= generate_sid
+          session = @storage.get(sid) || @storage.init(sid, {})
 
           [sid, session]
         end
@@ -78,8 +76,7 @@ module Rack
         end
       end
 
-      private
-
+      ##
       # A wrapper around Redic to simplify calls.
       class Storage
         # Redis commands.
@@ -103,13 +100,18 @@ module Rack
           @storage.call(EXISTS, id) != ZERO
         end
 
+        def init(sid, initial)
+          set(sid, initial)
+          initial
+        end
+
         def get(id)
           deserialize(@storage.call(GET, id))
         end
 
         def set(id, object)
           arguments = [SET, id, serialize(object)]
-          arguments = arguments + [EX, @expires] if @expires
+          arguments += [EX, @expires] if @expires
 
           @storage.call(*arguments)
         end
@@ -127,6 +129,7 @@ module Rack
 
         # Should always return the session object.
         def deserialize(string)
+          return unless string
           @marshaller.load(Zlib::Inflate.inflate(string.unpack(PACK).first))
         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.3.0'
+  spec.version = '0.4.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.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Evan Lecklider
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-03 00:00:00.000000000 Z
+date: 2016-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -73,7 +73,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.4
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Rack::Session in Redis via Redic