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/HTTPAccessLogger.java | 65 +++++++++++++-------------------
1 files changed, 27 insertions(+), 38 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/loggers/HTTPAccessLogger.java b/opendj3-server-dev/src/server/org/opends/server/loggers/HTTPAccessLogger.java
index 2d23227..655f126 100644
--- a/opendj3-server-dev/src/server/org/opends/server/loggers/HTTPAccessLogger.java
+++ b/opendj3-server-dev/src/server/org/opends/server/loggers/HTTPAccessLogger.java
@@ -21,7 +21,7 @@
* CDDL HEADER END
*
*
- * Copyright 2013 ForgeRock AS
+ * Copyright 2013-2014 ForgeRock AS
*/
package org.opends.server.loggers;
@@ -54,7 +54,7 @@
/**
* The constructor for this class.
*/
- public HTTPAccessLogger()
+ private HTTPAccessLogger()
{
super((Class) HTTPAccessLogPublisher.class,
ERR_CONFIG_LOGGER_INVALID_HTTP_ACCESS_LOGGER_CLASS);
@@ -70,10 +70,9 @@
/** {@inheritDoc} */
@Override
- protected LoggerStorage<HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg>,
- HTTPAccessLogPublisherCfg> getStorage()
+ protected Collection<HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg>> getLogPublishers()
{
- return loggerStorage;
+ return loggerStorage.getLogPublishers();
}
/**
@@ -87,39 +86,6 @@
}
/**
- * Add an HTTP access log publisher to the HTTP access logger.
- *
- * @param publisher
- * The HTTP access log publisher to add.
- */
- public synchronized static void addHTTPAccessLogPublisher(
- HTTPAccessLogPublisher publisher)
- {
- loggerStorage.addLogPublisher(publisher);
- }
-
- /**
- * Remove an HTTP access log publisher from the HTTP access logger.
- *
- * @param publisher
- * The HTTP access log publisher to remove.
- * @return The publisher that was removed or null if it was not found.
- */
- public synchronized static boolean removeHTTPAccessLogPublisher(
- HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg> publisher)
- {
- return loggerStorage.removeLogPublisher(publisher);
- }
-
- /**
- * Removes all existing HTTP access log publishers from the logger.
- */
- public synchronized static void removeAllHTTPAccessLogPublishers()
- {
- loggerStorage.removeAllLogPublishers();
- }
-
- /**
* Returns all the registered HTTP access log publishers.
*
* @return a Collection of {@link HTTPAccessLogPublisher} objects
@@ -144,4 +110,27 @@
}
}
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void addLogPublisher(
+ HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg> publisher)
+ {
+ loggerStorage.addLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized boolean removeLogPublisher(
+ HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg> publisher)
+ {
+ return loggerStorage.removeLogPublisher(publisher);
+ }
+
+ /** {@inheritDoc} */
+ @Override
+ public final synchronized void removeAllLogPublishers()
+ {
+ loggerStorage.removeAllLogPublishers();
+ }
+
}
--
Gitblit v1.10.0