Author: sletellier Date: 2010-06-11 12:32:51 +0200 (Fri, 11 Jun 2010) New Revision: 2002 Url: http://nuiton.org/repositories/revision/topia/2002 Log: Realy fork jass checkPermission method Modified: trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/TaasService.java trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/jaas/TaasPolicy.java Modified: trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/TaasService.java =================================================================== --- trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/TaasService.java 2010-06-10 23:26:35 UTC (rev 2001) +++ trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/TaasService.java 2010-06-11 10:32:51 UTC (rev 2002) @@ -341,10 +341,15 @@ // Code that note use realy jaas Set<TaasPrincipalWrapper> ps = subj.getPrincipals(TaasPrincipalWrapper.class); for (TaasPrincipalWrapper p : ps) { - if (!p.getPermissions().implies(myp)) { - throw new SecurityException("Access denied to object " + myp); + if (log.isDebugEnabled()) { + log.debug("Check permissions for principal wrapper : " + p); } + if (p.getPermissions().implies(myp)) { + return; + } } + throw new SecurityException("Access denied to object " + myp); + // Old code that use realy jaas //AccessController.checkPermission(myp); } Modified: trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/jaas/TaasPolicy.java =================================================================== --- trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/jaas/TaasPolicy.java 2010-06-10 23:26:35 UTC (rev 2001) +++ trunk/topia-service-security/src/main/java/org/nuiton/topia/taas/jaas/TaasPolicy.java 2010-06-11 10:32:51 UTC (rev 2002) @@ -51,7 +51,7 @@ import org.nuiton.topia.taas.TaasService; /** - * Implantation d'un policy avec une prise en compte des permissions � la vol�e. + * Implantation d'un policy avec une prise en compte des permissions à la volée. * @author ruchaud */ public class TaasPolicy extends Policy { @@ -118,7 +118,7 @@ } } } else { - log.error("R�cup�ration des Permissions impossible"); + log.error("Récupération des Permissions impossible"); } return pc; } catch (Throwable eee) { @@ -149,8 +149,8 @@ } /** - * Installe cette TopiaPolicy. Si la Policy existante est d�ja cette - * TopiaPolicy alors la m�thode n'a pas d'effet. Si une autre Policy existe + * Installe cette TopiaPolicy. Si la Policy existante est déja cette + * TopiaPolicy alors la méthode n'a pas d'effet. Si une autre Policy existe * deja alors cette TopiaPolicy, elle conserve l'ancienne Policy dans * parentPolicy et la remplace alors. */