diff --git a/CHANGELOG.md b/CHANGELOG.md index bb788c7866..f2e64006f3 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,11 @@ # Changelog +## Unreleased + +### Fixes + +- Do not log if `sentry.properties` in rundir has not been found ([#4929](https://github.com/getsentry/sentry-java/pull/4929)) + ## 8.27.0 ### Features diff --git a/sentry/src/main/java/io/sentry/config/FilesystemPropertiesLoader.java b/sentry/src/main/java/io/sentry/config/FilesystemPropertiesLoader.java index dcb1ea9100..c896091f00 100644 --- a/sentry/src/main/java/io/sentry/config/FilesystemPropertiesLoader.java +++ b/sentry/src/main/java/io/sentry/config/FilesystemPropertiesLoader.java @@ -14,10 +14,17 @@ final class FilesystemPropertiesLoader implements PropertiesLoader { private final @NotNull String filePath; private final @NotNull ILogger logger; + private boolean logNonExisting; public FilesystemPropertiesLoader(@NotNull String filePath, @NotNull ILogger logger) { + this(filePath, logger, true); + } + + public FilesystemPropertiesLoader( + @NotNull String filePath, @NotNull ILogger logger, boolean logNonExisting) { this.filePath = filePath; this.logger = logger; + this.logNonExisting = logNonExisting; } @Override @@ -31,10 +38,12 @@ public FilesystemPropertiesLoader(@NotNull String filePath, @NotNull ILogger log return properties; } } else if (!f.isFile()) { - logger.log( - SentryLevel.ERROR, - "Failed to load Sentry configuration since it is not a file or does not exist: %s", - filePath); + if (logNonExisting) { + logger.log( + SentryLevel.ERROR, + "Failed to load Sentry configuration since it is not a file or does not exist: %s", + filePath); + } } else if (!f.canRead()) { logger.log( SentryLevel.ERROR, diff --git a/sentry/src/main/java/io/sentry/config/PropertiesProviderFactory.java b/sentry/src/main/java/io/sentry/config/PropertiesProviderFactory.java index 7c9f50facb..f015522f9e 100644 --- a/sentry/src/main/java/io/sentry/config/PropertiesProviderFactory.java +++ b/sentry/src/main/java/io/sentry/config/PropertiesProviderFactory.java @@ -58,7 +58,7 @@ public final class PropertiesProviderFactory { } final Properties runDirectoryProperties = - new FilesystemPropertiesLoader("sentry.properties", logger).load(); + new FilesystemPropertiesLoader("sentry.properties", logger, false).load(); if (runDirectoryProperties != null) { providers.add(new SimplePropertiesProvider(runDirectoryProperties)); }