fix: add lifecycle cleanup hooks and error handling for async operations

src/matches/match-assistant/match-assistant.service.ts

@@ -255,6 +255,13 @@ export class MatchAssistantService {
         error,
       );
       if (error instanceof FailedToCreateOnDemandServer) {
+        if (tries >= 10) {
+          this.logger.error(
+            `[${matchId}] max retries reached for server assignment`,
+          );
+          await this.updateMatchStatus(matchId, "WaitingForServer");
+          return;
+        }
         setTimeout(async () => {
           this.logger.log(`[${matchId}] try retry assign server....`);
           await this.assignServer(matchId, ++tries);

src/redis/redis-manager/redis-manager.service.ts

@@ -1,10 +1,10 @@
-import { Injectable, Logger } from "@nestjs/common";
+import { Injectable, Logger, OnApplicationShutdown } from "@nestjs/common";
 import IORedis, { Redis, RedisOptions } from "ioredis";
 import { ConfigService } from "@nestjs/config";
 import { RedisConfig } from "../../configs/types/RedisConfig";
 
 @Injectable()
-export class RedisManagerService {
+export class RedisManagerService implements OnApplicationShutdown {
   private config: RedisConfig;
 
   protected connections: {
@@ -22,6 +22,15 @@ export class RedisManagerService {
     this.config = this.configService.get("redis");
   }
 
+  onApplicationShutdown() {
+    for (const [, interval] of Object.entries(this.healthCheckIntervals)) {
+      clearInterval(interval);
+    }
+    for (const [, conn] of Object.entries(this.connections)) {
+      conn.disconnect();
+    }
+  }
+
   public getConnection(connection = "default"): Redis {
     if (!this.connections[connection]) {
       const currentConnection: Redis = (this.connections[connection] =