svn commit: r1347698 - in /cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso: AbstractServiceProviderFilter.java RequestAssertionConsumerService.java

[coheigea]

Author: coheigea
Date: Thu Jun  7 16:06:07 2012
New Revision: 1347698

URL: http://svn.apache.org/viewvc?rev=1347698&view=rev
Log:
[CXF-4365] - SAML SSO does not set Destination attribute when creating a signed 
AuthnRequest

Modified:
    
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractServiceProviderFilter.java
    
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/RequestAssertionConsumerService.java

Modified: 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractServiceProviderFilter.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractServiceProviderFilter.java?rev=1347698&r1=1347697&r2=1347698&view=diff
==============================================================================
--- 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractServiceProviderFilter.java
 (original)
+++ 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractServiceProviderFilter.java
 Thu Jun  7 16:06:07 2012
@@ -244,6 +244,7 @@ public abstract class AbstractServicePro
                 m, getIssuerId(m), getAbsoluteAssertionServiceAddress(m)
             );
         if (isSignRequest()) {
+            authnRequest.setDestination(idpServiceAddress);
             signAuthnRequest(authnRequest);
         }
         Element authnRequestElement = OpenSAMLUtil.toDom(authnRequest, doc);

Modified: 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/RequestAssertionConsumerService.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/RequestAssertionConsumerService.java?rev=1347698&r1=1347697&r2=1347698&view=diff
==============================================================================
--- 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/RequestAssertionConsumerService.java
 (original)
+++ 
cxf/trunk/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/RequestAssertionConsumerService.java
 Thu Jun  7 16:06:07 2012
@@ -56,6 +56,7 @@ import org.apache.cxf.rs.security.saml.s
 import org.apache.cxf.rs.security.saml.sso.state.ResponseState;
 import org.apache.ws.security.WSSecurityException;
 import org.apache.ws.security.saml.ext.OpenSAMLUtil;
+import org.apache.ws.security.util.DOM2Writer;
 import org.opensaml.xml.XMLObject;
 
 @Path("sso")
@@ -255,6 +256,8 @@ public class RequestAssertionConsumerSer
             throw new WebApplicationException(400);
         }
         
+        LOG.fine("Received response: " + 
DOM2Writer.nodeToString(responseDoc.getDocumentElement()));
+        
         XMLObject responseObject = null;
         try {
             responseObject = 
OpenSAMLUtil.fromDom(responseDoc.getDocumentElement());