[jboss-identity-commits] JBoss Identity SVN: r710 - identity-federation/trunk/jboss-identity-web/src/main/java/org/jboss/identity/federation/web/util.
Author: marcelkolsteren
Date: 2009-08-16 06:12:11 -0400 (Sun, 16 Aug 2009)
New Revision: 710
Modified:
identity-federation/trunk/jboss-identity-web/src/main/java/org/jboss/identity/federation/web/util/IDPWebRequestUtil.java
Log:
Fixed problem in IDPWebRequestUtil: redirects contained an extra question mark in the
URL.
Modified:
identity-federation/trunk/jboss-identity-web/src/main/java/org/jboss/identity/federation/web/util/IDPWebRequestUtil.java
===================================================================
---
identity-federation/trunk/jboss-identity-web/src/main/java/org/jboss/identity/federation/web/util/IDPWebRequestUtil.java 2009-08-16
10:07:03 UTC (rev 709)
+++
identity-federation/trunk/jboss-identity-web/src/main/java/org/jboss/identity/federation/web/util/IDPWebRequestUtil.java 2009-08-16
10:12:11 UTC (rev 710)
@@ -313,32 +313,26 @@
boolean supportSignature)
{
StringBuilder sb = new StringBuilder();
- sb.append("?");
-
- if(redirectProfile)
+
+ if (supportSignature)
{
- if(supportSignature)
+ try
{
- try
- {
-
sb.append(RedirectBindingSignatureUtil.getSAMLResponseURLWithSignature(urlEncodedResponse,
- urlEncodedRelayState, keyManager.getSigningKey()));
- }
- catch (Exception e)
- {
- log.trace(e);
- }
- }
- else
+
sb.append(RedirectBindingSignatureUtil.getSAMLResponseURLWithSignature(urlEncodedResponse,
+ urlEncodedRelayState, keyManager.getSigningKey()));
+ }
+ catch (Exception e)
{
- sb.append("?SAMLResponse=").append(urlEncodedResponse);
- if(urlEncodedRelayState != null && urlEncodedRelayState.length() >
0)
- sb.append("&RelayState=").append(urlEncodedRelayState);
+ log.trace(e);
}
- return sb.toString();
-
}
- return null;
+ else
+ {
+ sb.append("?SAMLResponse=").append(urlEncodedResponse);
+ if (urlEncodedRelayState != null && urlEncodedRelayState.length() >
0)
+ sb.append("&RelayState=").append(urlEncodedRelayState);
+ }
+ return sb.toString();
}
/**