@flusys/nestjs-storage 3.0.0 → 4.0.0-rc

package/fesm/providers/local-provider.js

@@ -1,15 +1,4 @@
-/**
- * LOCAL File System Provider
- *
- * This provider uses Node.js built-in 'fs' module - NO external dependencies required!
- * Perfect for same-server storage scenarios.
- *
- * Usage:
- * import { LocalProvider } from '@flusys/nestjs-storage/providers/local-provider';
- * import { StorageProviderRegistry, FileLocationEnum } from '@flusys/nestjs-storage';
- *
- * StorageProviderRegistry.register(FileLocationEnum.LOCAL, LocalProvider);
- */ function _define_property(obj, key, value) {
+function _define_property(obj, key, value) {
     if (key in obj) {
         Object.defineProperty(obj, key, {
             value: value,
@@ -22,8 +11,20 @@
     }
     return obj;
 }
+/**
+ * LOCAL File System Provider
+ *
+ * This provider uses Node.js built-in 'fs' module - NO external dependencies required!
+ * Perfect for same-server storage scenarios.
+ *
+ * Usage:
+ * import { LocalProvider } from '@flusys/nestjs-storage/providers/local-provider';
+ * import { StorageProviderRegistry, FileLocationEnum } from '@flusys/nestjs-storage';
+ *
+ * StorageProviderRegistry.register(FileLocationEnum.LOCAL, LocalProvider);
+ */ import { BadRequestException } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { ImageCompressor } from '../utils/image-compressor.util';
-import { Logger } from '@nestjs/common';
 import * as fs from 'fs/promises';
 import * as path from 'path';
 import { v4 as uuidv4 } from 'uuid';
@@ -39,8 +40,10 @@ import { v4 as uuidv4 } from 'uuid';
         const normalizedBasePath = path.resolve(this.basePath);
         const normalizedTargetPath = path.resolve(targetPath);
         if (!normalizedTargetPath.startsWith(normalizedBasePath + path.sep) && normalizedTargetPath !== normalizedBasePath) {
-            this.logger.warn(`Path traversal attempt detected: ${targetPath}`);
-            throw new Error('Invalid path: Path traversal attempt detected');
+            throw new BadRequestException({
+                message: 'Invalid path: Path traversal attempt detected',
+                messageKey: SYSTEM_MESSAGES.PATH_TRAVERSAL_DETECTED
+            });
         }
     }
     /**
@@ -48,10 +51,16 @@ import { v4 as uuidv4 } from 'uuid';
    * @throws Error if key contains suspicious patterns
    */ validateKeyFormat(key) {
         if (!key || typeof key !== 'string' || key.trim().length === 0) {
-            throw new Error('Invalid file key: empty or invalid');
+            throw new BadRequestException({
+                message: 'Invalid file key: empty or invalid',
+                messageKey: SYSTEM_MESSAGES.INVALID_FILE_KEY
+            });
         }
         if (key.includes('\0')) {
-            throw new Error('Invalid file key: contains null bytes');
+            throw new BadRequestException({
+                message: 'Invalid file key: contains null bytes',
+                messageKey: SYSTEM_MESSAGES.INVALID_FILE_KEY
+            });
         }
     }
     /**
@@ -68,7 +77,6 @@ import { v4 as uuidv4 } from 'uuid';
         await fs.mkdir(this.basePath, {
             recursive: true
         });
-        this.logger.log(`Local Provider initialized: ${this.basePath} (relative: ${this.relativeBasePath})`);
     }
     async uploadFile(file, options) {
         let processedBuffer = file.buffer;
@@ -101,7 +109,6 @@ import { v4 as uuidv4 } from 'uuid';
         // Generate key that includes the base path (relative to cwd)
         // This makes the key self-contained for serving files
         const relativeKey = path.join(this.relativeBasePath, options.folderPath || '', fileName).replace(/\\/g, '/'); // Normalize path separators
-        this.logger.log(`Uploaded file to local storage: ${relativeKey}`);
         return {
             name: fileName,
             key: relativeKey,
@@ -131,26 +138,22 @@ import { v4 as uuidv4 } from 'uuid';
                 try {
                     await fs.access(fallbackPath);
                     filePath = fallbackPath;
-                    this.logger.debug(`Using fallback path for delete: ${fallbackPath}`);
                 } catch  {
-                    // File doesn't exist at either location
-                    this.logger.warn(`File not found for deletion: ${key}`);
+                    // File doesn't exist at either location - silent return
                     return;
                 }
             }
             await fs.unlink(filePath);
-            this.logger.log(`Deleted file from local storage: ${key}`);
         } catch (error) {
             // Re-throw security errors
             if (error instanceof Error && error.message.includes('Invalid')) {
                 throw error;
             }
-            this.logger.warn(`Failed to delete file from local storage: ${key}`);
+        // Silent failure for other errors
         }
     }
     async deleteMultipleFiles(keys) {
         await Promise.all(keys.map((key)=>this.deleteFile(key)));
-        this.logger.log(`Deleted ${keys.length} files from local storage`);
     }
     async generatePresignedUrl(key, _expiresInSeconds) {
         // Return public URL or relative path
@@ -172,7 +175,6 @@ import { v4 as uuidv4 } from 'uuid';
         }
     }
     constructor(){
-        _define_property(this, "logger", new Logger(LocalProvider.name));
         _define_property(this, "basePath", '');
         _define_property(this, "baseUrl", '');
         _define_property(this, "relativeBasePath", ''); // Path relative to cwd for key generation

package/fesm/providers/s3-provider.optional.js

@@ -14,7 +14,8 @@ function _define_property(obj, key, value) {
 /**
  * Optional AWS S3 Storage Provider
  * Requires: npm install @aws-sdk/client-s3 @aws-sdk/lib-storage @aws-sdk/s3-request-presigner
- */ import { Logger } from '@nestjs/common';
+ */ import { InternalServerErrorException } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { v4 as uuidv4 } from 'uuid';
 import { ImageCompressor } from '../utils/image-compressor.util';
 export class S3Provider {
@@ -31,9 +32,14 @@ export class S3Provider {
                     secretAccessKey: config.secretAccessKey
                 } : undefined
             });
-            this.logger.log(`S3 Provider initialized: region=${config.region}, bucket=${config.bucket}`);
         } catch  {
-            throw new Error('AWS SDK not found. Install: npm install @aws-sdk/client-s3 @aws-sdk/lib-storage @aws-sdk/s3-request-presigner');
+            throw new InternalServerErrorException({
+                message: 'AWS SDK not found',
+                messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED,
+                messageParams: {
+                    sdk: '@aws-sdk/client-s3'
+                }
+            });
         }
     }
     async uploadFile(file, options) {
@@ -77,7 +83,6 @@ export class S3Provider {
             Bucket: this.bucketName,
             Key: key
         }));
