[jboss-cvs] JBossAS SVN: r67473 - projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/plugins/authorization.
jboss-cvs-commits at lists.jboss.org
jboss-cvs-commits at lists.jboss.org
Mon Nov 26 20:42:02 EST 2007
Author: anil.saldhana at jboss.com
Date: 2007-11-26 20:42:02 -0500 (Mon, 26 Nov 2007)
New Revision: 67473
Modified:
projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/plugins/authorization/JBossAuthorizationContext.java
Log:
SECURITY-96: improve AuthorizationContext error report
Modified: projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/plugins/authorization/JBossAuthorizationContext.java
===================================================================
--- projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/plugins/authorization/JBossAuthorizationContext.java 2007-11-27 01:41:05 UTC (rev 67472)
+++ projects/security/security-jboss-sx/trunk/jbosssx/src/main/org/jboss/security/plugins/authorization/JBossAuthorizationContext.java 2007-11-27 01:42:02 UTC (rev 67473)
@@ -191,7 +191,7 @@
decision = module.authorize(resource);
}
catch(Exception ae)
- {
+ {
decision = DENY;
if(moduleException == null)
moduleException = new AuthorizationException(ae.getMessage());
@@ -229,10 +229,11 @@
}
//All the authorization modules have been visited.
+ String msg = getAdditionalErrorMessage(moduleException);
if(encounteredRequiredError)
- throw new AuthorizationException("Authorization Failed");
+ throw new AuthorizationException("Authorization Failed:"+ msg);
if(overallDecision == DENY && encounteredOptionalError)
- throw new AuthorizationException("Authorization Failed");
+ throw new AuthorizationException("Authorization Failed:" + msg);
if(overallDecision == DENY)
throw new AuthorizationException("Authorization Failed:No modules active.");
return PERMIT;
@@ -271,7 +272,7 @@
ClassLoader tcl = SecurityActions.getContextClassLoader();
try
{
- Class clazz = tcl.loadClass(name);
+ Class<?> clazz = tcl.loadClass(name);
am = (AuthorizationModule)clazz.newInstance();
}
catch ( Exception e)
@@ -335,4 +336,12 @@
}
return ai;
}
+
+ private String getAdditionalErrorMessage(Exception e)
+ {
+ StringBuilder msg = new StringBuilder(" ");
+ if(e != null)
+ msg.append(e.getLocalizedMessage());
+ return msg.toString();
+ }
}
\ No newline at end of file
More information about the jboss-cvs-commits
mailing list