fix(auth): reuse cache key constant and handle already-verified reset codes

Author: maksberegovoi

src/auth/application/use-cases/password/confirm-reset-password.use-case.ts

@@ -1,3 +1,4 @@
+import { RESET_PASSWORD_CACHE_KEY } from '@core/auth/infrastructure/constants';
 import { UpdatePasswordUseCase } from '@core/user';
 import { HttpStatus, Inject, Injectable } from '@nestjs/common';
 import { CACHE_SERVICE } from '@shared/adapters/cache/constants';
@@ -17,7 +18,7 @@ export class ConfirmResetPasswordUseCase {
     ) {}
 
     async execute(dto: PasswordResetConfirmDto) {
-        const redisKey = `pass:reset:${dto.email}`;
+        const redisKey = RESET_PASSWORD_CACHE_KEY(dto.email);
         const cachedData = await this.cacheService.getOne(redisKey);
 
         if (!cachedData) {

src/auth/application/use-cases/password/verify-reset-password.use-case.ts

@@ -1,3 +1,4 @@
+import { RESET_PASSWORD_CACHE_KEY } from '@core/auth/infrastructure/constants';
 import { HttpStatus, Inject, Injectable } from '@nestjs/common';
 import { CACHE_SERVICE } from '@shared/adapters/cache/constants';
 import { ICacheService } from '@shared/adapters/cache/ports';
@@ -15,7 +16,7 @@ export class VerifyResetPasswordUseCase {
     ) {}
 
     async execute(dto: VerifyResetCodeDto) {
-        const redisKey = `pass:reset:${dto.email}`;
+        const redisKey = RESET_PASSWORD_CACHE_KEY(dto.email);
         const cachedData = await this.cacheService.getOne(redisKey);
 
         if (!cachedData) {
@@ -43,14 +44,14 @@ export class VerifyResetPasswordUseCase {
             );
         }
 
-        if (!resetSession.isVerified) {
+        if (resetSession.isVerified) {
             throw new BaseException(
                 {
-                    code: AuthErrorCodes.CODE_NOT_VERIFIED,
-                    message: AuthErrorMessages[AuthErrorCodes.CODE_NOT_VERIFIED],
-                    details: [{ target: 'code', message: 'Код подтверждения не верифицирован' }],
+                    code: AuthErrorCodes.CODE_ALREADY_VERIFIED,
+                    message: AuthErrorMessages[AuthErrorCodes.CODE_ALREADY_VERIFIED],
+                    details: [{ target: 'code', message: 'Код подтверждения уже верифицирован' }],
                 },
-                HttpStatus.FORBIDDEN,
+                HttpStatus.BAD_REQUEST,
             );
         }
 

src/auth/domain/errors/auth.error.ts

@@ -10,6 +10,7 @@ export const AuthErrorCodes = {
     TOKEN_INVALID: 'AUTH.TOKEN_INVALID',
     REFRESH_FAILED: 'AUTH.REFRESH_FAILED',
     CODE_NOT_VERIFIED: 'AUTH.CODE_NOT_VERIFIED',
+    CODE_ALREADY_VERIFIED: 'AUTH.CODE_ALREADY_VERIFIED',
     SIGNUP_FAILED: 'AUTH.SIGNUP_FAILED',
     UNAUTHORIZED: 'AUTH.UNAUTHORIZED',
     RESET_SESSION_NOT_FOUND: 'AUTH.RESET_SESSION_NOT_FOUND',
@@ -39,6 +40,7 @@ export const AuthErrorMessages: Record<AuthErrorCode, string> = {
     [AuthErrorCodes.SESSION_REVOKED]: 'Сессия была отозвана. Выполните вход заново',
     [AuthErrorCodes.STRATEGY_NOT_FOUND]: 'Неизвестный контекст операции',
     [AuthErrorCodes.CODE_NOT_VERIFIED]: 'Код подтверждения еще не был верифицирован',
+    [AuthErrorCodes.CODE_ALREADY_VERIFIED]: 'Код подтверждения уже верифицирован',
     [AuthErrorCodes.UNAUTHORIZED]: 'Необходимо выполнить вход',
     [AuthErrorCodes.RESET_SESSION_NOT_FOUND]: 'Запрос на сброс пароля не найден',
     [AuthErrorCodes.REGISTRATION_EXPIRED]: 'Регистрация истекла. Зарегистрируйтесь заново',