You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by ma...@apache.org on 2017/01/15 21:42:05 UTC

[2/2] logging-log4j-boot git commit: Add fake ConfigurationFactory

Add fake ConfigurationFactory

Inspired by what Spring Boot does to hide an unnecessary error message from log4j when it can't find a default configuration file (which will be provided later on by this module anyways).


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/commit/3643f4b9
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/tree/3643f4b9
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/diff/3643f4b9

Branch: refs/heads/master
Commit: 3643f4b9ebf02c38abc3a58bee78fb25e3b38351
Parents: f5316ce
Author: Matt Sicker <bo...@gmail.com>
Authored: Sun Jan 15 15:41:57 2017 -0600
Committer: Matt Sicker <bo...@gmail.com>
Committed: Sun Jan 15 15:41:57 2017 -0600

----------------------------------------------------------------------
 .../spring/PrebootConfigurationFactory.java     | 52 ++++++++++++++++++++
 spring/src/main/resources/log4j2.preboot        |  4 ++
 2 files changed, 56 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/blob/3643f4b9/spring/src/main/java/org/apache/logging/log4j/boot/spring/PrebootConfigurationFactory.java
----------------------------------------------------------------------
diff --git a/spring/src/main/java/org/apache/logging/log4j/boot/spring/PrebootConfigurationFactory.java b/spring/src/main/java/org/apache/logging/log4j/boot/spring/PrebootConfigurationFactory.java
new file mode 100644
index 0000000..018cbbb
--- /dev/null
+++ b/spring/src/main/java/org/apache/logging/log4j/boot/spring/PrebootConfigurationFactory.java
@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.logging.log4j.boot.spring;
+
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.config.Configuration;
+import org.apache.logging.log4j.core.config.ConfigurationFactory;
+import org.apache.logging.log4j.core.config.ConfigurationSource;
+import org.apache.logging.log4j.core.config.DefaultConfiguration;
+import org.apache.logging.log4j.core.config.plugins.Plugin;
+import org.springframework.boot.logging.LoggingSystem;
+
+/**
+ * Fake ConfigurationFactory to prevent startup error log messages related to not finding a log4j configuration file.
+ * This log message is misleading because a default configuration file will be provided by this module.
+ */
+@Plugin(name = "PrebootConfigurationFactory", category = ConfigurationFactory.CATEGORY)
+public class PrebootConfigurationFactory extends ConfigurationFactory {
+
+    private static final String[] SUPPORTED_TYPES = {".preboot"};
+
+    @Override
+    protected String[] getSupportedTypes() {
+        return SUPPORTED_TYPES;
+    }
+
+    @Override
+    public Configuration getConfiguration(LoggerContext loggerContext, ConfigurationSource source) {
+        if (source != null && source != ConfigurationSource.NULL_SOURCE) {
+            if (LoggingSystem.get(loggerContext.getClass().getClassLoader()) != null) {
+                return new DefaultConfiguration();
+            }
+        }
+        return null;
+    }
+}

http://git-wip-us.apache.org/repos/asf/logging-log4j-boot/blob/3643f4b9/spring/src/main/resources/log4j2.preboot
----------------------------------------------------------------------
diff --git a/spring/src/main/resources/log4j2.preboot b/spring/src/main/resources/log4j2.preboot
new file mode 100644
index 0000000..6cfb7f3
--- /dev/null
+++ b/spring/src/main/resources/log4j2.preboot
@@ -0,0 +1,4 @@
+Dummy file to prevent log message:
+ERROR StatusLogger No log4j2 configuration file found. Using default configuration: logging only errors to the console.
+
+See PrebootConfigurationFactory.java
\ No newline at end of file