-        this.logger.log(`Deleted file from S3: ${key}`);
     }
     async deleteMultipleFiles(keys) {
         const { DeleteObjectsCommand } = await import('@aws-sdk/client-s3');
@@ -89,7 +94,6 @@ export class S3Provider {
                     }))
             }
         }));
-        this.logger.log(`Deleted ${keys.length} files from S3`);
     }
     async generatePresignedUrl(key, expiresInSeconds = 3600) {
         const { GetObjectCommand } = await import('@aws-sdk/client-s3');
@@ -113,7 +117,6 @@ export class S3Provider {
         }
     }
     constructor(){
-        _define_property(this, "logger", new Logger(S3Provider.name));
         _define_property(this, "s3", void 0);
         _define_property(this, "bucketName", '');
         _define_property(this, "region", '');

package/fesm/providers/sftp-provider.optional.js

@@ -1,3 +1,16 @@
+function _define_property(obj, key, value) {
+    if (key in obj) {
+        Object.defineProperty(obj, key, {
+            value: value,
+            enumerable: true,
+            configurable: true,
+            writable: true
+        });
+    } else {
+        obj[key] = value;
+    }
+    return obj;
+}
 /**
  * OPTIONAL SFTP Provider
  *
@@ -13,21 +26,9 @@
  * import { StorageProviderRegistry, FileLocationEnum } from '@flusys/nestjs-storage';
  *
  * StorageProviderRegistry.register(FileLocationEnum.SFTP, SftpProvider);
- */ function _define_property(obj, key, value) {
-    if (key in obj) {
-        Object.defineProperty(obj, key, {
-            value: value,
-            enumerable: true,
-            configurable: true,
-            writable: true
-        });
-    } else {
-        obj[key] = value;
-    }
-    return obj;
-}
+ */ import { InternalServerErrorException } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { ImageCompressor } from '../utils/image-compressor.util';
-import { Logger } from '@nestjs/common';
 import * as path from 'path';
 import { v4 as uuidv4 } from 'uuid';
 /**
@@ -54,10 +55,14 @@ import { v4 as uuidv4 } from 'uuid';
             await this.connect();
             // Ensure base directory exists
             await this.ensureDirectory(this.config.basePath);
-            this.logger.log(`SFTP Provider initialized: ${config.username}@${config.host}:${config.port || 22}`);
-        } catch (_error) {
-            this.logger.error('Failed to initialize SftpProvider. Make sure ssh2-sftp-client is installed.');
-            throw new Error('SFTP client not found. Install it with: npm install ssh2-sftp-client');
+        } catch  {
+            throw new InternalServerErrorException({
+                message: 'SFTP client not found',
+                messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED,
+                messageParams: {
+                    sdk: 'ssh2-sftp-client'
+                }
+            });
         }
     }
     async connect() {
@@ -75,7 +80,7 @@ import { v4 as uuidv4 } from 'uuid';
     async ensureDirectory(dirPath) {
         try {
             await this.sftp.mkdir(dirPath, true);
-        } catch (_error) {
+        } catch  {
         // Directory might already exist
         }
     }
@@ -115,14 +120,12 @@ import { v4 as uuidv4 } from 'uuid';
         await this.connect();
         try {
             await this.sftp.delete(key);
-            this.logger.log(`Deleted file from SFTP: ${key}`);
-        } catch (_error) {
-            this.logger.warn(`Failed to delete file from SFTP: ${key}`);
+        } catch  {
+        // Silent failure
         }
     }
     async deleteMultipleFiles(keys) {
         await Promise.all(keys.map((key)=>this.deleteFile(key)));
-        this.logger.log(`Deleted ${keys.length} files from SFTP`);
     }
     async generatePresignedUrl(key, _expiresInSeconds) {
         // SFTP doesn't support presigned URLs, return the key
@@ -145,7 +148,6 @@ import { v4 as uuidv4 } from 'uuid';
         }
     }
     constructor(){
-        _define_property(this, "logger", new Logger(SftpProvider.name));
         _define_property(this, "sftp", void 0); // SftpClient
         _define_property(this, "config", void 0);
         _define_property(this, "connected", false);

package/fesm/providers/storage-factory.service.js

@@ -25,8 +25,9 @@ function _ts_param(paramIndex, decorator) {
         decorator(target, key, paramIndex);
     };
 }
-import { Inject, Injectable, Logger, NotFoundException } from '@nestjs/common';
+import { Inject, Injectable, InternalServerErrorException, NotFoundException } from '@nestjs/common';
 import * as crypto from 'crypto';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { StorageConfigService } from '../services';
 import { StorageProviderRegistry } from './storage-provider.registry';
 export class StorageFactoryService {
@@ -36,17 +37,30 @@ export class StorageFactoryService {
         if (cached) return cached;
         const ProviderClass = StorageProviderRegistry.get(config.provider);
         if (!ProviderClass) {
-            throw new NotFoundException(`Storage provider '${config.provider}' not registered. Available: ${StorageProviderRegistry.getAll().join(', ')}`);
+            throw new NotFoundException({
+                message: `Storage provider '${config.provider}' not registered. Available: ${StorageProviderRegistry.getAll().join(', ')}`,
+                messageKey: SYSTEM_MESSAGES.SERVICE_NOT_AVAILABLE,
+                messageParams: {
+                    provider: config.provider,
+                    available: StorageProviderRegistry.getAll().join(', ')
+                }
+            });
         }
         try {
             const instance = new ProviderClass();
             await this.initializeProvider(instance, config);
             this.cache.set(cacheKey, instance);
-            this.logger.log(`Created provider: ${config.provider} (${cacheKey})`);
             return instance;
         } catch (error) {
-            const message = error instanceof Error ? error.message : 'Unknown error';
-            throw new Error(`Failed to initialize '${config.provider}': ${message}`);
+            const errorMessage = error instanceof Error ? error.message : 'Unknown error';
+            throw new InternalServerErrorException({
+                message: `Failed to initialize '${config.provider}': ${errorMessage}`,
+                messageKey: SYSTEM_MESSAGES.INTERNAL_ERROR,
+                messageParams: {
+                    provider: config.provider,
+                    error: errorMessage
+                }
+            });
         }
     }
     async getProvider(providerName) {
@@ -66,11 +80,9 @@ export class StorageFactoryService {
         this.cache.clear();
     }
     async onModuleDestroy() {
-        this.logger.log('Cleaning up storage providers...');
-        const closePromises = Array.from(this.cache.entries()).filter(([, p])=>'close' in p && typeof p.close === 'function').map(([key, p])=>p.close().then(()=>this.logger.debug(`Closed: ${key}`)).catch((e)=>this.logger.warn(`Failed to close ${key}: ${e.message}`)));
+        const closePromises = Array.from(this.cache.entries()).filter(([, p])=>'close' in p && typeof p.close === 'function').map(([, p])=>p.close().catch(()=>{}));
         await Promise.allSettled(closePromises);
         this.cache.clear();
-        this.logger.log('Storage provider cleanup complete');
     }
     // ─── Private Helpers ──────────────────────────────────────────────────────────
     generateCacheKey(config) {
@@ -91,17 +103,14 @@ export class StorageFactoryService {
                     ...config.config,
                     baseUrl: appUrl
                 };
-                this.logger.debug(`Using appUrl as baseUrl: ${appUrl}`);
             }
         }
         await instance.initialize(initConfig);
     }
     constructor(configService){
         _define_property(this, "configService", void 0);
-        _define_property(this, "logger", void 0);
         _define_property(this, "cache", void 0);
         this.configService = configService;
-        this.logger = new Logger(StorageFactoryService.name);
         this.cache = new Map();
     }
 }

package/fesm/services/file-manager.service.js

@@ -27,7 +27,8 @@ function _ts_param(paramIndex, decorator) {
 }
 import { HybridCache, RequestScopedApiService } from '@flusys/nestjs-shared/classes';
 import { UtilsService } from '@flusys/nestjs-shared/modules';
-import { applyCompanyFilter, ErrorHandler } from '@flusys/nestjs-shared/utils';
+import { applyCompanyFilter } from '@flusys/nestjs-shared/utils';
+import { FILE_MESSAGES, FOLDER_MESSAGES, UPLOAD_MESSAGES } from '../config';
 import { BadRequestException, Inject, Injectable, NotFoundException, Scope } from '@nestjs/common';
 import { Brackets, In } from 'typeorm';
 import { StorageConfigService } from './storage-config.service';
@@ -67,7 +68,10 @@ export class FileManagerService extends RequestScopedApiService {
                     id: dto.id
                 }
             });
-            if (!existing) throw new NotFoundException('Entity not found for update');
+            if (!existing) throw new NotFoundException({
+                message: 'Entity not found for update',
+                messageKey: FILE_MESSAGES.NOT_FOUND
+            });
             entity = existing;
         }
         const validatedFolder = dto.folderId ? await this.validateFolder(dto.folderId, user) : null;
@@ -97,7 +101,7 @@ export class FileManagerService extends RequestScopedApiService {
     }
     async getFilterQuery(query, filter, _user) {
         for (const [key, value] of Object.entries(filter)){
-            if (key === 'contentType') {
+            if (key === 'contentType' && typeof value === 'string') {
                 this.applyContentTypeFilter(query, value);
             } else {
                 query.andWhere(`${this.entityName}.${key} = :value`, {
@@ -169,8 +173,7 @@ export class FileManagerService extends RequestScopedApiService {
                     ...item,
                     providerName: item.storageConfigId ? nameMap.get(item.storageConfigId) : undefined
                 }));
-        } catch (error) {
-            this.logger.warn(`Failed to fetch provider names: ${ErrorHandler.getErrorMessage(error)}`);
+        } catch  {
             return items.map((item)=>({
                     ...item,
                     providerName: undefined
@@ -180,7 +183,10 @@ export class FileManagerService extends RequestScopedApiService {
     async getFiles(dtos, protocol, host, user) {
         await this.ensureRepositoryInitialized();
         const ids = dtos.map((d)=>d.id).filter(Boolean);
-        if (!ids.length) throw new BadRequestException('No valid file IDs provided');
+        if (!ids.length) throw new BadRequestException({
+            message: 'No valid file IDs provided',
+            messageKey: UPLOAD_MESSAGES.NO_FILES_PROVIDED
+        });
         const files = await this.repository.findBy({
             id: In(ids)
         });
@@ -194,17 +200,13 @@ export class FileManagerService extends RequestScopedApiService {
         if (updatedFiles.length) {
             try {
                 await this.repository.save(updatedFiles);
-            } catch (error) {
-                this.logger.error(`Failed to save updated URLs: ${ErrorHandler.getErrorMessage(error)}`);
+            } catch  {
+            // Silent failure - URL update is non-critical
             }
         }
         return responses;
     }
     // ─── Private Helpers ────────────────────────────────────────────────────────
-    async getStorageConfigRepository() {
-        const entity = this.storageConfig.isCompanyFeatureEnabled() ? (await import('../entities')).StorageConfigWithCompany : (await import('../entities')).StorageConfig;
-        return this.dataSourceProvider.getRepository(entity);
-    }
     buildWhereWithCompany(id, user) {
         const where = {
             id
@@ -214,6 +216,10 @@ export class FileManagerService extends RequestScopedApiService {
         }
         return where;
     }
+    async getStorageConfigRepository() {
+        const entity = this.storageConfig.isCompanyFeatureEnabled() ? (await import('../entities')).StorageConfigWithCompany : (await import('../entities')).StorageConfig;
+        return this.dataSourceProvider.getRepository(entity);
+    }
     async getStorageConfigBasePath(configId) {
         try {
             const repo = await this.getStorageConfigRepository();
@@ -227,8 +233,7 @@ export class FileManagerService extends RequestScopedApiService {
                 ]
             });
             return config?.config?.basePath?.replace(/^\.\//, '') ?? null;
-        } catch (error) {
-            this.logger.warn(`Failed to get basePath for ${configId}: ${ErrorHandler.getErrorMessage(error)}`);
+        } catch  {
             return null;
         }
     }
@@ -239,7 +244,13 @@ export class FileManagerService extends RequestScopedApiService {
             where: this.buildWhereWithCompany(folderId, user)
         });
         if (!folder) {
-            throw new BadRequestException(`Folder ${folderId} not found or access denied`);
+            throw new BadRequestException({
+                message: `Folder "${folderId}" not found or access denied`,
+                messageKey: FOLDER_MESSAGES.NOT_FOUND,
+                messageParams: {
+                    folderId
+                }
+            });
         }
         return folder;
     }
@@ -252,8 +263,7 @@ export class FileManagerService extends RequestScopedApiService {
                 where: this.buildWhereWithCompany(configId, user)
             });
             return config?.storage || dto.location || FileLocationEnum.LOCAL;
-        } catch (error) {
-            this.logger.warn(`Failed to resolve storage location: ${ErrorHandler.getErrorMessage(error)}`);
+        } catch  {
             return dto.location || FileLocationEnum.LOCAL;
         }
     }
@@ -299,12 +309,10 @@ export class FileManagerService extends RequestScopedApiService {
         return grouped;
     }
     getFileBaseUrl(protocol, host) {
-        return this.storageConfig.getAppUrl()?.replace(/\/$/, '') ?? `${protocol}://${host}`;
+        const appUrl = this.storageConfig.getAppUrl()?.replace(/\/$/, '');
+        return appUrl || `${protocol}://${host}`;
     }
     async refreshFileUrl(file, protocol, host, now, user) {
-        if (!file.storageConfigId) {
-            this.logger.warn(`File ${file.id} has no storageConfigId`);
-        }
         const isCloudProvider = file.location === FileLocationEnum.AWS || file.location === FileLocationEnum.AZURE;
         const needsNewUrl = !file.url || isCloudProvider && (typeof file.expiresAt !== 'number' || now >= file.expiresAt);
         if (needsNewUrl && file.storageConfigId) {
@@ -312,8 +320,7 @@ export class FileManagerService extends RequestScopedApiService {
                 file.url = await this.uploadService.makeFileUrl(file.key, file.storageConfigId, URL_EXPIRY_SECONDS, user);
                 file.expiresAt = now + URL_EXPIRY_SECONDS * 1000;
                 return true;
-            } catch (error) {
-                this.logger.error(`Failed to generate URL for ${file.id}: ${ErrorHandler.getErrorMessage(error)}`);
+            } catch  {
                 file.url = `${this.getFileBaseUrl(protocol, host)}/storage/upload/file/${file.key}`;
             }
         }
@@ -342,7 +349,7 @@ export class FileManagerService extends RequestScopedApiService {
         };
     }
     constructor(cacheManager, utilsService, uploadService, storageConfig, dataSourceProvider){
-        super('file_manager', null, cacheManager, utilsService, FileManagerService.name, true), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "uploadService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.uploadService = uploadService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
+        super('file_manager', null, cacheManager, utilsService, FileManagerService.name, true, 'storage'), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "uploadService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.uploadService = uploadService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
     }
 }
 FileManagerService = _ts_decorate([

package/fesm/services/folder.service.js

@@ -39,6 +39,7 @@ export class FolderService extends RequestScopedApiService {
     getDataSourceProvider() {
         return this.dataSourceProvider;
     }
+    // ─── Override Methods ───────────────────────────────────────────────────────
     async convertSingleDtoToEntity(dto, user) {
         const entity = await super.convertSingleDtoToEntity(dto, user);
         if (this.storageConfig.isCompanyFeatureEnabled()) {
@@ -74,7 +75,7 @@ export class FolderService extends RequestScopedApiService {
         return result;
     }
     constructor(cacheManager, utilsService, storageConfig, dataSourceProvider){
-        super('folder', null, cacheManager, utilsService, FolderService.name, true), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
+        super('folder', null, cacheManager, utilsService, FolderService.name, true, 'storage'), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
     }
 }
 FolderService = _ts_decorate([

package/fesm/services/storage-datasource.provider.js

@@ -26,7 +26,8 @@ function _ts_param(paramIndex, decorator) {
     };
 }
 import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
-import { Inject, Injectable, Logger, Optional, Scope } from '@nestjs/common';
+import { Inject, Injectable, InternalServerErrorException, Optional, Scope } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 import { REQUEST } from '@nestjs/core';
 import { Request } from 'express';
 import { StorageConfigService } from './storage-config.service';
@@ -90,7 +91,10 @@ export class StorageDataSourceProvider extends MultiTenantDataSourceService {
         }
         const config = this.getDefaultDatabaseConfig();
         if (!config) {
-            throw new Error('No database config available. Provide defaultDatabaseConfig or tenantDefaultDatabaseConfig.');
+            throw new InternalServerErrorException({
+                message: 'No database config available. Provide defaultDatabaseConfig or tenantDefaultDatabaseConfig.',
+                messageKey: SYSTEM_MESSAGES.DATABASE_CONFIG_NOT_AVAILABLE
+            });
         }
         // Create connection with lock to prevent race conditions
         const connectionPromise = this.createDataSourceFromConfig(config);
@@ -129,7 +133,7 @@ export class StorageDataSourceProvider extends MultiTenantDataSourceService {
         }
     }
     constructor(configService, request){
-        super(StorageDataSourceProvider.buildParentOptions(configService.getOptions()), request), _define_property(this, "configService", void 0), _define_property(this, "logger", void 0), this.configService = configService, this.logger = new Logger(StorageDataSourceProvider.name);
+        super(StorageDataSourceProvider.buildParentOptions(configService.getOptions()), request), _define_property(this, "configService", void 0), this.configService = configService;
     }
 }
 // Override parent's static properties to have Storage-specific cache

package/fesm/services/storage-provider-config.service.js

@@ -39,6 +39,7 @@ export class StorageProviderConfigService extends RequestScopedApiService {
     getDataSourceProvider() {
         return this.dataSourceProvider;
     }
+    // ─── Override Methods ───────────────────────────────────────────────────────
     async convertSingleDtoToEntity(dto, user) {
         const entity = await super.convertSingleDtoToEntity(dto, user);
         if (this.storageConfig.isCompanyFeatureEnabled()) {
@@ -120,7 +121,7 @@ export class StorageProviderConfigService extends RequestScopedApiService {
         });
     }
     constructor(cacheManager, utilsService, storageConfig, dataSourceProvider){
-        super('storageConfig', null, cacheManager, utilsService, StorageProviderConfigService.name, true), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
+        super('storageConfig', null, cacheManager, utilsService, StorageProviderConfigService.name, true, 'storage'), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "storageConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.storageConfig = storageConfig, this.dataSourceProvider = dataSourceProvider;
     }
 }
 StorageProviderConfigService = _ts_decorate([