Merge pull request #5763 from govuk-one-login/AUT-3895/add-timeout-fo…

…r-ipv-access-token-call AUT-3895: Correct and improve IPV token call.
govuk-one-login · Jan 16, 2025 · 88e58e3 · 88e58e3
2 parents 573f7b5 + f9d5137
commit 88e58e3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/ci/terraform/oidc/reverification-result.tf b/ci/terraform/oidc/reverification-result.tf
@@ -30,7 +30,7 @@ module "reverification_result" {
     INTERNAl_SECTOR_URI                           = var.internal_sector_uri
     REDIS_KEY                                     = local.redis_key
     IPV_AUDIENCE                                  = var.ipv_audience
-    IPV_AUTHORISATION_CALLBACK_URI                = var.ipv_authorisation_callback_uri
+    IPV_AUTHORISATION_CALLBACK_URI                = var.ipv_auth_authorize_callback_uri
     IPV_AUTHORISATION_CLIENT_ID                   = var.ipv_authorisation_client_id
     ENVIRONMENT                                   = var.environment
     IPV_REVERIFICATION_REQUESTS_SIGNING_KEY_ALIAS = aws_kms_alias.ipv_reverification_request_signing_key_alias.arn

diff --git a/.../main/java/uk/gov/di/authentication/frontendapi/services/ReverificationResultService.java b/.../main/java/uk/gov/di/authentication/frontendapi/services/ReverificationResultService.java
@@ -103,6 +103,12 @@ public HTTPResponse toHTTPResponse() {
             count++;
             try {
                 var httpRequest = tokenRequest.toHTTPRequest();
+
+                LOG.info("Sending IPV token request to {}", httpRequest.getURI());
+
+                httpRequest.setConnectTimeout(1000);
+                httpRequest.setReadTimeout(60 * 1000);
+
                 var httpResponse = httpRequest.send();
 
                 tokenResponse = TokenResponse.parse(httpResponse);