omniauth-cas 0.0.6.alpha → 0.0.6

data/History.md

@@ -1,4 +1,11 @@
 
+0.0.6 / 2012-03-07 
+==================
+
+  * Updated 'webmock' gem dependency
+  * Update 'bourne' gem dependency
+  * Updated 'simplecov' gem dependency.
+
 0.0.5 / 2012-02-10 
 ==================
 

data/README.md

@@ -43,8 +43,7 @@ Other configuration options:
   * `service_validate_url` - The URL to use to validate a user. Defaults to `'/serviceValidate'`
   * `logout_url` - The URL to use to logout a user. Defaults to `'/logout'`
   * `uid_key` - The user data attribute to use as your user's unique identifier. Defaults to `'user'` (which usually contains the user's login name)
-  * `ca_path` - Optional when `ssl` is `true`. Explicitly sets path of a CA certification directory.
-                Defaults to something sensible on Ubuntu and OS X. See [Net::HTTP][net_http] for more details
+  * `ca_path` - Optional when `ssl` is `true`. Sets path of a CA certification directory. See [Net::HTTP][net_http] for more details
 
 ## Contributing
 

data/lib/omniauth/cas/version.rb

@@ -1,5 +1,5 @@
 module Omniauth
   module Cas
-    VERSION = "0.0.6.alpha"
+    VERSION = "0.0.6"
   end
 end

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

@@ -9,19 +9,6 @@ module OmniAuth
 
         VALIDATION_REQUEST_HEADERS = { 'Accept' => '*/*' }
 
-        UbuntuCAPath  = '/etc/ssl/certs'
-        OSXCAPath     = '/opt/local/share/curl/curl-ca-bundle.crt'
-
-        DefaultCAPath = begin
-          if File.exists?(UbuntuCAPath)
-            UbuntuCAPath
-          elsif File.exists?(OSXCAPath)
-            OSXCAPath
-          else
-            nil
-          end
-        end
-
         # Build a validator from a +configuration+, a
         # +return_to+ URL, and a +ticket+.
         #
@@ -30,7 +17,7 @@ module OmniAuth
         # @param [String] ticket the service ticket to validate
         def initialize(strategy, options, return_to_url, ticket)
           @options = options
-          @uri = Addressable::URI.parse(strategy.service_validate_url(return_to_url, ticket))
+          @uri = URI.parse(strategy.service_validate_url(return_to_url, ticket))
         end
 
         # Request validation of the ticket from the CAS server's
@@ -89,22 +76,16 @@ module OmniAuth
         # retrieves the `<cas:serviceResponse>` XML from the CAS server
         def get_service_response_body
           result = ''
-
           http = Net::HTTP.new(@uri.host, @uri.port)
-          http.use_ssl = (@uri.port == 443 or @uri.scheme == "https")
-
+          http.use_ssl = @uri.port == 443 || @uri.instance_of?(URI::HTTPS)
           if http.use_ssl?
             http.verify_mode = OpenSSL::SSL::VERIFY_NONE if @options.disable_ssl_verification?
-
-            ca_path = @options.ca_path or DefaultCAPath
-            http.ca_path = ca_path if ca_path
+            http.ca_path = @options.ca_path
           end
-
           http.start do |c|
-            response = c.get( @uri.omit(:scheme, :authority).to_s, VALIDATION_REQUEST_HEADERS.dup )
+            response = c.get "#{@uri.path}?#{@uri.query}", VALIDATION_REQUEST_HEADERS.dup
             result = response.body
           end
-
           result
         end
 

data/omniauth-cas.gemspec

@@ -20,11 +20,11 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'addressable', '~> 2.2'
 
   gem.add_development_dependency 'rake',        '~> 0.9'
-  gem.add_development_dependency 'webmock',     '~> 1.7'
-  gem.add_development_dependency 'simplecov',   '~> 0.5.4'
+  gem.add_development_dependency 'webmock',     '~> 1.8.2'
+  gem.add_development_dependency 'simplecov',   '~> 0.6.1'
   gem.add_development_dependency 'rspec',       '~> 2.8'
   gem.add_development_dependency 'rack-test',   '~> 0.6'
-  gem.add_development_dependency 'bourne',      '~> 1.0'
+  gem.add_development_dependency 'bourne',      '~> 1.1.1'
 
   gem.add_development_dependency 'awesome_print'
 

data/spec/omniauth/strategies/cas/service_ticket_validator_spec.rb

@@ -22,7 +22,7 @@ describe OmniAuth::Strategies::CAS::ServiceTicketValidator do
 
   describe "#user_info" do
     before do
-      stub_request(:get, "https://example.org:80/serviceValidate").to_return(:status => 200, :body => '')
+      stub_request(:get, "https://example.org/serviceValidate?").to_return(:status => 200, :body => '')
       validator.user_info
     end
     it "should use the configured CA path" do

