Player UUID

[5eli]

Context
implemented UUID-based authentification mechanism for players and preventing new joins during an active game

Description
players now get assigned a unique UUID upon joining a lobby. This helps in preventing new players from joining a lobby once a game has started

Changes in the code base
LobbyService: Added logic to validate UUIDs upon reconnecting
LobbyController: updated joinLobby endpoint to accept an optional uuid parameter
Lobby: updated player management to support UUID validation
Tests: updated/added tests in LobbyTest and LobbyServiceTest

Additional information
N/A

[the-only-queen-anna]

Aktuell startet mit diesem Code bei mir der Server nicht, ich bekomme immer einen Error von der Datenbank, bitte mal selber probieren, ob das bei dir geht

[5eli]

Aktuell startet mit diesem Code bei mir der Server nicht, ich bekomme immer einen Error von der Datenbank, bitte mal selber probieren, ob das bei dir geht

Kannst du mir eventuell die genaue Fehlermeldung schicken, die du bekommst? Dann kann ich mir das genauer anschauen.

[the-only-queen-anna]

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'entityManagerFactory' defined in class path resource [org/springframework/boot/autoconfigure/orm/jpa/HibernateJpaConfiguration.class]: [PersistenceUnit: default] Unable to build Hibernate SessionFactory; nested exception is org.hibernate.tool.schema.spi.SchemaManagementException: Schema-validation: missing column [uuid] in table [player] ich bekomm immer das, ist das ein lokales Problem ? eigentlich baue ich immer alle container neu

[5eli]

Hab's mir angeschaut, die uuid-Spalte hat in der Datenbankmigration gefehlt. Das sollte jetzt mit der neuen Migration gefixt sein.

Ich kann es bei mir leider nicht vollständig testen, weil ich beim Starten des Servers aktuell immer einen PostgreSQL-Authentifizierungsfehler bekomme (password authentication failed for user "codenames_user"). Könntest du nochmal schauen, ob es bei dir jetzt funktioniert?

[XtophB]

Aktuell startet mit diesem Code bei mir der Server nicht, ich bekomme immer einen Error von der Datenbank, bitte mal selber probieren, ob das bei dir geht

Server DB problem should be fixed. Hab container mit dem Branch UUID Branch Build gestartet

[the-only-queen-anna]

Hab noch einige Kommentare...

[the-only-queen-anna]

Das führt bei jeder Request zu einer NullPointerException java.lang.NullPointerException: null
at java.base/java.util.Objects.requireNonNull(Objects.java:209) ~[na:na]
at com.codenames.codenames.backend.lobby.api.LobbyController.createLobby(LobbyController.java:55) ~[!/:0.0.1-SNAPSHOT]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:na]
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[na:na]
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
at java.base/java.lang.reflect.Method.invoke(Method.java:569) ~[na:na]

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
88.9% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[XtophB]

Tested this in meeting with @tasaje1 @the-only-queen-anna @ad-devel. It works 👍

[tasaje1]

tested during meeting works lgtm :)

[the-only-queen-anna]

Alle changes umgesetzt, LGTM