diff -Naur --exclude=.svn trunk/components/security/src/java/org/apache/jetspeed/security/impl/DefaultLoginModule.java js2.1dev/components/security/src/java/org/apache/jetspeed/security/impl/DefaultLoginModule.java
--- trunk/components/security/src/java/org/apache/jetspeed/security/impl/DefaultLoginModule.java	2006-10-10 18:44:58.796875000 +0200
+++ js2.1dev/components/security/src/java/org/apache/jetspeed/security/impl/DefaultLoginModule.java	2006-10-10 18:37:10.296875000 +0200
@@ -76,7 +76,7 @@
     private UserManager ums;
 
     /** The portal user role. */
-    private String portalUserRole;
+    protected String portalUserRole;
 
     /** <p>The user name.</p> */
     private String username;
diff -Naur --exclude=.svn trunk/components/security/src/java/org/apache/jetspeed/security/impl/ext/JBossLoginModule.java js2.1dev/components/security/src/java/org/apache/jetspeed/security/impl/ext/JBossLoginModule.java
--- trunk/components/security/src/java/org/apache/jetspeed/security/impl/ext/JBossLoginModule.java	2006-10-10 18:44:57.953125000 +0200
+++ js2.1dev/components/security/src/java/org/apache/jetspeed/security/impl/ext/JBossLoginModule.java	2006-10-10 18:38:11.046875000 +0200
@@ -14,6 +14,8 @@
 */
 package org.apache.jetspeed.security.impl.ext;
 
+import org.apache.jetspeed.security.impl.RolePrincipalImpl;
+
 import java.security.Principal;
 import java.security.acl.Group;
 import java.util.ArrayList;
@@ -95,6 +97,8 @@
     {
         // add UserPrincipal to subject
         subject.getPrincipals().add(getUserPrincipal(user));
-        subject.getPrincipals().add(new JBossGroup("Roles",getUserRoles(user)));
+	JBossGroup roles = new JBossGroup("Roles",getUserRoles(user));
+	roles.addMember(new RolePrincipalImpl(portalUserRole));
+	subject.getPrincipals().add(roles);
     }
 }