From e6b261cec2f77bd8f590dcbda1793864689e823a Mon Sep 17 00:00:00 2001 From: Greg Wilkins Date: Tue, 29 Dec 2009 22:51:49 +0000 Subject: [PATCH] javadoc and setRoles git-svn-id: svn+ssh://dev.eclipse.org/svnroot/rt/org.eclipse.jetty/jetty/trunk@1170 7e9141cc-0065-0410-87d8-b60c137991c4 --- .../security/ConstraintSecurityHandler.java | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java b/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java index d495e0146fe..c8daed4c992 100644 --- a/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java +++ b/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java @@ -64,6 +64,8 @@ public class ConstraintSecurityHandler extends SecurityHandler implements Constr * * * @param strict the strict to set + * @see #setRoles(Set) + * @see #setConstraintMappings(ConstraintMapping[], Set) */ public void setStrict(boolean strict) { @@ -128,9 +130,23 @@ public class ConstraintSecurityHandler extends SecurityHandler implements Constr } } } + setRoles(roles); + } + + /* ------------------------------------------------------------ */ + /** + * Set the known roles. + * This may be overridden by a subsequent call to {@link #setConstraintMappings(ConstraintMapping[])} or + * {@link #setConstraintMappings(ConstraintMapping[], Set)}. + * @see #setStrict(boolean) + * @param roles The known roles (or null to determine them from the mappings) + */ + public void setRoles(Set roles) + { + if (isStarted()) + throw new IllegalStateException("Started"); this._roles = roles; - } /* ------------------------------------------------------------ */