Merge branch 'main' into fix/4400-add-preview-view-to-default-masked-view-classes

Author: szijpeter

CHANGELOG.md

@@ -11,6 +11,8 @@
 - When merging tombstones with Native SDK, use the tombstone message if the Native SDK didn't explicitly provide one. ([#5095](https://github.com/getsentry/sentry-java/pull/5095))
 - Fix thread leak caused by eager creation of `SentryExecutorService` in `SentryOptions` ([#5093](https://github.com/getsentry/sentry-java/pull/5093))
   - There were cases where we created options that ended up unused but we failed to clean those up.
+- No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN ([#5075](https://github.com/getsentry/sentry-java/pull/5075))
+  - While this may have been useful to some, it caused lots of confusion.
 - Session Replay: Add `androidx.camera.view.PreviewView` to default `maskedViewClasses` to mask camera previews by default. ([#5097](https://github.com/getsentry/sentry-java/pull/5097))
 
 ### Dependencies

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

@@ -184,7 +184,10 @@ public void start() {
             Optional.ofNullable(transportFactory).ifPresent(options::setTransportFactory);
           });
     } catch (IllegalArgumentException e) {
-      LOGGER.warn("Failed to init Sentry during appender initialization: " + e.getMessage());
+      final @Nullable String errorMessage = e.getMessage();
+      if (errorMessage == null || !errorMessage.startsWith("DSN is required.")) {
+        LOGGER.warn("Failed to init Sentry during appender initialization: " + errorMessage);
+      }
     }
     addPackageAndIntegrationInfo();
     super.start();

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

@@ -71,7 +71,10 @@ public void start() {
       try {
         Sentry.init(options);
       } catch (IllegalArgumentException e) {
-        addWarn("Failed to init Sentry during appender initialization: " + e.getMessage());
+        final @Nullable String errorMessage = e.getMessage();
+        if (errorMessage == null || !errorMessage.startsWith("DSN is required.")) {
+          addWarn("Failed to init Sentry during appender initialization: " + errorMessage);
+        }
       }
     } else if (!Sentry.isEnabled()) {
       options