From 4789273d8ff86c77ed3f1f7c079c3112a86f34d2 Mon Sep 17 00:00:00 2001 From: Lars Simon Winzer Date: Tue, 7 Apr 2026 14:02:18 +0200 Subject: [PATCH] Add: Send ErrorResponse with 'NO_USER_ASSOCIATED' code when no user is associated with the session --- .../server/app/commands/logout/LogoutHandler.java | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/logout/LogoutHandler.java b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/logout/LogoutHandler.java index af08e0c..c8e00b6 100644 --- a/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/logout/LogoutHandler.java +++ b/src/main/java/ch/unibas/dmi/dbis/cs108/casono/server/app/commands/logout/LogoutHandler.java @@ -16,7 +16,15 @@ public class LogoutHandler implements CommandHandler { @Override public void execute(LogoutRequest request) { - userRegistry.removeBySessionId(request.getSessionId()); - responseDispatcher.dispatch(new OkResponse(request.getContext())); + boolean was_removed = userRegistry.removeBySessionId(request.getSessionId()); + if (was_removed) { + responseDispatcher.dispatch(new OkResponse(request.getContext())); + } else { + responseDispatcher.dispatch( + new ErrorResponse( + request.getContext(), + "NO_USER_ASSOCIATED", + "No user is associated with your session. Did you login before?")); + } } }