[jboss-svn-commits] JBL Code SVN: r21557 - labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security.

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

Author: jervisliu
Date: 2008-08-14 22:03:12 -0400 (Thu, 14 Aug 2008)
New Revision: 21557

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/DefaultAuthenticator.java
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/NilAuthenticator.java
Log:
remove the line that adds everyone as admin in authenticator, as this now is handled by a flag in configuration that can enable/disable authorization. 

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/DefaultAuthenticator.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/DefaultAuthenticator.java	2008-08-15 01:59:45 UTC (rev 21556)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/DefaultAuthenticator.java	2008-08-15 02:03:12 UTC (rev 21557)
@@ -36,9 +36,6 @@
             return false;
         }
         log.info( "User logged in via default authentication module (no security check).");
-        
-        //Add everyone as admin.
-		Identity.instance().addRole(RoleTypes.ADMIN);
 		
         return true;
     }

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/NilAuthenticator.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/NilAuthenticator.java	2008-08-15 01:59:45 UTC (rev 21556)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/server/security/NilAuthenticator.java	2008-08-15 02:03:12 UTC (rev 21557)
@@ -33,8 +33,6 @@
     
     public boolean authenticate() {
         log.info( "All users are guests.");
-        //Add everyone as admin.
-		Identity.instance().addRole(RoleTypes.ADMIN);
 
         return true;
     }