session_manager 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 99b4d5e7ccb73991486dc249c4bceb19e81eb216
-  data.tar.gz: 3c25975edf9d5991f3a7b868b7a4c0cedfd94ed5
+  metadata.gz: ea516539cd796d14c8b1a566d132fd0d1f1c8ecb
+  data.tar.gz: eb3afabe532a8c26dc1a3c037b3623bb63d20f19
 SHA512:
-  metadata.gz: 33adb3d3f40d04ee051a4b76ab424f48279538746d6c98d8419c05d3645e36a4bb6cd5aa7c77768fa3a27ee11615624b249e91ef66ecee35a259fed540e3787e
-  data.tar.gz: 386b9348f90d8c3ff634caebb22cabfcaf4a608b677cfda952217667193f3fa2597f02ef70db93eef4f975e47bab58ada13430dffce5bed0df3d881cb3bb5cc0
+  metadata.gz: 87310f3999d002fa50e3fb1f5ddbe63132d411149e910f9c2ddeeee2faa19601d40f9a5bdfc18730fc0f17e9527064f2d86ea4c6916b430cedee0d3672d3c50f
+  data.tar.gz: 50672d02f01dbd25ec73c86f3ca9aa666e79c38f3eb24143429ab58b5a5080c696a8af83ad4374c097153c014ee77f44632fd36af715951bbe14d785b307f5b1

data/lib/session_manager/session_handler.rb

@@ -6,24 +6,37 @@ require_relative 'base'
 class Session_Handler < SessionManager::Base
 
   def create_session(attribute,option)
-    param = parse_json(attribute)
-    collection = collection(option)
-    token = generate_token
-    session = session_generator(option,param,token)
-    @redis.sadd(collection,session.to_json)
-
+    if validate_json(attribute) 
+      param = parse_json(attribute)
+      collection = collection(option)
+      token = generate_token
+      session = session_generator(option,param,token)
+      @redis.sadd(collection,session.to_json)
+    else
+      false
+    end
   end
  
-  def verify_session(object,option) 
+  def verify_session(object,option)
+   if validate_json(object)   
     collection = collection(option)
     @redis.sismember(collection,object)
+   else
+       false
+   end
  end
 
  def delete_session(object,option)
+    if validate_json(object)
      collection = collection(option)
      @redis.srem(collection,object)
+    else
+        false
+    end
  end
 
+ private #####################################################################################################
+
  def collection(option)
      user = option.fetch(:user) || false
      app = option.fetch(:app)  || false
@@ -39,7 +52,22 @@ class Session_Handler < SessionManager::Base
    option[:user] ? { login: param['login'],token: token} : { api_key: param['api_key'],token: token} 
  end
 
+ def validate_json(param)
+    obj = parse_json(param)
+    hash = obj.to_hash
+    validate_argument(hash)
+ end
 
+ def validate_argument(param)
+   param.values.each do |v|
+    if v.nil? || v.empty?
+      return false
+      break
+    else
+      return true
+    end
+   end
+ end
 
 
 end

data/lib/session_manager/version.rb

@@ -1,3 +1,3 @@
 module SessionManager
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/session_handler_spec.rb

@@ -8,9 +8,9 @@ describe Session_Handler do
   let (:user_option) { {user: true, app: false}}
   let (:app_option) { {user: false, app: true}}
   let (:user_session) {{login: 'login',token: 'jdijenfdeifiirifffjr'}.to_json}
-  let (:not_user_session) {{login: 'login',token: 'jdijenfd'}.to_json}
+  #let (:not_user_session) {{login: 'login',token: 'jdijenfd'}.to_json}
   let (:app_session) {{api_key: '01',token: 'jdijenfdeifiirifffjr'}.to_json}
-  let (:not_app_session) {{api_key: '02',token: 'jdijenfdeifiirr'}.to_json}
+  #let (:not_app_session) {{api_key: '02',token: 'jdijenfdeifiirr'}.to_json}
 
   before :each do
     redis=Redis.current
@@ -22,11 +22,11 @@ describe Session_Handler do
   end
   describe "#create_session" do
     context "when a user's session is created" do
-      it { expect(session_handler.create_session(login,user_option ) ).to be_true}       
+      it { expect(session_handler.create_session(login,user_option)).to be_true}       
     end 
 
     context "When an app's session is created" do
-        it { expect(session_handler.create_session(api_key,app_option ) ).to be_true }
+        it { expect(session_handler.create_session(api_key,app_option)).to be_true }
     end
   end
 
@@ -37,6 +37,7 @@ describe Session_Handler do
       
 
       context "when a user's session is not found" do
+         not_user_session = {login: 'login',token: 'jdijenfd'}.to_json 
         it { expect(session_handler.verify_session(not_user_session,user_option) ).to be_false}
       end
       
@@ -46,7 +47,8 @@ describe Session_Handler do
       end
 
       context "when app's session is not found" do
-          it { expect(session_handler.verify_session(not_app_session,app_option) ).to be_false}
+        not_app_session = {api_key: '02',token: 'jdijenfdeifiirr'}.to_json
+        it { expect(session_handler.verify_session(not_app_session,app_option) ).to be_false}
       end
   end
 
@@ -56,6 +58,7 @@ describe Session_Handler do
       end
 
       context "when a user's session is not deleted" do
+         not_user_session = {login: 'login',token: 'jdijenfd'}.to_json 
         it { expect(session_handler.delete_session(not_user_session,user_option)).to be_false}
       end
 
@@ -64,6 +67,7 @@ describe Session_Handler do
       end
 
       context "when a app's session is not found" do
+        not_app_session = {api_key: '02',token: 'jdijenfdeifiirr'}.to_json
           it { expect(session_handler.delete_session(not_app_session,app_option)).to be_false}
       end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: session_manager
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - fabira && atakem
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-07 00:00:00.000000000 Z
+date: 2014-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler