Skip to content

Commit 72c2c53

Browse files
adinauerclaude
andcommitted
ref: Add Millis suffix to ExternalOptions timeout fields
Rename shutdownTimeout/sessionFlushTimeout to shutdownTimeoutMillis/sessionFlushTimeoutMillis in ExternalOptions for consistency with SentryOptions naming convention. Also rename the sentry.properties keys from shutdown-timeout to shutdown-timeout-millis and session-flush-timeout to session-flush-timeout-millis. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 5a9dbd4 commit 72c2c53

File tree

4 files changed

+8
-7
lines changed

4 files changed

+8
-7
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
### Features
66

7-
- Allow configuring `shutdown-timeout` and `session-flush-timeout` via `sentry.properties` and environment variables (`SENTRY_SHUTDOWN_TIMEOUT`, `SENTRY_SESSION_FLUSH_TIMEOUT`) ([#4641](https://github.com/getsentry/sentry-java/pull/4641))
7+
- Allow configuring `shutdown-timeout-millis` and `session-flush-timeout-millis` via `sentry.properties` and environment variables (`SENTRY_SHUTDOWN_TIMEOUT_MILLIS`, `SENTRY_SESSION_FLUSH_TIMEOUT_MILLIS`) ([#4641](https://github.com/getsentry/sentry-java/pull/4641))
88
- Add experimental Sentry Android Distribution module for integrating with Sentry Build Distribution to check for and install updates ([#4804](https://github.com/getsentry/sentry-java/pull/4804))
99

1010
### Fixes
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
release=release from sentry.properties
22
logs.enabled=true
3-
shutdown-timeout=0
4-
session-flush-timeout=0
3+
shutdown-timeout-millis=0
4+
session-flush-timeout-millis=0

sentry/src/main/java/io/sentry/ExternalOptions.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,8 +135,9 @@ public final class ExternalOptions {
135135
options.addBundleId(bundleId);
136136
}
137137
options.setIdleTimeout(propertiesProvider.getLongProperty("idle-timeout"));
138-
options.setShutdownTimeoutMillis(propertiesProvider.getLongProperty("shutdown-timeout"));
139-
options.setSessionFlushTimeoutMillis(propertiesProvider.getLongProperty("session-flush-timeout"));
138+
options.setShutdownTimeoutMillis(propertiesProvider.getLongProperty("shutdown-timeout-millis"));
139+
options.setSessionFlushTimeoutMillis(
140+
propertiesProvider.getLongProperty("session-flush-timeout-millis"));
140141

141142
options.setIgnoredErrors(propertiesProvider.getListOrNull("ignored-errors"));
142143

sentry/src/test/java/io/sentry/ExternalOptionsTest.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,12 +177,12 @@ class ExternalOptionsTest {
177177

178178
@Test
179179
fun `creates options with shutdownTimeoutMillis using external properties`() {
180-
withPropertiesFile("shutdown-timeout=2000") { assertEquals(2000L, it.shutdownTimeoutMillis) }
180+
withPropertiesFile("shutdown-timeout-millis=2000") { assertEquals(2000L, it.shutdownTimeoutMillis) }
181181
}
182182

183183
@Test
184184
fun `creates options with sessionFlushTimeoutMillis using external properties`() {
185-
withPropertiesFile("session-flush-timeout=2000") { assertEquals(2000L, it.sessionFlushTimeoutMillis) }
185+
withPropertiesFile("session-flush-timeout-millis=2000") { assertEquals(2000L, it.sessionFlushTimeoutMillis) }
186186
}
187187

188188
@Test

0 commit comments

Comments
 (0)