1.0.7 (#584)

* adjusting log statement (#583) * adjusting log statement * code review * Introduce new version 1.0.7
corona-warn-app · Jun 15, 2020 · 9429244 · 9429244
1 parent c225375
commit 9429244
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/.mvn/maven.config b/.mvn/maven.config
@@ -1,4 +1,4 @@
--Drevision=1.0.6
+-Drevision=1.0.7
 -Dlicense.projectName=Corona-Warn-App
 -Dlicense.inceptionYear=2020
 -Dlicense.licenseName=apache_v2
diff --git a/...ion/src/main/java/app/coronawarn/server/services/submission/verification/TanVerifier.java b/...ion/src/main/java/app/coronawarn/server/services/submission/verification/TanVerifier.java
@@ -57,7 +57,8 @@ public boolean verifyTan(String tanString) {
 
       return verifyWithVerificationService(tan);
     } catch (IllegalArgumentException e) {
-      logger.error("TAN Syntax check failed for TAN: {}", tanString.trim());
+      logger.error("TAN Syntax check failed for TAN: {}, length: {}",
+          tanString.substring(0, Math.min(36, tanString.length())), tanString.length());
       return false;
     }
   }

diff --git a/...src/test/java/app/coronawarn/server/services/submission/verification/TanVerifierTest.java b/...src/test/java/app/coronawarn/server/services/submission/verification/TanVerifierTest.java
@@ -109,6 +109,18 @@ void checkInvalidTan() {
     assertThat(tanVerificationResponse).isFalse();
   }
 
+  @Test
+  void checkTooLongTan() {
+    server.stubFor(
+        post(urlEqualTo(verificationPath))
+            .withHeader(CONTENT_TYPE, equalTo(MediaType.APPLICATION_JSON.toString()))
+            .willReturn(aResponse().withStatus(HttpStatus.NOT_FOUND.value())));
+
+    boolean tanVerificationResponse = tanVerifier.verifyTan(randomUUID + randomUUID);
+
+    assertThat(tanVerificationResponse).isFalse();
+  }
+
   @Test
   void checkInternalServerError() {
     server.stubFor(