Skip to content

Commit 8fa9d2c

Browse files
committed
utils: fix SAMLUtils's authnrequest maker
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
1 parent 2a264cc commit 8fa9d2c

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

utils/src/org/apache/cloudstack/utils/auth/SAMLUtils.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,9 @@
6464
public class SAMLUtils {
6565
public static final Logger s_logger = Logger.getLogger(SAMLUtils.class);
6666

67-
public static final String SAML_NS = "saml-";
67+
public static final String SAML_NS = "saml://";
68+
69+
public static final String CERTIFICATE_NAME = "SAMLSP_CERTIFICATE";
6870

6971
public static String createSAMLId(String uid) {
7072
return SAML_NS + uid;
@@ -108,15 +110,15 @@ public static AuthnRequest buildAuthnRequestObject(String authnId, String spId,
108110
authnRequest.setID(authnId);
109111
authnRequest.setDestination(idpUrl);
110112
authnRequest.setVersion(SAMLVersion.VERSION_20);
111-
authnRequest.setForceAuthn(true);
113+
authnRequest.setForceAuthn(false);
112114
authnRequest.setIsPassive(false);
113115
authnRequest.setIssuer(issuer);
114116
authnRequest.setIssueInstant(new DateTime());
115-
authnRequest.setProviderName(spId);
116117
authnRequest.setProtocolBinding(SAMLConstants.SAML2_REDIRECT_BINDING_URI);
117118
authnRequest.setAssertionConsumerServiceURL(consumerUrl);
118-
authnRequest.setNameIDPolicy(nameIdPolicy);
119-
authnRequest.setRequestedAuthnContext(requestedAuthnContext);
119+
//authnRequest.setProviderName(spId);
120+
//authnRequest.setNameIDPolicy(nameIdPolicy);
121+
//authnRequest.setRequestedAuthnContext(requestedAuthnContext);
120122

121123
return authnRequest;
122124
}

0 commit comments

Comments
 (0)