Move SentryLogs out of experimental (#4710)

stefanosiano · web-flow · commit c8125f3c9015 · 2025-09-10T12:51:18.000Z
* removed all @experimental annotations from SentryLogs code
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@
 
 ### Features
 
+- Move SentryLogs out of experimental ([#4710](https://github.com/getsentry/sentry-java/pull/4710))
 - Add support for w3c traceparent header ([#4671](https://github.com/getsentry/sentry-java/pull/4671))
   - This feature is disabled by default. If enabled, outgoing requests will include the w3c `traceparent` header.
   - See https://develop.sentry.dev/sdk/telemetry/traces/distributed-tracing/#w3c-trace-context-header for more details.
diff --git a/sentry/src/main/java/io/sentry/ExternalOptions.java b/sentry/src/main/java/io/sentry/ExternalOptions.java
@@ -522,12 +522,10 @@ public void setCaptureOpenTelemetryEvents(final @Nullable Boolean captureOpenTel
     return captureOpenTelemetryEvents;
   }
 
-  @ApiStatus.Experimental
   public void setEnableLogs(final @Nullable Boolean enableLogs) {
     this.enableLogs = enableLogs;
   }
 
-  @ApiStatus.Experimental
   public @Nullable Boolean isEnableLogs() {
     return enableLogs;
   }
diff --git a/sentry/src/main/java/io/sentry/HubAdapter.java b/sentry/src/main/java/io/sentry/HubAdapter.java
@@ -384,7 +384,6 @@ public void reportFullyDisplayed() {
     return Sentry.getCurrentScopes().getRateLimiter();
   }
 
-  @ApiStatus.Experimental
   @Override
   public @NotNull ILoggerApi logger() {
     return Sentry.getCurrentScopes().logger();
diff --git a/sentry/src/main/java/io/sentry/HubScopesWrapper.java b/sentry/src/main/java/io/sentry/HubScopesWrapper.java
@@ -370,7 +370,6 @@ public void reportFullyDisplayed() {
     return scopes.captureReplay(replay, hint);
   }
 
-  @ApiStatus.Experimental
   @Override
   public @NotNull ILoggerApi logger() {
     return scopes.logger();
diff --git a/sentry/src/main/java/io/sentry/IScopes.java b/sentry/src/main/java/io/sentry/IScopes.java
@@ -742,7 +742,6 @@ default boolean isNoOp() {
   @NotNull
   SentryId captureReplay(@NotNull SentryReplayEvent replay, @Nullable Hint hint);
 
-  @ApiStatus.Experimental
   @NotNull
   ILoggerApi logger();
 }
diff --git a/sentry/src/main/java/io/sentry/ISentryClient.java b/sentry/src/main/java/io/sentry/ISentryClient.java
@@ -304,7 +304,6 @@ SentryId captureProfileChunk(
   @ApiStatus.Experimental
   SentryId captureCheckIn(@NotNull CheckIn checkIn, @Nullable IScope scope, @Nullable Hint hint);
 
-  @ApiStatus.Experimental
   void captureLog(@NotNull SentryLogEvent logEvent, @Nullable IScope scope);
 
   @ApiStatus.Internal
diff --git a/sentry/src/main/java/io/sentry/NoOpHub.java b/sentry/src/main/java/io/sentry/NoOpHub.java
@@ -327,7 +327,6 @@ public boolean isNoOp() {
     return true;
   }
 
-  @ApiStatus.Experimental
   @Override
   public @NotNull ILoggerApi logger() {
     return NoOpLoggerApi.getInstance();
diff --git a/sentry/src/main/java/io/sentry/NoOpScopes.java b/sentry/src/main/java/io/sentry/NoOpScopes.java
@@ -324,7 +324,6 @@ public boolean isNoOp() {
     return SentryId.EMPTY_ID;
   }
 
-  @ApiStatus.Experimental
   @Override
   public @NotNull ILoggerApi logger() {
     return NoOpLoggerApi.getInstance();
diff --git a/sentry/src/main/java/io/sentry/NoOpSentryClient.java b/sentry/src/main/java/io/sentry/NoOpSentryClient.java
@@ -84,7 +84,6 @@ public SentryId captureEnvelope(@NotNull SentryEnvelope envelope, @Nullable Hint
     return SentryId.EMPTY_ID;
   }
 
-  @ApiStatus.Experimental
   @Override
   public void captureLog(@NotNull SentryLogEvent logEvent, @Nullable IScope scope) {
     // do nothing
diff --git a/sentry/src/main/java/io/sentry/ScopesAdapter.java b/sentry/src/main/java/io/sentry/ScopesAdapter.java
@@ -381,7 +381,6 @@ public void reportFullyDisplayed() {
     return Sentry.getCurrentScopes().captureReplay(replay, hint);
   }
 
-  @ApiStatus.Experimental
   @Override
   public @NotNull ILoggerApi logger() {
     return Sentry.getCurrentScopes().logger();
diff --git a/sentry/src/main/java/io/sentry/Sentry.java b/sentry/src/main/java/io/sentry/Sentry.java
@@ -1291,7 +1291,6 @@ public interface OptionsConfiguration<T extends SentryOptions> {
     return getCurrentScopes().captureCheckIn(checkIn);
   }
 
-  @ApiStatus.Experimental
   @NotNull
   public static ILoggerApi logger() {
     return getCurrentScopes().logger();
diff --git a/sentry/src/main/java/io/sentry/SentryOptions.java b/sentry/src/main/java/io/sentry/SentryOptions.java
@@ -3474,20 +3474,19 @@ public void setDefaultRecoveryThreshold(@Nullable Long defaultRecoveryThreshold)
   public static final class Logs {
 
     /** Whether Sentry Logs feature is enabled and Sentry.logger() usages are sent to Sentry. */
-    @ApiStatus.Experimental private boolean enable = false;
+    private boolean enable = false;
 
     /**
      * This function is called with an SDK specific log event object and can return a modified event
      * object or nothing to skip reporting the log item
      */
-    @ApiStatus.Experimental private @Nullable BeforeSendLogCallback beforeSend;
+    private @Nullable BeforeSendLogCallback beforeSend;
 
     /**
      * Whether Sentry Logs feature is enabled and Sentry.logger() usages are sent to Sentry.
      *
      * @return true if Sentry Logs should be enabled
      */
-    @ApiStatus.Experimental
     public boolean isEnabled() {
       return enable;
     }
@@ -3497,7 +3496,6 @@ public boolean isEnabled() {
      *
      * @param enableLogs true if Sentry Logs should be enabled
      */
-    @ApiStatus.Experimental
     public void setEnabled(boolean enableLogs) {
       this.enable = enableLogs;
     }
@@ -3507,7 +3505,6 @@ public void setEnabled(boolean enableLogs) {
      *
      * @return the beforeSendLog callback or null if not set
      */
-    @ApiStatus.Experimental
     public @Nullable BeforeSendLogCallback getBeforeSend() {
       return beforeSend;
     }
@@ -3517,7 +3514,6 @@ public void setEnabled(boolean enableLogs) {
      *
      * @param beforeSendLog the beforeSendLog callback
      */
-    @ApiStatus.Experimental
     public void setBeforeSend(@Nullable BeforeSendLogCallback beforeSendLog) {
       this.beforeSend = beforeSendLog;
     }
diff --git a/sentry/src/main/java/io/sentry/logger/ILoggerApi.java b/sentry/src/main/java/io/sentry/logger/ILoggerApi.java
@@ -2,11 +2,9 @@
 
 import io.sentry.SentryDate;
 import io.sentry.SentryLogLevel;
-import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
-@ApiStatus.Experimental
 public interface ILoggerApi {
 
   void trace(final @Nullable String message, @Nullable Object... args);
diff --git a/sentry/src/main/java/io/sentry/logger/LoggerApi.java b/sentry/src/main/java/io/sentry/logger/LoggerApi.java
@@ -21,11 +21,9 @@
 import io.sentry.util.Platform;
 import io.sentry.util.TracingUtils;
 import java.util.HashMap;
-import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
-@ApiStatus.Experimental
 public final class LoggerApi implements ILoggerApi {
 
   private final @NotNull Scopes scopes;
diff --git a/sentry/src/main/java/io/sentry/logger/NoOpLoggerApi.java b/sentry/src/main/java/io/sentry/logger/NoOpLoggerApi.java
@@ -2,11 +2,9 @@
 
 import io.sentry.SentryDate;
 import io.sentry.SentryLogLevel;
-import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
-@ApiStatus.Experimental
 public final class NoOpLoggerApi implements ILoggerApi {
 
   private static final NoOpLoggerApi instance = new NoOpLoggerApi();
diff --git a/sentry/src/main/java/io/sentry/logger/NoOpLoggerBatchProcessor.java b/sentry/src/main/java/io/sentry/logger/NoOpLoggerBatchProcessor.java
@@ -1,10 +1,8 @@
 package io.sentry.logger;
 
 import io.sentry.SentryLogEvent;
-import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 
-@ApiStatus.Experimental
 public final class NoOpLoggerBatchProcessor implements ILoggerBatchProcessor {
 
   private static final NoOpLoggerBatchProcessor instance = new NoOpLoggerBatchProcessor();

Original file line number	Diff line number	Diff line change
`@@ -522,12 +522,10 @@ public void setCaptureOpenTelemetryEvents(final @Nullable Boolean captureOpenTel`
`522`	`522`	`return captureOpenTelemetryEvents;`
`523`	`523`	`}`
`524`	`524`
`525`		`- @ApiStatus.Experimental`
`526`	`525`	`public void setEnableLogs(final @Nullable Boolean enableLogs) {`
`527`	`526`	`this.enableLogs = enableLogs;`
`528`	`527`	`}`
`529`	`528`
`530`		`- @ApiStatus.Experimental`
`531`	`529`	`public @Nullable Boolean isEnableLogs() {`
`532`	`530`	`return enableLogs;`
`533`	`531`	`}`
Original file line number	Diff line number	Diff line change
`@@ -384,7 +384,6 @@ public void reportFullyDisplayed() {`
`384`	`384`	`return Sentry.getCurrentScopes().getRateLimiter();`
`385`	`385`	`}`
`386`	`386`
`387`		`- @ApiStatus.Experimental`
`388`	`387`	`@Override`
`389`	`388`	`public @NotNull ILoggerApi logger() {`
`390`	`389`	`return Sentry.getCurrentScopes().logger();`
Original file line number	Diff line number	Diff line change
`@@ -370,7 +370,6 @@ public void reportFullyDisplayed() {`
`370`	`370`	`return scopes.captureReplay(replay, hint);`
`371`	`371`	`}`
`372`	`372`
`373`		`- @ApiStatus.Experimental`
`374`	`373`	`@Override`
`375`	`374`	`public @NotNull ILoggerApi logger() {`
`376`	`375`	`return scopes.logger();`
Original file line number	Diff line number	Diff line change
`@@ -742,7 +742,6 @@ default boolean isNoOp() {`
`742`	`742`	`@NotNull`
`743`	`743`	`SentryId captureReplay(@NotNull SentryReplayEvent replay, @Nullable Hint hint);`
`744`	`744`
`745`		`- @ApiStatus.Experimental`
`746`	`745`	`@NotNull`
`747`	`746`	`ILoggerApi logger();`
`748`	`747`	`}`
Original file line number	Diff line number	Diff line change
`@@ -327,7 +327,6 @@ public boolean isNoOp() {`
`327`	`327`	`return true;`
`328`	`328`	`}`
`329`	`329`
`330`		`- @ApiStatus.Experimental`
`331`	`330`	`@Override`
`332`	`331`	`public @NotNull ILoggerApi logger() {`
`333`	`332`	`return NoOpLoggerApi.getInstance();`
Original file line number	Diff line number	Diff line change
`@@ -324,7 +324,6 @@ public boolean isNoOp() {`
`324`	`324`	`return SentryId.EMPTY_ID;`
`325`	`325`	`}`
`326`	`326`
`327`		`- @ApiStatus.Experimental`
`328`	`327`	`@Override`
`329`	`328`	`public @NotNull ILoggerApi logger() {`
`330`	`329`	`return NoOpLoggerApi.getInstance();`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,6 @@ public SentryId captureEnvelope(@NotNull SentryEnvelope envelope, @Nullable Hint`
`84`	`84`	`return SentryId.EMPTY_ID;`
`85`	`85`	`}`
`86`	`86`
`87`		`- @ApiStatus.Experimental`
`88`	`87`	`@Override`
`89`	`88`	`public void captureLog(@NotNull SentryLogEvent logEvent, @Nullable IScope scope) {`
`90`	`89`	`// do nothing`
Original file line number	Diff line number	Diff line change
`@@ -381,7 +381,6 @@ public void reportFullyDisplayed() {`
`381`	`381`	`return Sentry.getCurrentScopes().captureReplay(replay, hint);`
`382`	`382`	`}`
`383`	`383`
`384`		`- @ApiStatus.Experimental`
`385`	`384`	`@Override`
`386`	`385`	`public @NotNull ILoggerApi logger() {`
`387`	`386`	`return Sentry.getCurrentScopes().logger();`