data/spec/omniauth/strategies/cas_spec.rb

@@ -56,7 +56,7 @@ describe OmniAuth::Strategies::CAS, :type => :strategy do
 
   describe 'GET /auth/cas/callback with an invalid ticket' do
     before do
-      stub_request(:get, /^https:\/\/cas.example.org(:443)?(:80)?\/serviceValidate\?([^&]+&)?ticket=9391d/).
+      stub_request(:get, /^https:\/\/cas.example.org(:443)?\/serviceValidate\?([^&]+&)?ticket=9391d/).
          to_return( :body => File.read('spec/fixtures/cas_failure.xml') )
       get '/auth/cas/callback?ticket=9391d'
     end
@@ -72,7 +72,7 @@ describe OmniAuth::Strategies::CAS, :type => :strategy do
   describe 'GET /auth/cas/callback with a valid ticket' do
     let(:return_url) { "http://127.0.0.10/?some=parameter" }
     before do
-      stub_request(:get, /^https:\/\/cas.example.org(:443)?(:80)?\/serviceValidate\?([^&]+&)?ticket=593af/).
+      stub_request(:get, /^https:\/\/cas.example.org(:443)?\/serviceValidate\?([^&]+&)?ticket=593af/).
          with { |request| @request_uri = request.uri.to_s }.
          to_return( :body => File.read('spec/fixtures/cas_success.xml') )
 
@@ -84,7 +84,7 @@ describe OmniAuth::Strategies::CAS, :type => :strategy do
     end
 
     it "should properly encode the service URL" do
-      WebMock.should have_requested(:get, "https://cas.example.org:80/serviceValidate")
+      WebMock.should have_requested(:get, "https://cas.example.org/serviceValidate")
         .with(:query => {
           :ticket => "593af",
           :service => "http://example.org/auth/cas/callback?url=" + Rack::Utils.escape("http://127.0.0.10/?some=parameter")

metadata

@@ -1,19 +1,19 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-cas
 version: !ruby/object:Gem::Version
-  version: 0.0.6.alpha
-  prerelease: 6
+  version: 0.0.6
+  prerelease: 
 platform: ruby
 authors:
 - Derek Lindahl
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-21 00:00:00.000000000Z
+date: 2012-03-08 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth
-  requirement: &2160884240 !ruby/object:Gem::Requirement
+  requirement: &2160743320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.0.2
   type: :runtime
   prerelease: false
-  version_requirements: *2160884240
+  version_requirements: *2160743320
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &2160883720 !ruby/object:Gem::Requirement
+  requirement: &2160742740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: '1.5'
   type: :runtime
   prerelease: false
-  version_requirements: *2160883720
+  version_requirements: *2160742740
 - !ruby/object:Gem::Dependency
   name: addressable
-  requirement: &2160883260 !ruby/object:Gem::Requirement
+  requirement: &2160742260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '2.2'
   type: :runtime
   prerelease: false
-  version_requirements: *2160883260
+  version_requirements: *2160742260
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2160882800 !ruby/object:Gem::Requirement
+  requirement: &2160741700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,32 +54,32 @@ dependencies:
         version: '0.9'
   type: :development
   prerelease: false
-  version_requirements: *2160882800
+  version_requirements: *2160741700
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &2160882340 !ruby/object:Gem::Requirement
+  requirement: &2160741200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: 1.8.2
   type: :development
   prerelease: false
-  version_requirements: *2160882340
+  version_requirements: *2160741200
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &2160871780 !ruby/object:Gem::Requirement
+  requirement: &2160740720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.5.4
+        version: 0.6.1
   type: :development
   prerelease: false
-  version_requirements: *2160871780
+  version_requirements: *2160740720
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2160870320 !ruby/object:Gem::Requirement
+  requirement: &2160740120 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +87,10 @@ dependencies:
         version: '2.8'
   type: :development
   prerelease: false
-  version_requirements: *2160870320
+  version_requirements: *2160740120
 - !ruby/object:Gem::Dependency
   name: rack-test
-  requirement: &2160869740 !ruby/object:Gem::Requirement
+  requirement: &2160739660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,21 +98,21 @@ dependencies:
         version: '0.6'
   type: :development
   prerelease: false
-  version_requirements: *2160869740
+  version_requirements: *2160739660
 - !ruby/object:Gem::Dependency
   name: bourne
-  requirement: &2160869160 !ruby/object:Gem::Requirement
+  requirement: &2160606980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.1.1
   type: :development
   prerelease: false
-  version_requirements: *2160869160
+  version_requirements: *2160606980
 - !ruby/object:Gem::Dependency
   name: awesome_print
-  requirement: &2160868660 !ruby/object:Gem::Requirement
+  requirement: &2160606580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,7 +120,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2160868660
+  version_requirements: *2160606580
 description: 
 email:
 - dlindahl@customink.com
@@ -163,9 +163,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.10