* Pong backend: Fixed matchmaking bugs

* Auth backend: reduced logs size on login

back/volume/src/auth/42.strategy.ts

@@ -30,9 +30,8 @@ export class FtStrategy extends PassportStrategy(Strategy, '42') {
     cb: VerifyCallback
   ): Promise<VerifyCallback> {
     request.session.accessToken = accessToken
-    console.log('accessToken', accessToken, 'refreshToken', refreshToken)
     const ftId = profile.id as number
-    console.log(profile)
+    console.log('Validated ', profile.username)
     if ((await this.usersService.findUser(ftId)) === null) {
       const newUser = new User()
       newUser.ftId = profile.id as number

back/volume/src/pong/game/MatchmakingQueue.ts

@@ -12,23 +12,24 @@ export class MatchmakingQueue {
     this.queue = []
   }
 
-  addPlayer (name: string, socket: WebSocket, uuid: string): boolean {
+  addPlayer (name: string, socket: WebSocket, uuid: string): void {
-    let succeeded: boolean = false
+    if (!this.isInQueue(name)) {
-    if (!this.alreadyInQueue(name)) {
+      console.log('Adding player to queue: ', name)
       this.queue.push({ name, socket, uuid })
       if (this.canCreateGame()) {
         this.createGame()
       }
-      succeeded = true
     }
-    return succeeded
   }
 
   removePlayer (name: string): void {
+    if (this.isInQueue(name)) {
+      console.log('Removing player from queue: ', name)
       this.queue = this.queue.filter((player) => player.name !== name)
     }
+  }
 
-  alreadyInQueue (name: string): boolean {
+  isInQueue (name: string): boolean {
     return this.queue.some((player) => player.name === name)
   }
 

back/volume/src/pong/pong.gateway.ts

@@ -62,6 +62,7 @@ export class PongGateway implements OnGatewayConnection, OnGatewayDisconnect {
     }
     if (name !== undefined) {
       console.log('Disconnected ', this.socketToPlayerName.get(client))
+      this.matchmakingQueue.removePlayer(name)
       this.socketToPlayerName.delete(client)
     }
   }
@@ -202,20 +203,19 @@ export class PongGateway implements OnGatewayConnection, OnGatewayDisconnect {
       client: WebSocketWithId,
       @MessageBody() matchmakingUpdateData: MatchmakingDtoValidated
   ): { event: string, data: MatchmakingDtoValidated } {
-    let isMatchmaking: boolean = false
+    let matchmaking: boolean = false
     const name: string | undefined = this.socketToPlayerName.get(client)
     if (name !== undefined) {
-      if (matchmakingUpdateData.matchmaking) {
+      if (matchmakingUpdateData.matchmaking && !this.games.isInAGame(name)) {
-        if (this.matchmakingQueue.addPlayer(name, client, client.id)) {
+        this.matchmakingQueue.addPlayer(name, client, client.id)
-          isMatchmaking = true
-        }
       } else {
         this.matchmakingQueue.removePlayer(name)
       }
+      matchmaking = this.matchmakingQueue.isInQueue(name)
     }
     return {
       event: GAME_EVENTS.MATCHMAKING,
-      data: { matchmaking: isMatchmaking }
+      data: { matchmaking }
     }
   }