From 54f16e552a3d6c5579282de5c8d6a92395f6c049 Mon Sep 17 00:00:00 2001
From: Thomas <>
Date: Sat, 13 May 2023 15:18:03 +0200
Subject: refact(core): remove double serialization in central error handling

---
 .../core/impl/idp/auth/services/ProtocolAuthenticationService.java    | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java')

diff --git a/eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java b/eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java
index 76687749..98ea339e 100644
--- a/eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java
+++ b/eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java
@@ -37,7 +37,6 @@ import org.springframework.context.ApplicationContext;
 import org.springframework.lang.NonNull;
 import org.springframework.lang.Nullable;
 import org.springframework.stereotype.Service;
-import org.springframework.util.SerializationUtils;
 
 import at.gv.egiz.components.eventlog.api.EventConstants;
 import at.gv.egiz.eaaf.core.api.IRequest;
@@ -248,12 +247,11 @@ public class ProtocolAuthenticationService implements IProtocolAuthenticationSer
         if (errorData.getErrorIdTokenForRedirect() != null) {
           // Put pending request
           final ExceptionContainer exceptionContainer = new ExceptionContainer(protocolRequest, throwable);
-          final byte[] serialized = SerializationUtils.serialize(exceptionContainer);
           log.debug("Put error into cache to support SP forwarding ... ");
           String internalErrorToken = pendingReqIdGenerationStrategy.getPendingRequestIdWithOutChecks(
               errorData.getErrorIdTokenForRedirect());
           log.trace("errorIdToken: {}", internalErrorToken);
-          transactionStorage.put(internalErrorToken, serialized, -1);
+          transactionStorage.put(internalErrorToken, exceptionContainer, -1);
           
         } else {
           log.debug("No errorTokenId. Forwarding to SP will not be available");
-- 
cgit v1.2.3