[java-identity-provider COMMIT] /trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInit...

[noreply at shibboleth.net]

Author: scantor
Date: Fri Aug 15 19:49:10 2014
New Revision: 6428

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=6428&view=rev
Log:
Remove TODO.

Modified:
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInitiatedSSORequestMessageDecoder.java

Modified: trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInitiatedSSORequestMessageDecoder.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInitiatedSSORequestMessageDecoder.java?rev=6428&r1=6427&r2=6428&view=diff
==============================================================================
--- trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInitiatedSSORequestMessageDecoder.java (original)
+++ trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/impl/IdPInitiatedSSORequestMessageDecoder.java Fri Aug 15 19:49:10 2014
@@ -124,10 +124,6 @@
         final NameIDPolicy nip = nipBuilder.buildObject();
         nip.setAllowCreate(true);
         authnRequest.setNameIDPolicy(nip);
-        
-        // TODO see UnsolicitedSSODecoder in v2. Do we need to support adding a statically configured ProtocolBinding,
-        // and dynamically resolved ACS URL if not in the request? The latter we might not do in the decoder.
-        
         authnRequest.setAssertionConsumerServiceURL(ssoRequest.getAssertionConsumerServiceURL());
         authnRequest.setIssueInstant(new DateTime(ssoRequest.getTime(), ISOChronology.getInstanceUTC()));
         authnRequest.setVersion(SAMLVersion.VERSION_20);