lib/omni_auth/strategies/saml.rb in cul_omniauth-0.5.3 vs lib/omni_auth/strategies/saml.rb in cul_omniauth-0.5.5

@@ -77,11 +77,13 @@
       credentials do
         prune!({ ticket: @ticket })
       end
 
       def login_url(service)
-        cas_url + append_params(options.login_url, { TARGET: service })
+        target_url = service.split('?').first
+        parms = { TARGET: target_url }
+        cas_url + append_params(options.login_url, parms)
       end
       def logout_url(service)
         cas_url + append_params(options.logout_url, { service: service})
       end
       # Build a CAS host with protocol and port
@@ -115,14 +117,13 @@
           raise ArgumentError.new(":host and :login_url MUST be provided")
         end
       end
 
       def service_validate_url(service_url, ticket)
-        service_url = Addressable::URI.parse(service_url).origin
+        target_url = service_url.split('?').first
+
         parms = {
-          TARGET: service_url,
-#          service: service_url,
-#          ticket: ticket
+          TARGET: target_url,
         }
         r = cas_url + append_params(options.service_validate_url, parms)
         r
       end
 
\ No newline at end of file