[jboss-cvs] Picketlink SVN: r262 - 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-05-26 15:50:42 -0400 (Wed, 26 May 2010)
New Revision: 262

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

Modified: federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPPostFormAuthenticator.java
===================================================================
--- federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPPostFormAuthenticator.java	2010-05-26 19:26:58 UTC (rev 261)
+++ federation/trunk/picketlink-bindings/src/main/java/org/picketlink/identity/federation/bindings/tomcat/sp/SPPostFormAuthenticator.java	2010-05-26 19:50:42 UTC (rev 262)
@@ -25,7 +25,6 @@
 
 import java.io.IOException;
 import java.security.Principal;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Set;
 
@@ -36,7 +35,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.bindings.tomcat.sp.holder.ServiceProviderSAMLContext;
 import org.picketlink.identity.federation.bindings.util.ValveUtil;
@@ -232,9 +230,8 @@
                //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();
                }