Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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.

### Dependencies

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
Loading