ref(log4j2)!: Remove deprecated appender constructor

Remove the constructor superseded by the minimumLevel overload in 8.16.0
at the v9 breaking-change boundary.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: adinauer

CHANGELOG.md

@@ -4,9 +4,10 @@
 
 ### Breaking Changes
 
-- Add a `logsEnabled` argument to the preferred Log4j2 `SentryAppender` constructor and
-  `createAppender` factory. Pass `true` to opt in to Sentry Logs capture or `false` to keep it
-  disabled. ([#5941](https://github.com/getsentry/sentry-java/pull/5941))
+- Remove the deprecated Log4j2 `SentryAppender` constructor without `minimumLevel`, and add a
+  `logsEnabled` argument to the remaining constructor and `createAppender` factory. Pass `true` to
+  opt in to Sentry Logs capture or `false` to keep it disabled.
+  ([#5941](https://github.com/getsentry/sentry-java/pull/5941))
 - Require a `logsEnabled` argument when manually constructing `SentryTimberIntegration` or
   `SentryTimberTree`. Replace `SentryTimberIntegration()` with
   `SentryTimberIntegration(logsEnabled = false)` in Kotlin or `new SentryTimberIntegration(false)`

sentry-log4j2/api/sentry-log4j2.api

@@ -5,7 +5,6 @@ public final class io/sentry/log4j2/BuildConfig {
 
 public class io/sentry/log4j2/SentryAppender : org/apache/logging/log4j/core/appender/AbstractAppender {
 	public static final field MECHANISM_TYPE Ljava/lang/String;
-	public fun <init> (Ljava/lang/String;Lorg/apache/logging/log4j/core/Filter;Ljava/lang/String;Lorg/apache/logging/log4j/Level;Lorg/apache/logging/log4j/Level;Ljava/lang/Boolean;Lio/sentry/ITransportFactory;Lio/sentry/IScopes;[Ljava/lang/String;)V
 	public fun <init> (Ljava/lang/String;Lorg/apache/logging/log4j/core/Filter;Ljava/lang/String;Lorg/apache/logging/log4j/Level;Lorg/apache/logging/log4j/Level;Lorg/apache/logging/log4j/Level;ZLjava/lang/Boolean;Lio/sentry/ITransportFactory;Lio/sentry/IScopes;[Ljava/lang/String;)V
 	public fun append (Lorg/apache/logging/log4j/core/LogEvent;)V
 	protected fun captureLog (Lorg/apache/logging/log4j/core/LogEvent;)V

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

@@ -65,37 +65,6 @@ public class SentryAppender extends AbstractAppender {
         .addPackage("maven:io.sentry:sentry-log4j2", BuildConfig.VERSION_NAME);
   }
 
-  /**
-   * @deprecated This constructor is deprecated. Please use {@link #SentryAppender(String, Filter,
-   *     String, Level, Level, Level, boolean, Boolean, ITransportFactory, IScopes, String[])}
-   *     instead.
-   */
-  @Deprecated
-  @SuppressWarnings("InlineMeSuggester")
-  public SentryAppender(
-      final @NotNull String name,
-      final @Nullable Filter filter,
-      final @Nullable String dsn,
-      final @Nullable Level minimumBreadcrumbLevel,
-      final @Nullable Level minimumEventLevel,
-      final @Nullable Boolean debug,
-      final @Nullable ITransportFactory transportFactory,
-      final @NotNull IScopes scopes,
-      final @Nullable String[] contextTags) {
-    this(
-        name,
-        filter,
-        dsn,
-        minimumBreadcrumbLevel,
-        minimumEventLevel,
-        null,
-        false,
-        debug,
-        transportFactory,
-        scopes,
-        contextTags);
-  }
-
   public SentryAppender(
       final @NotNull String name,
       final @Nullable Filter filter,

sentry-log4j2/src/test/kotlin/io/sentry/log4j2/SentryAppenderTest.kt

@@ -310,26 +310,14 @@ class SentryAppenderTest {
   }
 
   @Test
-  fun `constructors can opt out of logs capture`() {
+  fun `constructor can opt out of logs capture`() {
     val scopes = mock<IScopes>()
     val event = mock<LogEvent>()
     whenever(event.level).thenReturn(Level.INFO)
 
-    val deprecatedAppender =
-      SentryAppender(
-        "deprecated",
-        null,
-        null,
-        Level.OFF,
-        Level.OFF,
-        null,
-        null,
-        scopes,
-        null,
-      )
-    val existingAppender =
+    val appender =
       SentryAppender(
-        "existing",
+        "sentry",
         null,
         null,
         Level.OFF,
@@ -342,8 +330,7 @@ class SentryAppenderTest {
         null,
       )
 
-    deprecatedAppender.append(event)
-    existingAppender.append(event)
+    appender.append(event)
 
     verify(scopes, never()).logger()
   }