From 10bac0ad21fca0c5ac8bf6d7dc712c5899c2d0e6 Mon Sep 17 00:00:00 2001
From: Nicolas Capponi <nicolas.capponi@forgerock.com>
Date: Thu, 06 Feb 2014 13:51:13 +0000
Subject: [PATCH] Fix OPENDJ-1289 Runtime debug configuration changes have no effect
---
opendj3-server-dev/src/server/org/opends/server/loggers/AccessLogger.java | 53 +++++++++++++++++++++++++++--------------------------
1 files changed, 27 insertions(+), 26 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/loggers/AccessLogger.java b/opendj3-server-dev/src/server/org/opends/server/loggers/AccessLogger.java
index 4c08083..c76e11b 100644
--- a/opendj3-server-dev/src/server/org/opends/server/loggers/AccessLogger.java
+++ b/opendj3-server-dev/src/server/org/opends/server/loggers/AccessLogger.java
@@ -52,13 +52,13 @@
loggerStorage = new LoggerStorage
<AccessLogPublisher<AccessLogPublisherCfg>, AccessLogPublisherCfg>();
- /** The singleton instance of this class for configuration purposes. */
+ /** The singleton instance of this class. */
private static final AccessLogger instance = new AccessLogger();
/**
* The constructor for this class.
*/
- public AccessLogger()
+ private AccessLogger()
{
super((Class) AccessLogPublisher.class,
ERR_CONFIG_LOGGER_INVALID_ACCESS_LOGGER_CLASS);
@@ -74,10 +74,9 @@
/** {@inheritDoc} */
@Override
- protected LoggerStorage<AccessLogPublisher<AccessLogPublisherCfg>,
- AccessLogPublisherCfg> getStorage()
+ protected Collection<AccessLogPublisher<AccessLogPublisherCfg>> getLogPublishers()
{
- return loggerStorage;
+ return loggerStorage.getLogPublishers();
}
/**
@@ -90,28 +89,7 @@
return instance;
}
- /**
- * Add an access log publisher to the access logger.
- *
- * @param publisher The access log publisher to add.
- */
- public synchronized static void addAccessLogPublisher(
- AccessLogPublisher publisher)
- {
- loggerStorage.addLogPublisher(publisher);
- }
- /**
- * Remove an access log publisher from the access logger.
- *
- * @param publisher The access log publisher to remove.
- * @return The publisher that was removed or null if it was not found.
- */
- public synchronized static boolean removeAccessLogPublisher(
- AccessLogPublisher<AccessLogPublisherCfg> publisher)
- {
- return loggerStorage.removeLogPublisher(publisher);
- }
/**
* Removes all existing access log publishers from the logger.
@@ -532,5 +510,28 @@
publisher.logUnbind(unbindOperation);
}
}
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void addLogPublisher(
+ AccessLogPublisher<AccessLogPublisherCfg> publisher)
+ {
+ loggerStorage.addLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized boolean removeLogPublisher(
+ AccessLogPublisher<AccessLogPublisherCfg> publisher)
+ {
+ return loggerStorage.removeLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void removeAllLogPublishers()
+ {
+ loggerStorage.removeAllLogPublishers();
+ }
}
--
Gitblit v1.10.0