Rename to Log4jConstants.

Author: phipag

powertools-logging/powertools-logging-log4j/src/main/java/software/amazon/lambda/powertools/logging/log4j/BufferingAppender.java

@@ -14,6 +14,8 @@
 
 package software.amazon.lambda.powertools.logging.log4j;
 
+import static software.amazon.lambda.powertools.logging.log4j.Log4jConstants.BUFFERING_APPENDER_PLUGIN_NAME;
+
 import java.io.Serializable;
 import java.util.Deque;
 
@@ -37,8 +39,6 @@
 import software.amazon.lambda.powertools.logging.internal.BufferManager;
 import software.amazon.lambda.powertools.logging.internal.KeyBuffer;
 
-import static software.amazon.lambda.powertools.logging.log4j.BufferingAppenderConstants.NAME;
-
 /**
  * A Log4j2 appender that buffers log events by AWS X-Ray trace ID for optimized Lambda logging.
  * 
@@ -82,7 +82,7 @@
  * 
  * @see software.amazon.lambda.powertools.logging.PowertoolsLogging#flushLogBuffer()
  */
-@Plugin(name = NAME, category = Core.CATEGORY_NAME, elementType = Appender.ELEMENT_TYPE)
+@Plugin(name = BUFFERING_APPENDER_PLUGIN_NAME, category = Core.CATEGORY_NAME, elementType = Appender.ELEMENT_TYPE)
 public class BufferingAppender extends AbstractAppender implements BufferManager {
 
     private final AppenderRef[] appenderRefs;

powertools-logging/powertools-logging-log4j/src/main/java/software/amazon/lambda/powertools/logging/log4j/BufferingAppenderConstants.java renamed to powertools-logging/powertools-logging-log4j/src/main/java/software/amazon/lambda/powertools/logging/log4j/Log4jConstants.java

@@ -15,16 +15,16 @@
 package software.amazon.lambda.powertools.logging.log4j;
 
 /**
- * Constants for BufferingAppender configuration and references.
+ * Constants for Log4j2 configuration and references.
  */
-public final class BufferingAppenderConstants {
+public final class Log4jConstants {
 
     /**
-     * The name used for BufferingAppender in Log4j2 configuration and references.
+     * The plugin name for BufferingAppender in Log4j2 configuration.
      */
-    public static final String NAME = "BufferingAppender";
+    public static final String BUFFERING_APPENDER_PLUGIN_NAME = "BufferingAppender";
 
-    private BufferingAppenderConstants() {
+    private Log4jConstants() {
         // Utility class
     }
-}
+}

powertools-logging/powertools-logging-log4j/src/main/java/software/amazon/lambda/powertools/logging/log4j/internal/Log4jLoggingManager.java

@@ -14,6 +14,8 @@
 
 package software.amazon.lambda.powertools.logging.log4j.internal;
 
+import static software.amazon.lambda.powertools.logging.log4j.Log4jConstants.BUFFERING_APPENDER_PLUGIN_NAME;
+
 import org.apache.logging.log4j.Level;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.core.LoggerContext;
@@ -24,8 +26,6 @@
 import software.amazon.lambda.powertools.logging.internal.LoggingManager;
 import software.amazon.lambda.powertools.logging.log4j.BufferingAppender;
 
-import static software.amazon.lambda.powertools.logging.log4j.BufferingAppenderConstants.NAME;
-
 /**
  * LoggingManager for Log4j2 that provides log level management and buffer operations.
  * Implements both {@link LoggingManager} and {@link BufferManager} interfaces.
@@ -59,7 +59,7 @@ public org.slf4j.event.Level getLogLevel(Logger logger) {
     public void flushBuffer() {
         LoggerContext ctx = (LoggerContext) LogManager.getContext(false);
         BufferingAppender bufferingAppender = (BufferingAppender) ctx.getConfiguration()
-                .getAppender(NAME);
+                .getAppender(BUFFERING_APPENDER_PLUGIN_NAME);
         if (bufferingAppender != null) {
             bufferingAppender.flushBuffer();
         }
@@ -72,7 +72,7 @@ public void flushBuffer() {
     public void clearBuffer() {
         LoggerContext ctx = (LoggerContext) LogManager.getContext(false);
         BufferingAppender bufferingAppender = (BufferingAppender) ctx.getConfiguration()
-                .getAppender(NAME);
+                .getAppender(BUFFERING_APPENDER_PLUGIN_NAME);
         if (bufferingAppender != null) {
             bufferingAppender.clearBuffer();
         }

powertools-logging/powertools-logging-log4j/src/test/java/software/amazon/lambda/powertools/logging/log4j/BufferingAppenderTest.java

@@ -112,6 +112,6 @@ void shouldClearBufferManually() {
 
     private BufferingAppender getBufferingAppender() {
         return (BufferingAppender) ((org.apache.logging.log4j.core.LoggerContext) LogManager.getContext(false))
-                .getConfiguration().getAppender(BufferingAppenderConstants.NAME);
+                .getConfiguration().getAppender(Log4jConstants.BUFFERING_APPENDER_PLUGIN_NAME);
     }
 }