From 7cf412c3943d58b388dde6b89a8b58c4da66aa95 Mon Sep 17 00:00:00 2001
From: neil_a_wilson <neil_a_wilson@localhost>
Date: Tue, 10 Jul 2007 16:03:42 +0000
Subject: [PATCH] Fix a set of problems with the configuration interface in which there were a number of cases in which insufficient validation was performed. In particular, if a new configuration object was added over protocol or an existing configuration object was changed from disabled to enabled, then the server would only perform generic validation for that component and would not have any way to perform more detailed validation that could detect larger numbers of problems.
---
opendj-sdk/opends/src/server/org/opends/server/api/AccessControlHandler.java | 37 +++++++++++++++++++++++++++++++++++++
1 files changed, 37 insertions(+), 0 deletions(-)
diff --git a/opendj-sdk/opends/src/server/org/opends/server/api/AccessControlHandler.java b/opendj-sdk/opends/src/server/org/opends/server/api/AccessControlHandler.java
index 0d8a46a..ef0b4ad 100644
--- a/opendj-sdk/opends/src/server/org/opends/server/api/AccessControlHandler.java
+++ b/opendj-sdk/opends/src/server/org/opends/server/api/AccessControlHandler.java
@@ -27,6 +27,9 @@
package org.opends.server.api;
+
+import java.util.List;
+
import org.opends.server.admin.std.server.AccessControlHandlerCfg;
import org.opends.server.config.ConfigException;
import org.opends.server.core.*;
@@ -72,6 +75,40 @@
/**
+ * Indicates whether the provided configuration is acceptable for
+ * this access control handler. It should be possible to call this
+ * method on an uninitialized access control handler instance in
+ * order to determine whether the handler would be able to use the
+ * provided configuration.
+ * <BR><BR>
+ * Note that implementations which use a subclass of the provided
+ * configuration class will likely need to cast the configuration
+ * to the appropriate subclass type.
+ *
+ * @param configuration The access control handler
+ * configuration for which to make the
+ * determination.
+ * @param unacceptableReasons A list that may be used to hold the
+ * reasons that the provided
+ * configuration is not acceptable.
+ *
+ * @return {@code true} if the provided configuration is acceptable
+ * for this access control handler, or {@code false} if
+ * not.
+ */
+ public boolean isConfigurationAcceptable(
+ AccessControlHandlerCfg configuration,
+ List<String> unacceptableReasons)
+ {
+ // This default implementation does not perform any special
+ // validation. It should be overridden by access control handler
+ // implementations that wish to perform more detailed validation.
+ return true;
+ }
+
+
+
+ /**
* Performs any necessary finalization for the access control
* handler implementation. This will be called just after the
* handler has been deregistered with the server but before it has
--
Gitblit v1.10.0