omniauth-cas 1.0.3 → 1.0.4

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDMxYjQ1ZmY1NjAyNGMzOTMyZDE1MTlkNzVjNGIxOTA3YzdjNDNhOQ==
+    OTU0MmMyOWM1MWY0YzBiNjMzZDRkZmE4NTljYTA4NTc3M2NmM2FiYg==
   data.tar.gz: !binary |-
-    YzhjYTZiMDY2NDhhMWMxMjA2ZDA0M2QxNDUyMTY3MzE4MWZmN2I1MQ==
+    YjM4MmU1MTAwNThmNjJkNWU2OTJiZjJiMTc5ZDNlYjM0ZDQ2M2E3ZQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NTEzYmQ2NTA3NGRlNjBhZjAxNGYwM2U4N2Y3ZDE0YzlhMmM0ZDRmYWRlZTky
-    MTBlNmI4ZmYwZDU3NTU4MjZmMWY1MjZmNjEwZWQyNGI3ZGJiOTZjMjZlNGVm
-    YmI5YzIyNTE0YzUzN2FlMGJlZDkzZjFhZTE3NDZiYmFmNzQ5MmM=
+    ZTdkZjEzMDY4ODVjMzY1NDQ2MmUxZTAyMzRkMGUwNzcxYzAyOGM5MzBlNDZi
+    ZjU1OTk0OWU1ODIzNTIwYTIxNWM5ZDEzNzFiY2UyODJhYzY3NDRkZmM5ODcy
+    NTBlMGM5YWFhZWU1MDBlMTI2YWQ0NTRmODRkZjhiYmVmMjc2ZDI=
   data.tar.gz: !binary |-
-    YmZlZDFmMWM0ZjE2ODExY2U4ZGU0YzBiY2U1OGVkMWQzM2ZlMzViODc0NzJj
-    NmQ2ZTc4MjAxZTllYzE5YTlmNmJlNTE4ZGRmNjlmOGVhYzc3MTFlZjQ3Nzcy
-    NzRhMjhiNTg5ZGVmY2EyYmZhNGExZWZiODE0ZWY1ZjkzMGY2NWQ=
+    OTY0NjU1NTFhZGZmNjdlN2E2OTY5MjNkMTU0NDY5NWQ1ZjQ3N2RiZWFjNmZj
+    MDA3MmY4M2U1MjMwOThkZmRhNzYwZDI5NzljYjRiYzk5YjY5MWFlNjBjYjY0
+    NzBhNjE1OWViZjMzMGVmYzkzMWM5MTk5MGNmNjk1MTU3YjYxYWU=

data/lib/omniauth/cas/version.rb

@@ -1,5 +1,5 @@
 module Omniauth
   module Cas
-    VERSION = '1.0.3'
+    VERSION = '1.0.4'
   end
 end

data/lib/omniauth/strategies/cas/service_ticket_validator.rb

@@ -41,15 +41,20 @@ module OmniAuth
 
           {}.tap do |hash|
             node.children.each do |e|
+              node_name = e.name.sub(/^cas:/, '')
               unless e.kind_of?(Nokogiri::XML::Text) ||
-                     e.name == 'cas:proxies' ||
-                     e.name == 'proxies'
+                     node_name == 'proxies'
                 # There are no child elements
                 if e.element_children.count == 0
-                  hash[e.name.sub(/^cas:/, '')] = e.content
+                  hash[node_name] = e.content
                 elsif e.element_children.count
-                  hash[e.name.sub(/^cas:/, '')] = [] if hash[e.name.sub(/^cas:/, '')].nil?
-                  hash[e.name.sub(/^cas:/, '')].push parse_user_info e
+                  # JASIG style extra attributes
+                  if node_name == 'attributes'
+                    hash.merge! parse_user_info e
+                  else
+                    hash[node_name] = [] if hash[node_name].nil?
+                    hash[node_name].push parse_user_info e
+                  end
                 end
               end
             end

data/spec/fixtures/cas_success_jasig.xml

@@ -0,0 +1,16 @@
+<cas:serviceResponse xmlns:cas='http://www.yale.edu/tp/cas'>
+  <cas:authenticationSuccess>
+    <cas:user>psegel</cas:user>
+    <cas:attributes>
+      <cas:employeeid>54</cas:employeeid>
+      <cas:first_name>P. Segel</cas:first_name>
+      <cas:first_name>Peter</cas:first_name>
+      <cas:last_name>Segel</cas:last_name>
+      <cas:email>psegel@intridea.com</cas:email>
+      <cas:location>Washington, D.C.</cas:location>
+      <cas:image>/images/user.jpg</cas:image>
+      <cas:phone>555-555-5555</cas:phone>
+      <cas:hire_date>2004-07-13</cas:hire_date>
+    </cas:attributes>
+  </cas:authenticationSuccess>
+</cas:serviceResponse>

data/spec/omniauth/strategies/cas_spec.rb

@@ -81,72 +81,84 @@ describe OmniAuth::Strategies::CAS, type: :strategy do
   end
 
   describe 'GET /auth/cas/callback with a valid ticket' do
-    let(:return_url) { 'http://127.0.0.10/?some=parameter' }
+    shared_examples :successful_validation do
+      before do
+        stub_request(:get, /^http:\/\/cas.example.org:8080?\/serviceValidate\?([^&]+&)?ticket=593af/)
+          .with { |request| @request_uri = request.uri.to_s }
+          .to_return( body: File.read("spec/fixtures/#{xml_file_name}") )
 
