[jboss-cvs] Picketlink SVN: r158 - federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2010-03-03 15:34:15 -0500 (Wed, 03 Mar 2010)
New Revision: 158

Modified:
   federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPRedirectFormAuthenticator.java
Log:
PLFED-45: remove the stale cast

Modified: federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPRedirectFormAuthenticator.java
===================================================================
--- federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPRedirectFormAuthenticator.java	2010-03-03 16:18:59 UTC (rev 157)
+++ federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPRedirectFormAuthenticator.java	2010-03-03 20:34:15 UTC (rev 158)
@@ -27,7 +27,6 @@
 import java.io.IOException;
 import java.security.GeneralSecurityException;
 import java.security.Principal;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Set;
 import java.util.StringTokenizer;
@@ -41,7 +40,6 @@
 import org.apache.catalina.connector.Request;
 import org.apache.catalina.connector.Response;
 import org.apache.catalina.deploy.LoginConfig;
-import org.apache.catalina.realm.GenericPrincipal;
 import org.apache.log4j.Logger;
 import org.picketlink.identity.federation.api.saml.v2.request.SAML2Request;
 import org.picketlink.identity.federation.bindings.tomcat.sp.holder.ServiceProviderSAMLContext;
@@ -258,10 +256,9 @@
 
                //Map to JBoss specific principal
                if((new ServerDetector()).isJboss() || jbossEnv)
-               {
-                  GenericPrincipal gp = (GenericPrincipal) principal;
+               { 
                   //Push a context
-                  ServiceProviderSAMLContext.push(username, Arrays.asList(gp.getRoles()));
+                  ServiceProviderSAMLContext.push(username, roles);
                   principal = context.getRealm().authenticate(username, password); 
                   ServiceProviderSAMLContext.clear();
                }