From: cantor <cantor@cb58f699-b61c-0410-a6fe-9272a202ed29>
Date: Wed, 26 Mar 2008 01:37:44 +0000 (+0000)
Subject: Fix for SSPCPP-84.
X-Git-Url: http://www.project-moonshot.org/gitweb/?p=shibboleth%2Fsp.git;a=commitdiff_plain;h=d50976370ec333da4fc19bf12480365dff1483cd

Fix for SSPCPP-84.

git-svn-id: https://svn.middleware.georgetown.edu/cpp-sp/trunk@2789 cb58f699-b61c-0410-a6fe-9272a202ed29
---

diff --git a/shibsp/handler/impl/SAML2SessionInitiator.cpp b/shibsp/handler/impl/SAML2SessionInitiator.cpp
index 9906907..0454e4d 100644
--- a/shibsp/handler/impl/SAML2SessionInitiator.cpp
+++ b/shibsp/handler/impl/SAML2SessionInitiator.cpp
@@ -242,11 +242,9 @@ pair<bool,long> SAML2SessionInitiator::run(SPRequest& request, string& entityID,
         }
 
         option = request.getParameter("target");
-        if (option)
+        if (option) {
             target = option;
-        if (acsByIndex.first && !acsByIndex.second) {
-            // Since we're passing the ACS by value, we need to compute the return URL,
-            // so we'll need the target resource for real.
+            // Always need to recover target URL to compute handler below.
             recoverRelayState(request.getApplication(), request, request, target, false);
         }