-    before do
-      stub_request(:get, /^http:\/\/cas.example.org:8080?\/serviceValidate\?([^&]+&)?ticket=593af/)
-        .with { |request| @request_uri = request.uri.to_s }
-        .to_return( body: File.read('spec/fixtures/cas_success.xml') )
+        get "/auth/cas/callback?ticket=593af&url=#{return_url}"
+      end
 
-      get "/auth/cas/callback?ticket=593af&url=#{return_url}"
-    end
+      it 'should strip the ticket parameter from the callback URL' do
+        @request_uri.scan('ticket=').length.should == 1
+      end
 
-    it 'should strip the ticket parameter from the callback URL' do
-      @request_uri.scan('ticket=').length.should == 1
-    end
+      it 'should properly encode the service URL' do
+        WebMock.should have_requested(:get, 'http://cas.example.org:8080/serviceValidate')
+          .with(query: {
+            ticket:  '593af',
+            service: 'http://example.org/auth/cas/callback?url=' + Rack::Utils.escape('http://127.0.0.10/?some=parameter')
+          })
+      end
 
-    it 'should properly encode the service URL' do
-      WebMock.should have_requested(:get, 'http://cas.example.org:8080/serviceValidate')
-        .with(query: {
-          ticket:  '593af',
-          service: 'http://example.org/auth/cas/callback?url=' + Rack::Utils.escape('http://127.0.0.10/?some=parameter')
-        })
-    end
+      context "request.env['omniauth.auth']" do
+        subject { last_request.env['omniauth.auth'] }
 
-    context "request.env['omniauth.auth']" do
-      subject { last_request.env['omniauth.auth'] }
+        it { should be_kind_of Hash }
 
-      it { should be_kind_of Hash }
+        its(:provider) { should == :cas }
 
-      its(:provider) { should == :cas }
+        its(:uid) { should == '54'}
 
-      its(:uid) { should == '54'}
+        context 'the info hash' do
+          subject { last_request.env['omniauth.auth']['info'] }
 
-      context 'the info hash' do
-        subject { last_request.env['omniauth.auth']['info'] }
+          it { should have(6).items }
 
-        it { should have(6).items }
+          its(:name)       { should == 'Peter Segel' }
+          its(:first_name) { should == 'Peter' }
+          its(:last_name)  { should == 'Segel' }
+          its(:email)      { should == 'psegel@intridea.com' }
+          its(:location)   { should == 'Washington, D.C.' }
+          its(:image)      { should == '/images/user.jpg' }
+          its(:phone)      { should == '555-555-5555' }
+        end
 
-        its(:name)       { should == 'Peter Segel' }
-        its(:first_name) { should == 'Peter' }
-        its(:last_name)  { should == 'Segel' }
-        its(:email)      { should == 'psegel@intridea.com' }
-        its(:location)   { should == 'Washington, D.C.' }
-        its(:image)      { should == '/images/user.jpg' }
-        its(:phone)      { should == '555-555-5555' }
-      end
+        context 'the extra hash' do
+          subject { last_request.env['omniauth.auth']['extra'] }
 
-      context 'the extra hash' do
-        subject { last_request.env['omniauth.auth']['extra'] }
+          it { should have(3).items }
 
-        it { should have(3).items }
+          its(:user)       { should == 'psegel' }
+          its(:employeeid) { should == '54' }
+          its(:hire_date)  { should == '2004-07-13' }
+        end
 
-        its(:user)       { should == 'psegel' }
-        its(:employeeid) { should == '54' }
-        its(:hire_date)  { should == '2004-07-13' }
-      end
+        context 'the credentials hash' do
+          subject { last_request.env['omniauth.auth']['credentials'] }
 
-      context 'the credentials hash' do
-        subject { last_request.env['omniauth.auth']['credentials'] }
+          it { should have(1).items }
 
-        it { should have(1).items }
+          its(:ticket) { should == '593af' }
+        end
+      end
 
-        its(:ticket) { should == '593af' }
+      it 'should call through to the master app' do
+        last_response.body.should == 'true'
       end
     end
 
-    it 'should call through to the master app' do
-      last_response.body.should == 'true'
+    let(:return_url) { 'http://127.0.0.10/?some=parameter' }
+
+    context 'with JASIG flavored XML' do
+      let(:xml_file_name) { 'cas_success_jasig.xml' }
+      it_behaves_like :successful_validation
+    end
+
+    context 'with classic XML' do
+      let(:xml_file_name) { 'cas_success.xml' }
+      it_behaves_like :successful_validation
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-cas
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.4
 platform: ruby
 authors:
 - Derek Lindahl
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-27 00:00:00.000000000 Z
+date: 2013-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth
@@ -160,6 +160,7 @@ files:
 - omniauth-cas.gemspec
 - spec/fixtures/cas_failure.xml
 - spec/fixtures/cas_success.xml
+- spec/fixtures/cas_success_jasig.xml
 - spec/omniauth/strategies/cas/configuration_spec.rb
 - spec/omniauth/strategies/cas/service_ticket_validator_spec.rb
 - spec/omniauth/strategies/cas_spec.rb
@@ -190,6 +191,7 @@ summary: CAS Strategy for OmniAuth
 test_files:
 - spec/fixtures/cas_failure.xml
 - spec/fixtures/cas_success.xml
+- spec/fixtures/cas_success_jasig.xml
 - spec/omniauth/strategies/cas/configuration_spec.rb
 - spec/omniauth/strategies/cas/service_ticket_validator_spec.rb
 - spec/omniauth/strategies/cas_spec.rb