Add serverside LOGOUT-command #41

New Issue

2026-04-01T13:33:55+02:00

lars.winzer commented

2026-04-01 13:33:55 +02:00

(Migrated from git.scicore.unibas.ch)

Task

Summary

Adds parsers and handlers essential to logging out and destroying the user on the server.

Context

CommandParser, Request, CommandHandler

Current Progress

LogoutCommandParser
LogoutRequest
LogoutCommandHandler

Notes

Request

LOGOUT

Response

+OK

Checklist

I outlined checkpoints describing phases of my work related to this task
I added relevant labels
I linked to other issues or branches that must be completed first

## Task  ### Summary Adds parsers and handlers essential to logging out and destroying the user on the server. ### Context CommandParser, Request, CommandHandler ### Current Progress - [x] LogoutCommandParser - [x] LogoutRequest - [x] LogoutCommandHandler ### Notes #### Request ``` LOGOUT ``` #### Response ``` +OK ``` ### Checklist - [x] I outlined checkpoints describing phases of my work related to this task - [x] I added relevant labels - [x] I linked to other issues or branches that must be completed first

lars.winzer commented

2026-04-01 13:34:05 +02:00

(Migrated from git.scicore.unibas.ch)

marked this task as related to #37

lars.winzer commented

2026-04-01 13:34:10 +02:00

(Migrated from git.scicore.unibas.ch)

marked the checklist item I linked to other issues or branches that must be completed first as completed

marked the checklist item **I linked to other issues or branches that must be completed first** as completed

lars.winzer commented

2026-04-07 12:25:23 +02:00

(Migrated from git.scicore.unibas.ch)

assigned to @lars.winzer

lars.winzer commented

2026-04-07 12:25:41 +02:00

(Migrated from git.scicore.unibas.ch)

created branch feat/41-add-serverside-logout-command to address this issue

created branch [`feat/41-add-serverside-logout-command`](/cs108-fs26/Gruppe-13/-/compare/main...feat%2F41-add-serverside-logout-command) to address this issue

lars.winzer commented

2026-04-07 12:31:31 +02:00

(Migrated from git.scicore.unibas.ch)

marked the checklist item LogoutRequest as completed

marked the checklist item **LogoutRequest** as completed

lars.winzer commented

2026-04-07 12:31:59 +02:00

(Migrated from git.scicore.unibas.ch)

marked the checklist item LogoutCommandParser as completed

marked the checklist item **LogoutCommandParser** as completed

lars.winzer commented

2026-04-07 12:41:52 +02:00

(Migrated from git.scicore.unibas.ch)

marked the checklist item LogoutCommandHandler as completed

marked the checklist item **LogoutCommandHandler** as completed

lars.winzer commented

2026-04-07 12:57:51 +02:00

(Migrated from git.scicore.unibas.ch)

The current implementation of the LogoutHandler is insufficient. It works, but not as intended.

@Override
public void execute(LogoutRequest request) {
    userRegistry.onDisconnect(request.getSessionId());
    responseDispatcher.dispatch(new OkResponse(request.getContext()));
}

With the current implementation, the removal of the user from the UserRegistry is delegated to the UserCleanupJob.

The UserRegistry needs to be expanded to allow for removal of users.

The current implementation of the LogoutHandler is insufficient. It works, but not as intended. ```java @Override public void execute(LogoutRequest request) { userRegistry.onDisconnect(request.getSessionId()); responseDispatcher.dispatch(new OkResponse(request.getContext())); } ``` With the current implementation, the removal of the user from the UserRegistry is delegated to the UserCleanupJob. The UserRegistry needs to be expanded to allow for removal of users.

lars.winzer commented

2026-04-07 13:02:09 +02:00

(Migrated from git.scicore.unibas.ch)

mentioned in task #54