Skip to content

Commit 2f10f1c

Browse files
committed
Do not log if sentry.properties in rundir has not been found
1 parent abfcc92 commit 2f10f1c

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

sentry/src/main/java/io/sentry/config/FilesystemPropertiesLoader.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,16 @@
1414
final class FilesystemPropertiesLoader implements PropertiesLoader {
1515
private final @NotNull String filePath;
1616
private final @NotNull ILogger logger;
17+
private boolean logNonExisting;
1718

1819
public FilesystemPropertiesLoader(@NotNull String filePath, @NotNull ILogger logger) {
20+
this(filePath, logger, true);
21+
}
22+
23+
public FilesystemPropertiesLoader(@NotNull String filePath, @NotNull ILogger logger, boolean logNonExisting) {
1924
this.filePath = filePath;
2025
this.logger = logger;
26+
this.logNonExisting = logNonExisting;
2127
}
2228

2329
@Override
@@ -31,10 +37,12 @@ public FilesystemPropertiesLoader(@NotNull String filePath, @NotNull ILogger log
3137
return properties;
3238
}
3339
} else if (!f.isFile()) {
34-
logger.log(
40+
if (logNonExisting) {
41+
logger.log(
3542
SentryLevel.ERROR,
3643
"Failed to load Sentry configuration since it is not a file or does not exist: %s",
3744
filePath);
45+
}
3846
} else if (!f.canRead()) {
3947
logger.log(
4048
SentryLevel.ERROR,

sentry/src/main/java/io/sentry/config/PropertiesProviderFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ public final class PropertiesProviderFactory {
5858
}
5959

6060
final Properties runDirectoryProperties =
61-
new FilesystemPropertiesLoader("sentry.properties", logger).load();
61+
new FilesystemPropertiesLoader("sentry.properties", logger, false).load();
6262
if (runDirectoryProperties != null) {
6363
providers.add(new SimplePropertiesProvider(runDirectoryProperties));
6464
}

0 commit comments

Comments
 (0)