Skip to content

Commit 5a9dbd4

Browse files
adinauerclaude
andcommitted
ref: Add Millis suffix to ExternalOptions timeout fields
Rename shutdownTimeout/sessionFlushTimeout to shutdownTimeoutMillis/sessionFlushTimeoutMillis for consistency with SentryOptions naming convention. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent eddda96 commit 5a9dbd4

File tree

5 files changed

+26
-26
lines changed

5 files changed

+26
-26
lines changed

sentry/api/sentry.api

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -498,8 +498,8 @@ public final class io/sentry/ExternalOptions {
498498
public fun getRelease ()Ljava/lang/String;
499499
public fun getSendClientReports ()Ljava/lang/Boolean;
500500
public fun getServerName ()Ljava/lang/String;
501-
public fun getSessionFlushTimeout ()Ljava/lang/Long;
502-
public fun getShutdownTimeout ()Ljava/lang/Long;
501+
public fun getSessionFlushTimeoutMillis ()Ljava/lang/Long;
502+
public fun getShutdownTimeoutMillis ()Ljava/lang/Long;
503503
public fun getSpotlightConnectionUrl ()Ljava/lang/String;
504504
public fun getTags ()Ljava/util/Map;
505505
public fun getTracePropagationTargets ()Ljava/util/List;
@@ -546,8 +546,8 @@ public final class io/sentry/ExternalOptions {
546546
public fun setSendDefaultPii (Ljava/lang/Boolean;)V
547547
public fun setSendModules (Ljava/lang/Boolean;)V
548548
public fun setServerName (Ljava/lang/String;)V
549-
public fun setSessionFlushTimeout (Ljava/lang/Long;)V
550-
public fun setShutdownTimeout (Ljava/lang/Long;)V
549+
public fun setSessionFlushTimeoutMillis (Ljava/lang/Long;)V
550+
public fun setShutdownTimeoutMillis (Ljava/lang/Long;)V
551551
public fun setSpotlightConnectionUrl (Ljava/lang/String;)V
552552
public fun setTag (Ljava/lang/String;Ljava/lang/String;)V
553553
public fun setTracesSampleRate (Ljava/lang/Double;)V

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

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ public final class ExternalOptions {
3434
private final @NotNull List<String> contextTags = new CopyOnWriteArrayList<>();
3535
private @Nullable String proguardUuid;
3636
private @Nullable Long idleTimeout;
37-
private @Nullable Long shutdownTimeout;
38-
private @Nullable Long sessionFlushTimeout;
37+
private @Nullable Long shutdownTimeoutMillis;
38+
private @Nullable Long sessionFlushTimeoutMillis;
3939
private final @NotNull Set<Class<? extends Throwable>> ignoredExceptionsForType =
4040
new CopyOnWriteArraySet<>();
4141
private @Nullable List<String> ignoredErrors;
@@ -135,8 +135,8 @@ public final class ExternalOptions {
135135
options.addBundleId(bundleId);
136136
}
137137
options.setIdleTimeout(propertiesProvider.getLongProperty("idle-timeout"));
138-
options.setShutdownTimeout(propertiesProvider.getLongProperty("shutdown-timeout"));
139-
options.setSessionFlushTimeout(propertiesProvider.getLongProperty("session-flush-timeout"));
138+
options.setShutdownTimeoutMillis(propertiesProvider.getLongProperty("shutdown-timeout"));
139+
options.setSessionFlushTimeoutMillis(propertiesProvider.getLongProperty("session-flush-timeout"));
140140

141141
options.setIgnoredErrors(propertiesProvider.getListOrNull("ignored-errors"));
142142

@@ -397,20 +397,20 @@ public void setIdleTimeout(final @Nullable Long idleTimeout) {
397397
this.idleTimeout = idleTimeout;
398398
}
399399

400-
public @Nullable Long getShutdownTimeout() {
401-
return shutdownTimeout;
400+
public @Nullable Long getShutdownTimeoutMillis() {
401+
return shutdownTimeoutMillis;
402402
}
403403

404-
public void setShutdownTimeout(final @Nullable Long shutdownTimeout) {
405-
this.shutdownTimeout = shutdownTimeout;
404+
public void setShutdownTimeoutMillis(final @Nullable Long shutdownTimeoutMillis) {
405+
this.shutdownTimeoutMillis = shutdownTimeoutMillis;
406406
}
407407

408-
public @Nullable Long getSessionFlushTimeout() {
409-
return sessionFlushTimeout;
408+
public @Nullable Long getSessionFlushTimeoutMillis() {
409+
return sessionFlushTimeoutMillis;
410410
}
411411

412-
public void setSessionFlushTimeout(final @Nullable Long sessionFlushTimeout) {
413-
this.sessionFlushTimeout = sessionFlushTimeout;
412+
public void setSessionFlushTimeoutMillis(final @Nullable Long sessionFlushTimeoutMillis) {
413+
this.sessionFlushTimeoutMillis = sessionFlushTimeoutMillis;
414414
}
415415

416416
public @Nullable List<String> getIgnoredErrors() {

sentry/src/main/java/io/sentry/SentryOptions.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3307,11 +3307,11 @@ public void merge(final @NotNull ExternalOptions options) {
33073307
if (options.getIdleTimeout() != null) {
33083308
setIdleTimeout(options.getIdleTimeout());
33093309
}
3310-
if (options.getShutdownTimeout() != null) {
3311-
setShutdownTimeoutMillis(options.getShutdownTimeout());
3310+
if (options.getShutdownTimeoutMillis() != null) {
3311+
setShutdownTimeoutMillis(options.getShutdownTimeoutMillis());
33123312
}
3313-
if (options.getSessionFlushTimeout() != null) {
3314-
setSessionFlushTimeoutMillis(options.getSessionFlushTimeout());
3313+
if (options.getSessionFlushTimeoutMillis() != null) {
3314+
setSessionFlushTimeoutMillis(options.getSessionFlushTimeoutMillis());
33153315
}
33163316
for (String bundleId : options.getBundleIds()) {
33173317
addBundleId(bundleId);

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,13 +176,13 @@ class ExternalOptionsTest {
176176
}
177177

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

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

188188
@Test

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -384,8 +384,8 @@ class SentryOptionsTest {
384384
externalOptions.addContextTag("requestId")
385385
externalOptions.proguardUuid = "1234"
386386
externalOptions.idleTimeout = 1500L
387-
externalOptions.shutdownTimeout = 1499L
388-
externalOptions.sessionFlushTimeout = 1498L
387+
externalOptions.shutdownTimeoutMillis = 1499L
388+
externalOptions.sessionFlushTimeoutMillis = 1498L
389389
externalOptions.bundleIds.addAll(
390390
listOf("12ea7a02-46ac-44c0-a5bb-6d1fd9586411 ", " faa3ab42-b1bd-4659-af8e-1682324aa744")
391391
)

0 commit comments

Comments
 (0)