From a5db9d69b3341ecdb6a04b1ce11ce30e681f14b8 Mon Sep 17 00:00:00 2001 From: Rayyan <60314224+RayyanHermanto@users.noreply.github.com> Date: Mon, 3 Aug 2026 23:16:56 +0700 Subject: [PATCH] Fix password still exist --- .../identity/domain/entities/user.entity.ts | 16 ++++++---------- .../infrastructure/mappers/prisma-user.mapper.ts | 6 ++---- .../identity/presentation/dto/create-user.dto.ts | 5 +---- .../identity/presentation/dto/update-user.dto.ts | 5 +---- 4 files changed, 10 insertions(+), 22 deletions(-) diff --git a/src/modules/identity/domain/entities/user.entity.ts b/src/modules/identity/domain/entities/user.entity.ts index 335462e..e63b1f4 100644 --- a/src/modules/identity/domain/entities/user.entity.ts +++ b/src/modules/identity/domain/entities/user.entity.ts @@ -1,13 +1,13 @@ import { RoleData } from "./role.entity"; import { PermissionData } from "./permission.entity"; +import crypto from 'crypto'; export class UserData { private constructor( public readonly id: string, public authentikId: string | null, public username: string, - public email: string, - public password: string | null, + public email: string, public roles: RoleData[], public permissions: PermissionData[], public isActive: boolean, @@ -20,9 +20,8 @@ export class UserData { //#region Create static create(data: { - username: string; + username: string; email: string; - password?: string | null; authentikId?: string | null; storageQuota?: number; storageUsed?: number; @@ -34,8 +33,7 @@ export class UserData { data.authentikId || null, data.username, data.email, - data.password ?? null, - [], + [], [], true, null, @@ -118,8 +116,7 @@ export class UserData { id: string; authentikId?: string | null; username: string; - email: string; - password: string | null; + email: string; roles?: RoleData[]; permissions?: PermissionData[]; isActive?: boolean; @@ -133,8 +130,7 @@ export class UserData { props.id, props.authentikId || null, props.username, - props.email, - props.password, + props.email, props.roles || [], props.permissions || [], props.isActive !== undefined ? props.isActive : true, diff --git a/src/modules/identity/infrastructure/mappers/prisma-user.mapper.ts b/src/modules/identity/infrastructure/mappers/prisma-user.mapper.ts index 2a8dcad..6c3cbc0 100644 --- a/src/modules/identity/infrastructure/mappers/prisma-user.mapper.ts +++ b/src/modules/identity/infrastructure/mappers/prisma-user.mapper.ts @@ -41,8 +41,7 @@ export class PrismaUserMapper { id: model.id, authentikId: model.authentikUserId || model.authentikId || null, username: model.username || model.name || model.username, - email: model.email, - password: model.password ?? null, + email: model.email, roles, permissions, isActive: model.isActive ?? true, @@ -59,8 +58,7 @@ export class PrismaUserMapper { authentikUserId: (userData as any).authentikUserId || userData.authentikId, authentikId: userData.authentikId, username: userData.username, - email: userData.email, - password: userData.password ?? null, + email: userData.email, isActive: userData.isActive, deletedAt: userData.deletedAt, lastSeenAt: userData.lastSeenAt, diff --git a/src/modules/identity/presentation/dto/create-user.dto.ts b/src/modules/identity/presentation/dto/create-user.dto.ts index ee203a6..8996836 100644 --- a/src/modules/identity/presentation/dto/create-user.dto.ts +++ b/src/modules/identity/presentation/dto/create-user.dto.ts @@ -11,10 +11,7 @@ export class CreateUserDto { @IsEmail() email!: string; - @ApiProperty({ example: 'strongpassword' }) - @IsString() - @Length(8, 128) - password!: string; + @ApiPropertyOptional({ example: 10737418240 }) @IsOptional() diff --git a/src/modules/identity/presentation/dto/update-user.dto.ts b/src/modules/identity/presentation/dto/update-user.dto.ts index 968d85a..0fa3899 100644 --- a/src/modules/identity/presentation/dto/update-user.dto.ts +++ b/src/modules/identity/presentation/dto/update-user.dto.ts @@ -14,10 +14,7 @@ export class UpdateUserDto extends PartialType(CreateUserDto) { @IsEmail() email?: string; - @ApiPropertyOptional() - @IsOptional() - @Length(8, 128) - password?: string; + @ApiPropertyOptional() @IsOptional()