Permalink
Browse files

Merge pull request #123 from pedroigor/master

Changed to consider the request.getMethod during the saml token retrieving/parsing. Instead of idpUsesPostBinding.
  • Loading branch information...
2 parents 6c7deac + bb6bf8a commit 1e4c558f4ffd48cec8c8696b0fb6df209afc516f @pedroigor pedroigor committed Jun 26, 2012
View
2 ...va/org/picketlink/identity/federation/bindings/tomcat/sp/AbstractSPFormAuthenticator.java
@@ -360,7 +360,7 @@ private boolean handleSAMLRequest(Request request, Response response, LoginConfi
try {
ServiceProviderSAMLRequestProcessor requestProcessor = new ServiceProviderSAMLRequestProcessor(
- isPOSTBindingResponse(), this.serviceURL);
+ request.getMethod().equals("POST"), this.serviceURL);
requestProcessor.setTrustKeyManager(keyManager);
requestProcessor.setConfiguration(spConfiguration);
boolean result = requestProcessor.process(samlRequest, httpContext, handlers, chainLock);
View
1 ...ink/test/identity/federation/bindings/workflow/SAML2LogoutTomcatWorkflowUnitTestCase.java
@@ -225,6 +225,7 @@ public String getName() {
request = new MockCatalinaRequest();
request.setSession(session);
+ request.setMethod("GET");
request.setParameter("SAMLRequest", RedirectBindingUtil.urlDecode(logoutRequest));
request.setParameter("RelayState", relayState);

0 comments on commit 1e4c558

Please sign in to comment.