add 'isAbortedByUser' flag to pending-request to indicate that this request is aborted, but the auth. process is not broken

1 files changed, 12 insertions, 10 deletions

diff --git a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/auth/modules/internal/tasks/EvaluateSSOConsentsTaskImpl.java b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/auth/modules/internal/tasks/EvaluateSSOConsentsTaskImpl.java
index d52b76ebd..5b53a43bd 100644
--- a/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/auth/modules/internal/tasks/EvaluateSSOConsentsTaskImpl.java
+++ b/id/server/idserverlib/src/main/java/at/gv/egovernment/moa/id/auth/modules/internal/tasks/EvaluateSSOConsentsTaskImpl.java
@@ -90,19 +90,21 @@ public class EvaluateSSOConsentsTaskImpl extends AbstractAuthServletTask {
 			if (ssoConsents) {
 				//authenticate pending-request
 				pendingReq.setAuthenticated(true);
-				
-				//store pending-request
-				requestStoreage.storePendingRequest(pendingReq);
-				
-				//redirect to auth. protocol finalization
-				performRedirectToProtocolFinialization(pendingReq, response);
-				
+				pendingReq.setAbortedByUser(false);
+								
 			} else {
 				//user deny single sign-on authentication
-				throw new AuthenticationException("auth.21", new Object[] {});
-				
+				Logger.debug("User deny the Single Sign-On authentication for SP: " + pendingReq.getOAURL());
+				pendingReq.setAbortedByUser(true);
+			
 			}
-						
+			
+			//store pending-request			
+			requestStoreage.storePendingRequest(pendingReq);
+
+			//redirect to auth. protocol finalization
+			performRedirectToProtocolFinialization(pendingReq, response);
+			
 		} catch (MOAIDException e) {
 			throw new TaskExecutionException(pendingReq, e.getMessage(), e);