mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Jean-Noel Rouvignac
03.05.2013 4d1367d39d32236158f4dccf9409c18fb5fd1fe2
OPENDJ-879 Add HTTP access log


Warn about non existing HTTP access logger when the HTTP connection handler is starting (was when loggers are initialized).


LoggerConfigManager.java, HTTPConnectionHandler.java:
Moved the warning about non existing HTTP access logger from LoggerConfigManager.initializeLoggerConfig() to HTTPConnectionHandler.startServer().

HTTPAccessLogger.java:
Made getHTTPAccessLogPublishers() public.
3 files modified
13 ■■■■■ changed files
opends/src/server/org/opends/server/core/LoggerConfigManager.java 4 ●●●● patch | view | raw | blame | history
opends/src/server/org/opends/server/loggers/HTTPAccessLogger.java 2 ●●● patch | view | raw | blame | history
opends/src/server/org/opends/server/protocols/http/HTTPConnectionHandler.java 7 ●●●●● patch | view | raw | blame | history
opends/src/server/org/opends/server/core/LoggerConfigManager.java
@@ -135,10 +135,6 @@
    {
      logError(WARN_CONFIG_LOGGER_NO_ACTIVE_ACCESS_LOGGERS.get());
    }
    if (httpAccessPublisherCfgs.isEmpty())
    {
      logError(WARN_CONFIG_LOGGER_NO_ACTIVE_HTTP_ACCESS_LOGGERS.get());
    }
    if (errorPublisherCfgs.isEmpty())
    {
      logError(WARN_CONFIG_LOGGER_NO_ACTIVE_ERROR_LOGGERS.get());
opends/src/server/org/opends/server/loggers/HTTPAccessLogger.java
@@ -125,7 +125,7 @@
   *
   * @return a Collection of {@link HTTPAccessLogPublisher} objects
   */
  private static Collection<HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg>>
  public static Collection<HTTPAccessLogPublisher<HTTPAccessLogPublisherCfg>>
      getHTTPAccessLogPublishers()
  {
    return loggerStorage.getLogPublishers();
opends/src/server/org/opends/server/protocols/http/HTTPConnectionHandler.java
@@ -26,6 +26,7 @@
 */
package org.opends.server.protocols.http;
import static org.opends.messages.ConfigMessages.*;
import static org.opends.messages.ProtocolMessages.*;
import static org.opends.server.loggers.ErrorLogger.*;
import static org.opends.server.loggers.debug.DebugLogger.*;
@@ -91,6 +92,7 @@
import org.opends.server.core.DirectoryServer;
import org.opends.server.extensions.NullKeyManagerProvider;
import org.opends.server.extensions.NullTrustManagerProvider;
import org.opends.server.loggers.HTTPAccessLogger;
import org.opends.server.loggers.debug.DebugTracer;
import org.opends.server.types.ConfigChangeResult;
import org.opends.server.types.DN;
@@ -700,6 +702,11 @@
    // silence Grizzly's own logging
    Logger.getLogger("org.glassfish.grizzly").setLevel(Level.OFF);
    if (HTTPAccessLogger.getHTTPAccessLogPublishers().isEmpty())
    {
      logError(WARN_CONFIG_LOGGER_NO_ACTIVE_HTTP_ACCESS_LOGGERS.get());
    }
    this.httpServer =
        HttpServer.createSimpleServer("./", initConfig.getListenPort());