summaryrefslogtreecommitdiff
path: root/eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java
diff options
context:
space:
mode:
Diffstat (limited to 'eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java')
-rw-r--r--eaaf_core/src/main/java/at/gv/egiz/eaaf/core/impl/idp/auth/services/ProtocolAuthenticationService.java22
1 files changed, 8 insertions, 14 deletions
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 5f84d118..fbf19429 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
@@ -181,13 +181,6 @@ public class ProtocolAuthenticationService implements IProtocolAuthenticationSer
new Object[] {}),
req, resp, pendingReq);
- // do not remove the full active SSO-Session
- // in case of only one Service-Provider authentication request is aborted
- if (!pendingReq.needSingleSignOnFunctionality()) {
- requestStorage.removePendingRequest(pendingReq.getPendingRequestId());
-
- }
-
// check if pending-request are authenticated
} else if (pendingReq.isAuthenticated() && !pendingReq.isNeedUserConsent()) {
internalFinalizeAuthenticationProcess(req, resp, pendingReq);
@@ -198,7 +191,8 @@ public class ProtocolAuthenticationService implements IProtocolAuthenticationSer
pendingReq.isAuthenticated(), pendingReq.isNeedUserConsent());
if (pendingReq.isNeedUserConsent()) {
log.error(
- "PendingRequest NEEDS user-consent. Can NOT fininalize authentication --> Abort authentication process!");
+ "PendingRequest NEEDS user-consent. "
+ + "Can NOT fininalize authentication --> Abort authentication process!");
} else {
log.error("PendingRequest is NOT authenticated --> Abort authentication process!");
@@ -213,13 +207,13 @@ public class ProtocolAuthenticationService implements IProtocolAuthenticationSer
log.error("Finalize authentication protocol FAILED.", e);
buildProtocolSpecificErrorResponse(e, req, resp, pendingReq);
+ } finally {
+ // remove pending-request
+ requestStorage.removePendingRequest(pendingReq.getPendingRequestId());
+ revisionsLogger.logEvent(EventConstants.TRANSACTION_DESTROYED,
+ pendingReq.getUniqueTransactionIdentifier());
}
-
- // remove pending-request
- requestStorage.removePendingRequest(pendingReq.getPendingRequestId());
- revisionsLogger.logEvent(EventConstants.TRANSACTION_DESTROYED,
- pendingReq.getUniqueTransactionIdentifier());
-
+
}
@Override