@flusys/nestjs-auth 1.1.0-beta → 2.0.0-rc.0

package/cjs/modules/auth.module.js

@@ -12,16 +12,15 @@ const _modules = require("@flusys/nestjs-shared/modules");
 const _common = require("@nestjs/common");
 const _jwt = require("@nestjs/jwt");
 const _passport = require("@nestjs/passport");
-const _typeorm = require("@nestjs/typeorm");
-const _authconfigservice = require("../services/auth-config.service");
-const _authdatasourceprovider = require("../services/auth-datasource.provider");
+const _guards = require("@flusys/nestjs-shared/guards");
 const _authconstants = require("../config/auth.constants");
 const _controllers = require("../controllers");
 const _entities = require("../entities");
-const _guards = require("@flusys/nestjs-shared/guards");
 const _cleartokeninterceptor = require("../interceptors/clear-token.interceptor");
 const _settokeninterceptor = require("../interceptors/set-token.interceptor");
 const _services = require("../services");
+const _authconfigservice = require("../services/auth-config.service");
+const _authdatasourceprovider = require("../services/auth-datasource.provider");
 const _jwtstrategy = require("../strategies/jwt.strategy");
 function _ts_decorate(decorators, target, key, desc) {
     var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
@@ -31,82 +30,42 @@ function _ts_decorate(decorators, target, key, desc) {
 }
 let AuthModule = class AuthModule {
     static forRoot(options) {
-        return this.buildDynamicModule(options);
+        const optionsProvider = {
+            provide: _authconstants.AUTH_MODULE_OPTIONS,
+            useValue: options
+        };
+        return this.buildModule(options, [
+            optionsProvider,
+            ...options.providers ?? []
+        ]);
     }
-    static forRootAsync(asyncOptions) {
-        const { global = false, includeController = true, bootstrapAppConfig: { enableCompanyFeature } = {}, imports: externalImports = [], providers: additionalProviders = [] } = asyncOptions;
-        const asyncProviders = this.createAsyncProviders(asyncOptions);
-        // Build imports
-        const imports = [
-            ...externalImports,
-            _passport.PassportModule.register({
-                defaultStrategy: 'jwt'
-            }),
-            _jwt.JwtModule.register({}),
-            _modules.CacheModule,
-            _modules.UtilsModule
-        ];
-        // Build controllers based on configuration
-        const controllers = [];
-        if (includeController) {
-            // Core authentication controller (always included)
-            controllers.push(_controllers.AuthenticationController);
-            // User controller (always included)
-            controllers.push(_controllers.UserController);
-            // Company feature controllers (conditional)
-            if (enableCompanyFeature) {
-                controllers.push(_controllers.CompanySelectionController, _controllers.CompanyController, _controllers.BranchController, _controllers.UserPermissionController);
-            }
-        }
-        // Build providers
-        const providers = [
+    static forRootAsync(options) {
+        const asyncProviders = this.createAsyncProviders(options);
+        return this.buildModule(options, [
             ...asyncProviders,
-            ...additionalProviders,
-            // Config & Infrastructure
-            _authconfigservice.AuthConfigService,
-            _authdatasourceprovider.AuthDataSourceProvider,
-            // Auth core
-            _jwtstrategy.JwtStrategy,
-            _guards.JwtAuthGuard,
-            _services.AuthenticationService,
-            _services.AuthEmailService,
-            // Interceptors
-            _settokeninterceptor.SetToken,
-            _cleartokeninterceptor.ClearToken,
-            // Core services (always included)
-            _services.UserService
-        ];
-        // Company feature services (conditional)
-        if (enableCompanyFeature) {
-            providers.push(_services.CompanySelectionSessionService, _services.CompanyService, _services.BranchService, _services.UserPermissionService);
-        }
-        // Build exports
-        const exports1 = [
-            _passport.PassportModule,
-            _authconstants.AUTH_MODULE_OPTIONS,
-            _authconfigservice.AuthConfigService,
-            _authdatasourceprovider.AuthDataSourceProvider,
-            _jwtstrategy.JwtStrategy,
-            _guards.JwtAuthGuard,
-            _services.AuthenticationService,
-            _services.AuthEmailService,
-            _services.UserService
-        ];
-        if (enableCompanyFeature) {
-            exports1.push(_services.CompanySelectionSessionService, _services.CompanyService, _services.BranchService, _services.UserPermissionService);
-        }
+            ...options.providers ?? []
+        ], options.imports);
+    }
+    /** Build the dynamic module with shared logic */ static buildModule(options, providers, externalImports = []) {
+        const { global = false, includeController = true, bootstrapAppConfig } = options;
+        const enableCompanyFeature = bootstrapAppConfig?.enableCompanyFeature ?? false;
         return {
             module: AuthModule,
             global,
-            imports,
-            controllers,
-            providers,
-            exports: exports1
+            imports: [
+                ...externalImports,
+                ...this.buildImports()
+            ],
+            controllers: this.buildControllers(includeController, enableCompanyFeature),
+            providers: [
+                ...providers,
+                ...this.buildProviders(enableCompanyFeature)
+            ],
+            exports: this.buildExports(enableCompanyFeature)
         };
     }
-    static buildDynamicModule(options) {
-        const { bootstrapAppConfig: { enableCompanyFeature } = {}, includeController = true, global = false, providers: additionalProviders = [] } = options;
-        const imports = [
+    /** Common module imports */ static buildImports() {
+        return [
             _passport.PassportModule.register({
                 defaultStrategy: 'jwt'
             }),
@@ -114,42 +73,36 @@ let AuthModule = class AuthModule {
             _modules.CacheModule,
             _modules.UtilsModule
         ];
-        // Build controllers based on configuration
-        const controllers = [];
-        if (includeController) {
-            // Core authentication controller (always included)
-            controllers.push(_controllers.AuthenticationController);
-            // User controller (always included)
-            controllers.push(_controllers.UserController);
-            // Company feature controllers (conditional)
-            if (enableCompanyFeature) {
-                controllers.push(_controllers.CompanySelectionController, _controllers.CompanyController, _controllers.BranchController, _controllers.UserPermissionController);
-            }
+    }
+    /** Build controllers based on feature flags */ static buildControllers(includeController, enableCompanyFeature) {
+        if (!includeController) return [];
+        const controllers = [
+            _controllers.AuthenticationController,
+            _controllers.UserController
+        ];
+        if (enableCompanyFeature) {
+            controllers.push(_controllers.CompanySelectionController, _controllers.CompanyController, _controllers.BranchController, _controllers.UserPermissionController);
         }
+        return controllers;
+    }
+    /** Core providers (without AUTH_MODULE_OPTIONS - that's added separately) */ static buildProviders(enableCompanyFeature) {
         const providers = [
-            {
-                provide: _authconstants.AUTH_MODULE_OPTIONS,
-                useValue: options
-            },
-            ...additionalProviders,
-            // Config & Infrastructure
             _authconfigservice.AuthConfigService,
             _authdatasourceprovider.AuthDataSourceProvider,
-            // Auth core
             _jwtstrategy.JwtStrategy,
             _guards.JwtAuthGuard,
             _services.AuthenticationService,
             _services.AuthEmailService,
-            // Interceptors
             _settokeninterceptor.SetToken,
             _cleartokeninterceptor.ClearToken,
-            // Core services (always included)
             _services.UserService
         ];
-        // Company feature services (conditional)
         if (enableCompanyFeature) {
             providers.push(_services.CompanySelectionSessionService, _services.CompanyService, _services.BranchService, _services.UserPermissionService);
         }
+        return providers;
+    }
+    /** Build module exports */ static buildExports(enableCompanyFeature) {
         const exports1 = [
             _passport.PassportModule,
             _authconstants.AUTH_MODULE_OPTIONS,
@@ -164,16 +117,9 @@ let AuthModule = class AuthModule {
         if (enableCompanyFeature) {
             exports1.push(_services.CompanySelectionSessionService, _services.CompanyService, _services.BranchService, _services.UserPermissionService);
         }
-        return {
-            module: AuthModule,
-            global,
-            imports,
-            controllers,
-            providers,
-            exports: exports1
-        };
+        return exports1;
     }
-    static createAsyncProviders(options) {
+    /** Create async providers for forRootAsync */ static createAsyncProviders(options) {
         const staticOptions = {
             global: options.global,
             includeController: options.includeController,
@@ -183,13 +129,10 @@ let AuthModule = class AuthModule {
             return [
                 {
                     provide: _authconstants.AUTH_MODULE_OPTIONS,
-                    useFactory: async (...args)=>{
-                        const config = await options.useFactory(...args);
-                        return {
+                    useFactory: async (...args)=>({
                             ...staticOptions,
-                            config
-                        };
-                    },
+                            config: await options.useFactory(...args)
+                        }),
                     inject: options.inject || []
                 }
             ];
@@ -202,13 +145,10 @@ let AuthModule = class AuthModule {
                 },
                 {
                     provide: _authconstants.AUTH_MODULE_OPTIONS,
-                    useFactory: async (factory)=>{
-                        const config = await factory.createAuthOptions();
-                        return {
+                    useFactory: async (factory)=>({
                             ...staticOptions,
-                            config
-                        };
-                    },
+                            config: await factory.createAuthOptions()
+                        }),
                     inject: [
                         options.useClass
                     ]
@@ -219,13 +159,10 @@ let AuthModule = class AuthModule {
             return [
                 {
                     provide: _authconstants.AUTH_MODULE_OPTIONS,
-                    useFactory: async (factory)=>{
-                        const config = await factory.createAuthOptions();
-                        return {
+                    useFactory: async (factory)=>({
                             ...staticOptions,
-                            config
-                        };
-                    },
+                            config: await factory.createAuthOptions()
+                        }),
                     inject: [
                         options.useExisting
                     ]
@@ -239,35 +176,14 @@ let AuthModule = class AuthModule {
             }
         ];
     }
-    /**
-   * Create repository providers that use AuthDataSourceProvider
-   * This replaces TypeOrmModule.forFeature() functionality
-   * Note: Must be REQUEST-scoped since AuthDataSourceProvider is REQUEST-scoped
-   */ static getRepositoryProviders(enableCompanyFeature) {
-        const entities = this.getEntities(enableCompanyFeature);
-        return entities.map((entity)=>({
-                provide: (0, _typeorm.getRepositoryToken)(entity),
-                scope: _common.Scope.REQUEST,
-                useFactory: async (dataSourceProvider)=>{
-                    return await dataSourceProvider.getRepository(entity);
-                },
-                inject: [
-                    _authdatasourceprovider.AuthDataSourceProvider
-                ]
-            }));
-    }
-    static getEntities(enableCompanyFeature = false) {
-        const coreEntities = [
-            _entities.User
-        ];
-        const companyEntities = enableCompanyFeature ? [
+    /** Get auth entities based on feature flags */ static getEntities(enableCompanyFeature = false) {
+        return enableCompanyFeature ? [
+            _entities.User,
             _entities.Company,
             _entities.CompanyBranch,
             _entities.UserCompanyPermission
-        ] : [];
-        return [
-            ...coreEntities,
-            ...companyEntities
+        ] : [
+            _entities.User
         ];
     }
 };

package/cjs/services/auth-config.service.js

@@ -9,8 +9,8 @@ Object.defineProperty(exports, "AuthConfigService", {
     }
 });
 const _common = require("@nestjs/common");
-const _authmoduleoptionsinterface = require("../interfaces/auth-module-options.interface");
 const _authconstants = require("../config/auth.constants");
+const _authmoduleoptionsinterface = require("../interfaces/auth-module-options.interface");
 function _define_property(obj, key, value) {
     if (key in obj) {
         Object.defineProperty(obj, key, {
@@ -39,26 +39,39 @@ function _ts_param(paramIndex, decorator) {
     };
 }
 let AuthConfigService = class AuthConfigService {
-    // Database Mode
-    /** Get database mode (single or multi-tenant) */ getDatabaseMode() {
+    // Database Mode (required by IModuleConfigService)
+    getDatabaseMode() {
         return this.options.bootstrapAppConfig?.databaseMode ?? 'single';
     }
-    /** Check if running in multi-tenant mode */ isMultiTenant() {
+    isMultiTenant() {
         return this.getDatabaseMode() === 'multi-tenant';
     }
     // Feature Flags
-    /** Check if company feature is enabled */ isCompanyFeatureEnabled() {
+    isCompanyFeatureEnabled() {
         return this.options.bootstrapAppConfig?.enableCompanyFeature ?? false;
     }
     // JWT Configuration
     /** Get JWT secret key */ getJwtSecret() {
-        return this.config.jwtSecret ?? 'default-secret-change-in-production';
+        if (!this.config.jwtSecret) {
+            throw new _common.InternalServerErrorException('[AuthModule] JWT_SECRET is required - set via environment or module config');
+        }
+        return this.config.jwtSecret;
     }
     /** Get JWT expiration time */ getJwtExpiration() {
         return this.config.jwtExpiration ?? '1h';
     }
     /** Get refresh token secret key */ getRefreshTokenSecret() {
-        return this.config.refreshTokenSecret ?? this.getJwtSecret();
+        if (this.config.refreshTokenSecret) {
+            return this.config.refreshTokenSecret;
+        }
+        if (process.env.NODE_ENV === 'production') {
+            throw new _common.InternalServerErrorException('[AuthModule] REFRESH_TOKEN_SECRET is required in production - ' + 'using the same secret for access and refresh tokens is a security risk');
+        }
+        if (!this.refreshTokenSecretWarned) {
+            this.refreshTokenSecretWarned = true;
+            this.logger.warn('REFRESH_TOKEN_SECRET not configured - using JWT_SECRET as fallback. ' + 'For better security, configure a separate refresh token secret.');
+        }
+        return this.getJwtSecret();
     }
     /** Get refresh token expiration time */ getRefreshTokenExpiration() {
         return this.config.refreshTokenExpiration ?? '7d';
@@ -70,30 +83,11 @@ let AuthConfigService = class AuthConfigService {
     /** Get frontend URL for email links */ getFrontendUrl() {
         return this.config.frontendUrl;
     }
-    // Database Configuration
-    /** Get default database config */ getDefaultDatabaseConfig() {
-        return this.config.defaultDatabaseConfig ?? this.config.tenantDefaultDatabaseConfig;
-    }
-    /** Get tenant default database config (for multi-tenant) */ getTenantDefaultDatabaseConfig() {
-        return this.config.tenantDefaultDatabaseConfig;
-    }
-    /** Get registered tenants */ getTenants() {
-        return this.config.tenants ?? [];
-    }
-    // Raw Options
-    /** Get full module options */ getOptions() {
-        return {
-            ...this.options
-        };
-    }
-    /** Get runtime config */ getConfig() {
-        return {
-            ...this.config
-        };
-    }
     constructor(injectedOptions){
+        _define_property(this, "logger", new _common.Logger(AuthConfigService.name));
         _define_property(this, "options", void 0);
         _define_property(this, "config", void 0);
+        _define_property(this, "refreshTokenSecretWarned", false);
         this.options = injectedOptions ?? {
             global: false,
             includeController: true

package/cjs/services/auth-datasource.provider.js

@@ -92,7 +92,7 @@ let AuthDataSourceProvider = class AuthDataSourceProvider extends _modules.Multi
             const lockPromise = (async ()=>{
                 const config = this.getDefaultDatabaseConfig();
                 if (!config) {
-                    throw new Error('Default database config is not available');
+                    throw new _common.InternalServerErrorException('Default database config is not available');
                 }
                 const ds = await this.createDataSourceFromConfig(config);
                 AuthDataSourceProvider.singleDataSource = ds;
@@ -141,40 +141,28 @@ let AuthDataSourceProvider = class AuthDataSourceProvider extends _modules.Multi
         };
     }
     // Feature Flags
-    /** Get global enable company feature flag */ getEnableCompanyFeature() {
-        return this.authOptions.bootstrapAppConfig?.enableCompanyFeature ?? false;
-    }
-    /** Get enable company feature for specific tenant. Falls back to global setting. */ getEnableCompanyFeatureForTenant(tenant) {
-        return tenant?.enableCompanyFeature ?? this.getEnableCompanyFeature();
-    }
-    /** Get enable company feature for current request context */ getEnableCompanyFeatureForCurrentTenant() {
-        return this.getEnableCompanyFeatureForTenant(this.getCurrentTenant() ?? undefined);
+    /** Check if company feature is enabled (tenant-specific or global fallback) */ isCompanyFeatureEnabled(tenant) {
+        const effectiveTenant = tenant ?? this.getCurrentTenant();
+        return effectiveTenant?.enableCompanyFeature ?? this.authOptions.bootstrapAppConfig?.enableCompanyFeature ?? false;
     }
     // Entity Management
-    /** Get auth entities based on company feature flag */ async getAuthEntities(enableCompanyFeature) {
-        const enable = enableCompanyFeature ?? this.getEnableCompanyFeature();
+    /** Get auth entities based on company feature flag */ async getAuthEntities() {
         const { Company, CompanyBranch, User, UserCompanyPermission, UserToken, PendingRegistration } = await Promise.resolve().then(()=>/*#__PURE__*/ _interop_require_wildcard(require("../entities")));
         const coreEntities = [
             User,
             UserToken,
             PendingRegistration
         ];
-        const companyEntities = enable ? [
+        return this.isCompanyFeatureEnabled() ? [
+            ...coreEntities,
             Company,
             CompanyBranch,
             UserCompanyPermission
-        ] : [];
-        return [
-            ...coreEntities,
-            ...companyEntities
-        ];
+        ] : coreEntities;
     }
     // Overrides
     /** Override to dynamically set entities based on tenant config */ async createDataSourceFromConfig(config) {
-        const currentTenant = this.getCurrentTenant();
-        const enableCompanyFeature = this.getEnableCompanyFeatureForTenant(currentTenant ?? undefined);
-        const entities = await this.getAuthEntities(enableCompanyFeature);
-        return super.createDataSourceFromConfig(config, entities);
+        return super.createDataSourceFromConfig(config, await this.getAuthEntities());
     }
     constructor(authOptions, request){
         super(AuthDataSourceProvider.buildParentOptions(authOptions), request), _define_property(this, "authOptions", void 0), _define_property(this, "logger", void 0), this.authOptions = authOptions, this.logger = new _common.Logger(AuthDataSourceProvider.name);