Author: anil.saldhana(a)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);