sso 0.1.0.beta1 → 0.1.0.beta2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 68585960729029037de661b49771b7766ae1c2c5
-  data.tar.gz: 74381326cc5605fdd6b8b3a1d044bf65fc4915e0
+  metadata.gz: 4c53a1e9c7573ea3d6e97727aa00e435c485b3d7
+  data.tar.gz: ff596d704a4e53c462d45251d64108cb85918d0f
 SHA512:
-  metadata.gz: c8ae9e8fa88060efe6c39e36f5bce7750d739c382e9137c9ba409dfe3587674774a16bd491369fb7fd6e05ccf00e5366f7eed4a06435807b7ff00be6c9f6a473
-  data.tar.gz: be505aafa56fe22050682280c7c09a53ceddb25044a01ee4ee7479eb77e992cb349151adc4f7becec0519c43b595b63872f86df31b3cd93e1c24e80e821ff834
+  metadata.gz: 4399983b1f108c80f30c497f5db66d406c95168271fcaa5c559e6b8b22f4ae99772b721db7b4e325694b896b29ef16e48a8cbc6a17e7df06d00df62ad5c87393
+  data.tar.gz: 2895cdf553890ae507ca160e5f40afc6b5c098ac461a1cf458bd1e7d73befcd4232c476fa5570e2e9e369fdadcbc2fd8737a9cb64eb66f9af995e9a2727a8191

data/lib/sso/client/README.md

@@ -43,6 +43,7 @@ If you store the entire object, that would be a major security risk and allow fo
 class Warden::SessionSerializer
   def serialize(passport)
     Redis.set passport.id, passport.to_json
+    passport.id
   end
 
   def deserialize(passport_id)

data/lib/sso/client/authentications/passport.rb

@@ -41,6 +41,7 @@ module SSO
           debug { 'Retrieving Passport from server...' }
           if verification.success? && verification.code == :passport_valid_and_modified
             passport = verification.object
+            passport.modified!
 
             debug { "Successfully retrieved Passport with ID #{passport_id} from server." }
             return passport

data/lib/sso/client/warden/hooks/after_fetch.rb

@@ -41,7 +41,6 @@ module SSO
 
           private
 
-
           def verifier
             ::SSO::Client::PassportVerifier.new passport_id: passport.id, passport_state: passport.state, passport_secret: passport.secret, user_ip: ip, user_agent: agent, device_id: device_id
           end

data/spec/lib/sso/client/authentications/passport_spec.rb

@@ -32,7 +32,7 @@ RSpec.describe SSO::Client::Authentications::Passport, type: :request, db: true
 
   # Server
   let(:insider)          { false }
-  let!(:server_user)     { create :user, name: 'Emily', tags: %i(cool nice) }
+  let(:server_user)      { create :user, name: 'Emily', tags: %i(cool nice) }
   let!(:server_passport) { create :passport, user: server_user, owner_id: server_user.id, ip: ip, agent: agent, insider: insider }
 
   before do
@@ -53,6 +53,10 @@ RSpec.describe SSO::Client::Authentications::Passport, type: :request, db: true
         expect(passport).to be_verified
       end
 
+      it 'modifies the passport' do
+        expect(passport).to be_modified
+      end
+
       it 'tracks the immediate request IP' do
         expect(server_passport.reload.ip).to eq '127.0.0.1'
       end
@@ -76,6 +80,10 @@ RSpec.describe SSO::Client::Authentications::Passport, type: :request, db: true
         expect(passport).to be_verified
       end
 
+      it 'modifies the passport' do
+        expect(passport).to be_modified
+      end
+
       it 'tracks the untrusted client IP' do
         expect(server_passport.reload.ip).to eq ip
       end

data/spec/lib/sso/server/middleware/passport_destruction_spec.rb

@@ -20,14 +20,6 @@ RSpec.describe SSO::Server::Middleware::PassportDestruction, type: :request, db:
       delete "/oauth/sso/v1/passports/#{passport.id}"
       expect(updated_passport.revoked_at.to_i).to eq Time.now.to_i
     end
-
-    it 'logs out from warden' do
-      Warden.on_next_request do |proxy|
-        expect(proxy).to receive(:logout)
-      end
-
-      delete "/oauth/sso/v1/passports/#{passport.id}"
-    end
   end
 
 end

data/spec/spec_helper.rb

@@ -23,7 +23,6 @@ RSpec.configure do |config|
 
   config.include FactoryGirl::Syntax::Methods
   config.include SSO::Test::Helpers
-  config.include Warden::Test::Helpers
 
   config.color = true
   config.disable_monkey_patching!
@@ -46,7 +45,6 @@ RSpec.configure do |config|
 
   config.after :each do
     Timecop.return
-    Warden.test_reset!
   end
 
   config.after :each, db: true do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sso
 version: !ruby/object:Gem::Version
-  version: 0.1.0.beta1
+  version: 0.1.0.beta2
 platform: ruby
 authors:
 - halo