diff options
| author | Martin Ritchie <ritchiem@apache.org> | 2010-06-02 16:47:34 +0000 |
|---|---|---|
| committer | Martin Ritchie <ritchiem@apache.org> | 2010-06-02 16:47:34 +0000 |
| commit | e6697ce5f03df5b2b53ec203c1894d8f11c89f3d (patch) | |
| tree | 7a9653be6111bd0c842d3efc19356acd790a15d9 /java/broker-plugins/access-control/src/main | |
| parent | bb9a83ea69616960d2e56052ba1b782a0db685b0 (diff) | |
| download | qpid-python-e6697ce5f03df5b2b53ec203c1894d8f11c89f3d.tar.gz | |
QPID-2581 : Update Plugins to have a consistent configure(ConfigurationPlugin config) method.
Further work is required to ensure that all ConfigurationPlugins perform config validation rather than leaving that to the plugin.
The plugin should just use the config.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@950656 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'java/broker-plugins/access-control/src/main')
| -rw-r--r-- | java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/plugins/AccessControl.java | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/plugins/AccessControl.java b/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/plugins/AccessControl.java index 75846b26ef..98fc198cc8 100644 --- a/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/plugins/AccessControl.java +++ b/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/plugins/AccessControl.java @@ -60,17 +60,12 @@ public class AccessControl extends AbstractPlugin public AccessControl newInstance(ConfigurationPlugin config) throws ConfigurationException { - AccessControl plugin = new AccessControl(config); - plugin.configure(); + AccessControl plugin = new AccessControl(); + plugin.configure(config); return plugin; } }; - - public AccessControl(ConfigurationPlugin config) - { - _config = config.getConfiguration(AccessControlConfiguration.class); - } - + public Result getDefault() { return _ruleSet.getDefault(); @@ -112,9 +107,9 @@ public class AccessControl extends AbstractPlugin return _ruleSet.check(principal.getName(), operation, objectType, properties); } - @Override - public void configure() throws ConfigurationException + public void configure(ConfigurationPlugin config) throws ConfigurationException { + _config = config.getConfiguration(AccessControlConfiguration.class); AccessControlConfiguration accessConfig = (AccessControlConfiguration) _config; if (isConfigured()) |