@ciscode/authentication-kit 1.2.1 → 1.4.1

package/dist/services/users.service.js

@@ -19,70 +19,132 @@ const user_repository_1 = require("../repositories/user.repository");
 const role_repository_1 = require("../repositories/role.repository");
 const mongoose_1 = require("mongoose");
 const helper_1 = require("../utils/helper");
+const logger_service_1 = require("./logger.service");
 let UsersService = class UsersService {
-    constructor(users, rolesRepo) {
+    constructor(users, rolesRepo, logger) {
         this.users = users;
         this.rolesRepo = rolesRepo;
+        this.logger = logger;
     }
     async create(dto) {
-        // Generate username from fname-lname if not provided
-        if (!dto.username || dto.username.trim() === '') {
-            dto.username = (0, helper_1.generateUsernameFromName)(dto.fullname.fname, dto.fullname.lname);
+        try {
+            // Generate username from fname-lname if not provided
+            if (!dto.username || dto.username.trim() === '') {
+                dto.username = (0, helper_1.generateUsernameFromName)(dto.fullname.fname, dto.fullname.lname);
+            }
+            // Check for existing user
+            const [existingEmail, existingUsername, existingPhone] = await Promise.all([
+                this.users.findByEmail(dto.email),
+                this.users.findByUsername(dto.username),
+                dto.phoneNumber ? this.users.findByPhone(dto.phoneNumber) : null,
+            ]);
+            if (existingEmail || existingUsername || existingPhone) {
+                throw new common_1.ConflictException('An account with these credentials already exists');
+            }
+            // Hash password
+            let hashed;
+            try {
+                const salt = await bcryptjs_1.default.genSalt(10);
+                hashed = await bcryptjs_1.default.hash(dto.password, salt);
+            }
+            catch (error) {
+                this.logger.error(`Password hashing failed: ${error.message}`, error.stack, 'UsersService');
+                throw new common_1.InternalServerErrorException('User creation failed');
+            }
+            const user = await this.users.create({
+                fullname: dto.fullname,
+                username: dto.username,
+                email: dto.email,
+                phoneNumber: dto.phoneNumber,
+                avatar: dto.avatar,
+                jobTitle: dto.jobTitle,
+                company: dto.company,
+                password: hashed,
+                roles: [],
+                isVerified: true,
+                isBanned: false,
+                passwordChangedAt: new Date()
+            });
+            return { id: user._id, email: user.email };
         }
-        if (await this.users.findByEmail(dto.email))
-            throw new Error('Email already in use.');
-        if (await this.users.findByUsername(dto.username))
-            throw new Error('Username already in use.');
-        if (dto.phoneNumber && (await this.users.findByPhone(dto.phoneNumber))) {
-            throw new Error('Phone already in use.');
+        catch (error) {
+            if (error instanceof common_1.ConflictException || error instanceof common_1.InternalServerErrorException) {
+                throw error;
+            }
+            if ((error === null || error === void 0 ? void 0 : error.code) === 11000) {
+                throw new common_1.ConflictException('An account with these credentials already exists');
+            }
+            this.logger.error(`User creation failed: ${error.message}`, error.stack, 'UsersService');
+            throw new common_1.InternalServerErrorException('User creation failed');
         }
-        const salt = await bcryptjs_1.default.genSalt(10);
-        const hashed = await bcryptjs_1.default.hash(dto.password, salt);
-        const user = await this.users.create({
-            fullname: dto.fullname,
-            username: dto.username,
-            email: dto.email,
-            phoneNumber: dto.phoneNumber,
-            avatar: dto.avatar,
-            jobTitle: dto.jobTitle,
-            company: dto.company,
-            password: hashed,
-            roles: [],
-            isVerified: true,
-            isBanned: false,
-            passwordChangedAt: new Date()
-        });
-        return { id: user._id, email: user.email };
     }
     async list(filter) {
-        return this.users.list(filter);
+        try {
+            return this.users.list(filter);
+        }
+        catch (error) {
+            this.logger.error(`User list failed: ${error.message}`, error.stack, 'UsersService');
+            throw new common_1.InternalServerErrorException('Failed to retrieve users');
+        }
     }
     async setBan(id, banned) {
-        const user = await this.users.updateById(id, { isBanned: banned });
-        if (!user)
-            throw new Error('User not found.');
-        return { id: user._id, isBanned: user.isBanned };
+        try {
+            const user = await this.users.updateById(id, { isBanned: banned });
+            if (!user) {
+                throw new common_1.NotFoundException('User not found');
+            }
+            return { id: user._id, isBanned: user.isBanned };
+        }
+        catch (error) {
+            if (error instanceof common_1.NotFoundException) {
+                throw error;
+            }
+            this.logger.error(`Set ban status failed: ${error.message}`, error.stack, 'UsersService');
+            throw new common_1.InternalServerErrorException('Failed to update user ban status');
+        }
     }
     async delete(id) {
-        const user = await this.users.deleteById(id);
-        if (!user)
-            throw new Error('User not found.');
-        return { ok: true };
+        try {
+            const user = await this.users.deleteById(id);
+            if (!user) {
+                throw new common_1.NotFoundException('User not found');
+            }
+            return { ok: true };
+        }
+        catch (error) {
+            if (error instanceof common_1.NotFoundException) {
+                throw error;
+            }
+            this.logger.error(`User deletion failed: ${error.message}`, error.stack, 'UsersService');
+            throw new common_1.InternalServerErrorException('Failed to delete user');
+        }
     }
     async updateRoles(id, roles) {
-        const existing = await this.rolesRepo.findByIds(roles);
-        if (existing.length !== roles.length)
-            throw new Error('One or more roles not found.');
-        const roleIds = roles.map((r) => new mongoose_1.Types.ObjectId(r));
-        const user = await this.users.updateById(id, { roles: roleIds });
-        if (!user)
-            throw new Error('User not found.');
-        return { id: user._id, roles: user.roles };
+        try {
+            const existing = await this.rolesRepo.findByIds(roles);
+            if (existing.length !== roles.length) {
+                throw new common_1.NotFoundException('One or more roles not found');
+            }
+            const roleIds = roles.map((r) => new mongoose_1.Types.ObjectId(r));
+            const user = await this.users.updateById(id, { roles: roleIds });
+            if (!user) {
+                throw new common_1.NotFoundException('User not found');
+            }
+            return { id: user._id, roles: user.roles };
+        }
+        catch (error) {
+            if (error instanceof common_1.NotFoundException) {
+                throw error;
+            }
+            this.logger.error(`Update user roles failed: ${error.message}`, error.stack, 'UsersService');
+            throw new common_1.InternalServerErrorException('Failed to update user roles');
+        }
     }
 };
 exports.UsersService = UsersService;
 exports.UsersService = UsersService = __decorate([
     (0, common_1.Injectable)(),
     __metadata("design:paramtypes", [user_repository_1.UserRepository,
-        role_repository_1.RoleRepository])
+        role_repository_1.RoleRepository,
+        logger_service_1.LoggerService])
 ], UsersService);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ciscode/authentication-kit",
-  "version": "1.2.1",
+  "version": "1.4.1",
   "description": "NestJS auth kit with local + OAuth, JWT, RBAC, password reset.",
   "publishConfig": {
     "access": "public"