Merge pull request #303 from mild-blue/stack-overflow

Wrap exception that produces stack overflow
mild-blue · Jun 10, 2021 · a42942c · a42942c
2 parents 90edca3 + 3a5f984
commit a42942c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/backend/src/main/kotlin/blue/mild/covid/vaxx/service/IsinValidationService.kt b/backend/src/main/kotlin/blue/mild/covid/vaxx/service/IsinValidationService.kt
@@ -49,8 +49,10 @@ class IsinValidationService(
             logger.info { "Data retrieval from ISIN - success." }
         }.onFailure {
             logger.warn { "Data retrieval from ISIN - failure." }
-            // TODO we think that this is the place which produces stack overflow
-            logger.error(it) {
+            // TODO #287 we think that this is the place which produces stack overflow
+            val wrappingException =
+                Exception("An exception ${it.javaClass.canonicalName} was thrown! - ${it.message}\n${it.stackTraceToString()}")
+            logger.error(wrappingException) {
                 "Getting data from ISIN server failed for patient ${firstName}/${lastName}/${personalNumber}"
             }
         }.getOrNull() ?: return IsinValidationResultDto(status = PatientValidationResult.WAS_NOT_VERIFIED)