From 2f89e46e31661a688b9372a7586f8d64afcf0df4 Mon Sep 17 00:00:00 2001
From: Thomas <>
Date: Tue, 30 Jan 2024 07:18:10 +0100
Subject: chore(core): add some more logging improve session-linking between
eIDAS-Node and MS-specific parts
---
.../modules/auth/eidas/v2/tasks/GenerateAuthnRequestTask.java | 4 +++-
.../auth/eidas/v2/tasks/ReceiveAuthnResponseAlternativeTask.java | 2 +-
.../modules/auth/eidas/v2/tasks/ReceiveAuthnResponseTask.java | 6 +++---
3 files changed, 7 insertions(+), 5 deletions(-)
(limited to 'modules/authmodule-eIDAS-v2/src/main/java')
diff --git a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/GenerateAuthnRequestTask.java b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/GenerateAuthnRequestTask.java
index f2318386..607d3de5 100644
--- a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/GenerateAuthnRequestTask.java
+++ b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/GenerateAuthnRequestTask.java
@@ -37,8 +37,8 @@ import org.springframework.context.ApplicationContext;
import org.springframework.stereotype.Component;
import org.springframework.web.util.UriComponentsBuilder;
-import at.asitplus.eidas.specific.core.MsEventCodes;
import at.asitplus.eidas.specific.core.MsEidasNodeConstants;
+import at.asitplus.eidas.specific.core.MsEventCodes;
import at.asitplus.eidas.specific.core.gui.StaticGuiBuilderConfiguration;
import at.asitplus.eidas.specific.modules.auth.eidas.v2.Constants;
import at.asitplus.eidas.specific.modules.auth.eidas.v2.exception.EidPreProcessingException;
@@ -137,6 +137,8 @@ public class GenerateAuthnRequestTask extends AbstractAuthServletTask {
}
+ log.info("Requesting authentication from EidasNode: {} with Id:{}",
+ lightAuthnReq.getCitizenCountryCode(), lightAuthnReq.getId());
revisionsLogger.logEvent(pendingReq, MsEventCodes.EIDAS_NODE_CONNECTED, lightAuthnReq.getId());
log.info("Allowed LoA: {}",
StringUtils.join(pendingReq.getServiceProviderConfiguration().getRequiredLoA(),", "));
diff --git a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseAlternativeTask.java b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseAlternativeTask.java
index b05ce69d..aabfd16a 100644
--- a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseAlternativeTask.java
+++ b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseAlternativeTask.java
@@ -128,7 +128,7 @@ public class ReceiveAuthnResponseAlternativeTask extends AbstractAuthServletTask
log.warn("NO eIDAS response-message found.");
throw new EidasSAuthenticationException("eidas.01", null);
}
- log.debug("Receive eIDAS response with RespId: {} for ReqId: {}",
+ log.info("Receive eIDAS response with RespId: {} for ReqId: {}",
eidasResponse.getId(), eidasResponse.getInResponseToId());
log.trace("Full eIDAS-Resp: {}", eidasResponse);
return eidasResponse;
diff --git a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseTask.java b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseTask.java
index a736ae08..ad07cd25 100644
--- a/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseTask.java
+++ b/modules/authmodule-eIDAS-v2/src/main/java/at/asitplus/eidas/specific/modules/auth/eidas/v2/tasks/ReceiveAuthnResponseTask.java
@@ -35,8 +35,8 @@ import org.springframework.context.ApplicationContext;
import org.springframework.stereotype.Component;
import org.springframework.web.util.UriComponentsBuilder;
-import at.asitplus.eidas.specific.core.MsEventCodes;
import at.asitplus.eidas.specific.core.MsEidasNodeConstants;
+import at.asitplus.eidas.specific.core.MsEventCodes;
import at.asitplus.eidas.specific.modules.auth.eidas.v2.Constants;
import at.asitplus.eidas.specific.modules.auth.eidas.v2.exception.EidasSAuthenticationException;
import at.asitplus.eidas.specific.modules.auth.eidas.v2.exception.EidasValidationException;
@@ -98,7 +98,7 @@ public class ReceiveAuthnResponseTask extends AbstractAuthServletTask {
try {
final ILightResponse eidasResponse = extractEidasResponse(request);
String stagingEndpoint = pendingReq.getRawData(
- MsEidasNodeConstants.EXECCONTEXT_PARAM_MSCONNECTOR_STAGING, String.class);
+ MsEidasNodeConstants.EXECCONTEXT_PARAM_MSCONNECTOR_STAGING, String.class);
if (StringUtils.isNotEmpty(stagingEndpoint)) {
log.info("Find ms-connector staging to: {}. Forwarding to that endpoint ... ", stagingEndpoint);
forwardToOtherStage(response, executionContext, eidasResponse, stagingEndpoint);
@@ -159,7 +159,7 @@ public class ReceiveAuthnResponseTask extends AbstractAuthServletTask {
log.warn("NO eIDAS response-message found.");
throw new EidasSAuthenticationException("eidas.01", null);
}
- log.debug("Receive eIDAS response with RespId: {} for ReqId: {}",
+ log.info("Receive eIDAS response with RespId: {} for ReqId: {}",
eidasResponse.getId(), eidasResponse.getInResponseToId());
log.trace("Full eIDAS-Resp: {}", eidasResponse);
revisionsLogger.logEvent(pendingReq, MsEventCodes.RESPONSE_FROM_EIDAS_NODE, eidasResponse.getId());
--
cgit v1.2.3