omniauth-wsfed 0.3.1.pre.beta → 0.3.2.pre.beta

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6c0ad3503577aad70d428772c14a135661181aad
-  data.tar.gz: 71abed720db88b079785cda06a4afdd344f5764c
+  metadata.gz: 41837183b3cca16a41fb354df3df39a013c4c39c
+  data.tar.gz: f87ec79b820194dc55d78fc0d81ab2fa1d98fdb1
 SHA512:
-  metadata.gz: cd213f0cb47858742d2994ed1d5dc01cea7a3a21d199d735eae50202ad7b611c321ef5fbe51b969b3928fc3cf56bfc174d063c1f13b23ce9014b846a23ad4ce9
-  data.tar.gz: b43aba00a37ec14879c4b8014c1ffe13fe68a3a29e31c7f54e0fe1d28971ccdfd1685aa7f7565770416668f6bb1ee7f6ce98f65ee47fc3b369ba5ed1f0ab1f5b
+  metadata.gz: 682244ae78b8f3cb4c5af6a3b1e1999ff09a723bb337d314ea21d3710a7e70f39055205a292857f7e3fefaa5ec7d904ad3dcc9b2c7681022391ae41df921ca93
+  data.tar.gz: ce50e1823d95a415b848b268f7572d47b144275beae3d1e2b49fa3909e6f7bac5fdf5c12c5af745b20ff6c06d7a9ab220b7929b9c266c2dd9300fe693d53cf06

data/README.md

@@ -89,7 +89,7 @@ relationship. This option or `:idp_cert` must be present.
 IdP must issue a secure token. **Required**
 
 * `:reply` - The reply-to URL in your application for which a WSFed response should be
-posted. **Required**
+posted. Defaults to the OmniAuth callback URL. **Optional**
 
 * `:id_claim` - Name of the authentication claim that you want to use as OmniAuth's
 **uid** property.

data/lib/omniauth-wsfed/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module WSFed
-    VERSION = '0.3.1-beta'
+    VERSION = '0.3.2-beta'
   end
 end

data/lib/omniauth/strategies/wsfed.rb

@@ -19,7 +19,9 @@ module OmniAuth
 
       # Issues passive WS-Federation redirect for authentication...
       def request_phase
-        auth_request = OmniAuth::Strategies::WSFed::AuthRequest.new(options, :whr => @request.params['whr'])
+        settings = options.dup
+        settings[:reply] ||= callback_url
+        auth_request = OmniAuth::Strategies::WSFed::AuthRequest.new(settings, :whr => @request.params['whr'])
         redirect(auth_request.redirect_url)
       end
 

data/spec/omniauth/strategies/wsfed_spec.rb

@@ -105,4 +105,26 @@ describe OmniAuth::Strategies::WSFed, :type => :strategy do
   end
 end
 
+describe OmniAuth::Strategies::WSFed, :type => :strategy do
+  include OmniAuth::Test::StrategyTestCase
 
+  let(:home_realm_discovery) { '/auth/wsfed/home_realm_discovery' }
+  let(:wsfed_settings) do
+    {
+      :issuer => 'https://c4sc.accesscontrol.windows.net.com/v2/wsfederation',
+      :realm  => 'http://example.com/rp',
+    }
+  end
+  let(:strategy) { [OmniAuth::Strategies::WSFed, wsfed_settings] }
+  let(:home_realm) { 'http://identity.c4sc.com' }
+
+  describe 'request_phase: GET /auth/wsfed' do
+    context 'without :reply setting' do
+      it 'should use the default callback_url'  do
+        get 'auth/wsfed'
+        last_response.status.should   == 302
+        last_response.location.should include("wreply=http%3A%2F%2Fexample.org%2Fauth%2Fwsfed%2Fcallback")
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-wsfed
 version: !ruby/object:Gem::Version
-  version: 0.3.1.pre.beta
+  version: 0.3.2.pre.beta
 platform: ruby
 authors:
 - Keith Beckman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-19 00:00:00.000000000 Z
+date: 2015-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth