You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by jg...@apache.org on 2005/02/11 07:12:58 UTC

svn commit: r153368 - geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java

Author: jgenender
Date: Thu Feb 10 22:12:58 2005
New Revision: 153368

URL: http://svn.apache.org/viewcvs?view=rev&rev=153368
Log:
Added roleName to the 'Role does not exist in this configuration' error message.

Modified:
    geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java

Modified: geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java?view=diff&r1=153367&r2=153368
==============================================================================
--- geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java (original)
+++ geronimo/trunk/modules/jetty/src/java/org/apache/geronimo/jetty/interceptor/SecurityContextBeforeAfter.java Thu Feb 10 22:12:58 2005
@@ -367,7 +367,7 @@
             Set principalSet = new HashSet();
 
             if (!securityRoles.contains(roleName)) {
-                throw new GeronimoSecurityException("Role does not exist in this configuration");
+                throw new GeronimoSecurityException("Role '" + roleName + "' does not exist in this configuration");
             }
 
             Subject roleDesignate = new Subject();