Skip to content

Commit 32d33a3

Browse files
romtsnclaude
andcommitted
fix(log4j2): Move super.start() into start(OptionsConfiguration) overload
Ensures the appender is marked as started regardless of which entry point is used, preventing loggerContext.start() from re-triggering the no-arg start() and reinitializing Sentry without the test executor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 63276e3 commit 32d33a3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

sentry-log4j2/src/main/java/io/sentry/log4j2/SentryAppender.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,6 @@ public SentryAppender(
165165
@Override
166166
public void start() {
167167
start(getOptionsConfiguration(null));
168-
super.start();
169168
}
170169

171170
@NotNull
@@ -203,6 +202,7 @@ void start(final @NotNull Sentry.OptionsConfiguration<SentryOptions> optionsConf
203202
}
204203
}
205204
addPackageAndIntegrationInfo();
205+
super.start();
206206
}
207207

208208
@Override

0 commit comments

Comments
 (0)