[jboss-cvs] Picketlink SVN: r1298 - federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2011-10-10 11:40:56 -0400 (Mon, 10 Oct 2011)
New Revision: 1298

Modified:
   federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AuthenticationHandler.java
Log:
PLFED-240: move setting the destination right after determining it

Modified: federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AuthenticationHandler.java
===================================================================
--- federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AuthenticationHandler.java	2011-10-07 21:23:43 UTC (rev 1297)
+++ federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AuthenticationHandler.java	2011-10-10 15:40:56 UTC (rev 1298)
@@ -186,6 +186,8 @@
          if (trace)
             log.trace("Destination=" + destination);
 
+         response.setDestination(destination);
+
          HttpSession session = BaseSAML2Handler.getHttpSession(request);
          Principal userPrincipal = (Principal) session.getAttribute(GeneralConstants.PRINCIPAL_ID);
          if (userPrincipal == null)
@@ -210,7 +212,6 @@
                   .getAttribute(GeneralConstants.IDENTITY_SERVER);
             identityServer.stack().register(session.getId(), destination, isPost);
 
-            response.setDestination(destination);
             response.setResultingDocument(samlResponse);
             response.setRelayState(request.getRelayState());
             response.setPostBindingForResponse(isPost);