From cd248ef8a3786827f392222f28860480b7b83b24 Mon Sep 17 00:00:00 2001
From: Matthias Urhahn <matthias.urhahn@sap.com>
Date: Wed, 14 Oct 2020 15:06:36 +0200
Subject: [PATCH] Increase transaction timeout (EXPOSUREAPP-3238) (#1406)
* Increase transaction timeout to 8 minutes.
* Fix timeout variable test.
* Adjust code comment to mention the 10min limit.
Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>
Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>
---
.../src/main/java/de/rki/coronawarnapp/risk/TimeVariables.kt | 4 +++-
.../test/java/de/rki/coronawarnapp/risk/TimeVariablesTest.kt | 2 +-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/TimeVariables.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/TimeVariables.kt
index 99a10696c..dc90f38b4 100644
--- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/TimeVariables.kt
+++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/TimeVariables.kt
@@ -33,8 +33,10 @@ object TimeVariables {
/**
* The maximal runtime of a transaction
* In milliseconds
+ * Stay below 10min with this timeout!
+ * We only 10min background execution time via WorkManager.
*/
- private const val TRANSACTION_TIMEOUT = 180000L
+ private const val TRANSACTION_TIMEOUT = 8 * 60 * 1000L
/**
* Getter function for [TRANSACTION_TIMEOUT]
diff --git a/Corona-Warn-App/src/test/java/de/rki/coronawarnapp/risk/TimeVariablesTest.kt b/Corona-Warn-App/src/test/java/de/rki/coronawarnapp/risk/TimeVariablesTest.kt
index 4055ddf1f..4b4651965 100644
--- a/Corona-Warn-App/src/test/java/de/rki/coronawarnapp/risk/TimeVariablesTest.kt
+++ b/Corona-Warn-App/src/test/java/de/rki/coronawarnapp/risk/TimeVariablesTest.kt
@@ -16,7 +16,7 @@ class TimeVariablesTest {
@Test
fun getTransactionTimeout() {
- Assert.assertEquals(TimeVariables.getTransactionTimeout(), 180000L)
+ Assert.assertEquals(TimeVariables.getTransactionTimeout(), 480000L)
}
@Test
--
GitLab