Set by default enabled to true in log4j2 appender (#244)

* Set by default enabled to true in log4j2 appender Co-authored-by: Diego Veralli <[email protected]>
rollbar · Jun 9, 2021 · c6d7d43 · c6d7d43
1 parent d04c888
commit c6d7d43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/rollbar-log4j2/src/main/java/com/rollbar/log4j2/RollbarAppender.java b/rollbar-log4j2/src/main/java/com/rollbar/log4j2/RollbarAppender.java
@@ -83,7 +83,7 @@ public static RollbarAppender createAppender(
       @PluginAttribute("endpoint") final String endpoint,
       @PluginAttribute("environment") final String environment,
       @PluginAttribute("language") final String language,
-      @PluginAttribute("enabled") final boolean enabled,
+      @PluginAttribute(value = "enabled", defaultBoolean = true) final boolean enabled,
       @PluginAttribute("configProviderClassName") final String configProviderClassName,
       @PluginAttribute("name") @Required final String name,
       @PluginElement("Layout") Layout<? extends Serializable> layout,

diff --git a/rollbar-log4j2/src/test/java/com/rollbar/log4j2/RollbarAppenderIntegrationTest.java b/rollbar-log4j2/src/test/java/com/rollbar/log4j2/RollbarAppenderIntegrationTest.java
@@ -130,7 +130,7 @@ public static class TestConfigProvider implements ConfigProvider {
 
         @Override
         public Config provide(ConfigBuilder builder) {
-            return builder.accessToken(TEST_TOKEN).enabled(true).sender(SENDER.get()).build();
+            return builder.accessToken(TEST_TOKEN).sender(SENDER.get()).build();
         }
     }
 }