rubycas-client 2.3.3 → 2.3.5

data/History.txt

@@ -1,5 +1,11 @@
 = RubyCAS-Client Changelog
 
+== Version 2.3.5 :: 2011-11-17
+
+* Bug Fixes
+  * read_service_url will no longer include POST parameters on the
+    service url
+
 == Version 2.3.3 :: 2011-11-11
 
 * Bug Fixes

data/VERSION

@@ -1 +1 @@
-2.3.3
+2.3.5

data/lib/casclient/frameworks/rails/filter.rb

@@ -322,14 +322,16 @@ module CASClient
           def returning_from_gateway?(controller)
             controller.session[:cas_sent_to_gateway]
           end
-          
+
           def read_service_url(controller)
             if config[:service_url]
               log.debug("Using explicitly set service url: #{config[:service_url]}")
               return config[:service_url]
             end
-            
-            params = controller.params.dup
+
+            params = {}.with_indifferent_access
+            params.merge controller.request.query_parameters
+            params.merge controller.request.path_parameters
             params.delete(:ticket)
             service_url = controller.url_for(params)
             log.debug("Guessed service url: #{service_url.inspect}")

data/rubycas-client.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "rubycas-client"
-  s.version = "2.3.3"
+  s.version = "2.3.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Matt Zukowski", "Matt Walker", "Matt Campbell"]
-  s.date = "2011-11-11"
+  s.date = "2011-11-17"
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.rdoc"

data/spec/casclient/frameworks/rails/filter_spec.rb

@@ -5,12 +5,22 @@ require 'casclient/frameworks/rails/filter'
 describe CASClient::Frameworks::Rails::Filter do
 
   def controller_with_session(request = nil, session={})
+
+    query_parameters = {:ticket => "bogusticket", :renew => false}
+    parameters = query_parameters.dup
+
+    #TODO this really need to be replaced with a "real" rails controller
     request ||= mock_post_request
+    request.stub(:query_parameters) {query_parameters}
+    request.stub(:path_parameters) {{}}
     controller = double("Controller")
     controller.stub(:session) {session}
     controller.stub(:request) {request}
     controller.stub(:url_for) {"bogusurl"}
-    controller.stub(:params) {{:ticket => "bogusticket", :renew => false}}
+    controller.stub(:query_parameters) {query_parameters}
+    controller.stub(:path_parameters) {{}}
+    controller.stub(:parameters) {parameters}
+    controller.stub(:params) {parameters}
     controller
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubycas-client
 version: !ruby/object:Gem::Version
-  version: 2.3.3
+  version: 2.3.5
   prerelease: 
 platform: ruby
 authors:
@@ -11,11 +11,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-11-11 00:00:00.000000000Z
+date: 2011-11-17 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70335023438480 !ruby/object:Gem::Requirement
+  requirement: &70130070436840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -23,10 +23,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70335023438480
+  version_requirements: *70130070436840
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &70335023437940 !ruby/object:Gem::Requirement
+  requirement: &70130070436280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -34,10 +34,10 @@ dependencies:
         version: 1.6.1
   type: :development
   prerelease: false
-  version_requirements: *70335023437940
+  version_requirements: *70130070436280
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70335023437440 !ruby/object:Gem::Requirement
+  requirement: &70130070435760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -45,10 +45,10 @@ dependencies:
         version: 2.7.0
   type: :development
   prerelease: false
-  version_requirements: *70335023437440
+  version_requirements: *70130070435760
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70335023436920 !ruby/object:Gem::Requirement
+  requirement: &70130070435180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -56,10 +56,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70335023436920
+  version_requirements: *70130070435180
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70335023436400 !ruby/object:Gem::Requirement
+  requirement: &70130070434580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -67,10 +67,10 @@ dependencies:
         version: 1.6.2
   type: :development
   prerelease: false
-  version_requirements: *70335023436400
+  version_requirements: *70130070434580
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70335023435900 !ruby/object:Gem::Requirement
+  requirement: &70130070434100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -78,10 +78,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70335023435900
+  version_requirements: *70130070434100
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70335023435380 !ruby/object:Gem::Requirement
+  requirement: &70130070433500 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -89,10 +89,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70335023435380
+  version_requirements: *70130070433500
 - !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70335023434860 !ruby/object:Gem::Requirement
+  requirement: &70130070432960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -100,7 +100,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70335023434860
+  version_requirements: *70130070432960
 description: 
 email: 
 executables: []
@@ -177,7 +177,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2365484882712424697
+      hash: -4466061466971345462
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: