Merge pull request #228 from boostcampwm-2024/feat/#227-fix-user-decorator

Donggle0315 · web-flow · commit 468f99bc43a7 · 2024-11-29T00:37:28.000+09:00
🐛 fix: 유저 데코레이터 중복 로그인 핸들링
diff --git a/back/src/auth/service/auth.service.ts b/back/src/auth/service/auth.service.ts
@@ -12,12 +12,12 @@ export class AuthService {
     this.redis = this.redisService.getOrThrow();
   }
 
-  async getUserIdFromSession(sid: string): Promise<[number, string] | null> {
+  async getUserIdFromSession(sid: string): Promise<[number | null, string | null]> {
     const session = JSON.parse(await this.redis.get(`user:${sid}`));
-    if (!session) return null;
+    if (!session) return [null, null];
     const userId = session.id;
     const userLoginId = session.loginId;
-    if (!userId || !userLoginId) return null;
+    if (!userId || !userLoginId) return [null, null];
     return [userId, userLoginId];
   }
 

Original file line number	Diff line number	Diff line change
`@@ -12,12 +12,12 @@ export class AuthService {`
`12`	`12`	`this.redis = this.redisService.getOrThrow();`
`13`	`13`	`}`
`14`	`14`
`15`		`- async getUserIdFromSession(sid: string): Promise<[number, string] \| null> {`
	`15`	`+ async getUserIdFromSession(sid: string): Promise<[number \| null, string \| null]> {`
`16`	`16`	const session = JSON.parse(await this.redis.get(`user:${sid}`));
`17`		`- if (!session) return null;`
	`17`	`+ if (!session) return [null, null];`
`18`	`18`	`const userId = session.id;`
`19`	`19`	`const userLoginId = session.loginId;`
`20`		`- if (!userId \|\| !userLoginId) return null;`
	`20`	`+ if (!userId \|\| !userLoginId) return [null, null];`
`21`	`21`	`return [userId, userLoginId];`
`22`	`22`	`}`
`23`	`23`