From 17873bb476d280e6f533cd7cabf8bb8e3e91d518 Mon Sep 17 00:00:00 2001 From: Viren070 Date: Sun, 8 Jun 2025 16:25:23 +0100 Subject: [PATCH] fix: don't validate user data on retrieval within UserRepository --- packages/core/src/db/users.ts | 38 ++++++++++++++++++----------------- 1 file changed, 20 insertions(+), 18 deletions(-) diff --git a/packages/core/src/db/users.ts b/packages/core/src/db/users.ts index c175f5be..7f2c4640 100644 --- a/packages/core/src/db/users.ts +++ b/packages/core/src/db/users.ts @@ -168,30 +168,32 @@ export class UserRepository { // ) // ); // } - const { - success, - data: validatedConfig, - error, - } = UserDataSchema.safeParse(decryptedConfig); - if (!success) { - return Promise.reject( - new APIError( - constants.ErrorCode.USER_INVALID_CONFIG, - undefined, - formatZodError(error) - ) - ); - } - validatedConfig.trusted = + // const { + // success, + // data: validatedConfig, + // error, + // } = UserDataSchema.safeParse(decryptedConfig); + // if (!success) { + // return Promise.reject( + // new APIError( + // constants.ErrorCode.USER_INVALID_CONFIG, + // undefined, + // formatZodError(error) + // ) + // ); + // } + decryptedConfig.trusted = Env.TRUSTED_UUIDS?.split(',').some((u) => new RegExp(u).test(uuid)) ?? false; logger.info(`Retrieved configuration for user ${uuid}`); - return validatedConfig; + return decryptedConfig; } catch (error) { logger.error( `Error retrieving user ${uuid}: ${error instanceof Error ? error.message : String(error)}` ); - return Promise.reject(new APIError(constants.ErrorCode.USER_ERROR)); + return Promise.reject( + new APIError(constants.ErrorCode.INTERNAL_SERVER_ERROR) + ); } } @@ -359,7 +361,7 @@ export class UserRepository { return Promise.reject(new APIError(constants.ErrorCode.USER_ERROR)); } - return UserDataSchema.parse(JSON.parse(decryptedString)); + return JSON.parse(decryptedString); } private static async generateUUID(count: number = 1): Promise {