refactor(logging): replace console logging with centralized logger utility

- Updated all `console.error` and `console.warn` calls to use `logger.error` and `logger.warn` respectively.
- Integrated `fileLogger` utility into `fileManager`, `middleware`, and `routes` to ensure standardized logging.

Author: solan117

Servers/middleware/rateLimit.middleware.ts

@@ -15,6 +15,7 @@
 import rateLimit, { Options } from 'express-rate-limit';
 import { STATUS_CODE } from '../utils/statusCode.utils';
 import { Request, Response } from 'express';
+import logger from '../utils/logger/fileLogger';
 
 /**
  * Rate limit configuration with time window and request limits
@@ -53,7 +54,7 @@ const RATE_LIMIT_CONFIGS: Record<string, RateLimitConfig> = {
 const createRateLimitHandler = (message: string) => {
   return (req: Request, res: Response) => {
     const clientIp = req.ip || req.socket?.remoteAddress || 'unknown';
-    console.warn(`Rate limit exceeded for IP ${clientIp}: ${message}`);
+    logger.warn(`Rate limit exceeded for IP ${clientIp} on ${req.path}: ${message}`);
     res.status(429).json(STATUS_CODE[429](message));
   };
 };

Servers/routes/fileManager.route.ts

@@ -27,6 +27,7 @@ import { STATUS_CODE } from "../utils/statusCode.utils";
 import * as path from "path";
 import * as fs from "fs";
 import { ALLOWED_MIME_TYPES } from "../utils/validations/fileManagerValidation.utils";
+import logger from "../utils/logger/fileLogger";
 
 const router = express.Router();
 
@@ -100,16 +101,16 @@ const handleMulterError = (err: any, req: Request, res: Response, next: NextFunc
         fs.promises.unlink(resolvedPath).catch((cleanupError) => {
           // Ignore ENOENT (file already deleted), but log other errors
           if (cleanupError.code !== 'ENOENT') {
-            console.error("Failed to clean up temporary file:", cleanupError);
+            logger.error("Failed to clean up temporary file:", cleanupError);
           }
         });
       } else {
         // Log security violation attempt
-        console.warn("Security: Blocked cleanup attempt outside temp directory. Path: %s, Allowed: %s", resolvedPath, resolvedTempDir);
+        logger.warn(`Security: Blocked cleanup attempt outside temp directory. Path: ${resolvedPath}, Allowed: ${resolvedTempDir}`);
       }
     } catch (e) {
       // Unable to resolve file/directory (file may not exist), skip cleanup and continue
-      console.warn("Failed to resolve path for cleanup: %s", req.file.path, e);
+      logger.warn(`Failed to resolve path for cleanup: ${req.file.path}`, e);
       // Do not return - continue to error handling below
     }
   }

Servers/utils/fileManager.utils.ts

@@ -19,6 +19,7 @@ import * as fs from "fs";
 import { promisify } from "util";
 import { randomBytes } from "crypto";
 import { sanitizeFilename } from "./validations/fileManagerValidation.utils";
+import logger from "./logger/fileLogger";
 
 const writeFile = promisify(fs.writeFile);
 const mkdir = promisify(fs.mkdir);
@@ -106,7 +107,7 @@ export const uploadFileToManager = async (
     } catch (unlinkError: any) {
       // Ignore ENOENT (file already deleted), but log other errors
       if (unlinkError.code !== 'ENOENT') {
-        console.error(`Failed to cleanup orphaned file after DB error: ${permanentFilePath}`, unlinkError);
+        logger.error(`Failed to cleanup orphaned file after DB error: ${permanentFilePath}`, unlinkError);
       }
     }
     // Rethrow original database error
@@ -289,7 +290,7 @@ export const deleteFile = async (fileId: number, tenant: string): Promise<boolea
 
   if (!isContained) {
     const error = `Security violation: Attempted to delete file outside tenant directory. Target: ${targetPath}, Base: ${baseDir}`;
-    console.error(error);
+    logger.error(error);
     throw new Error(error);
   }
 
@@ -303,12 +304,12 @@ export const deleteFile = async (fileId: number, tenant: string): Promise<boolea
   } catch (error: any) {
     // Only tolerate ENOENT (file already deleted/doesn't exist)
     if (error.code === 'ENOENT') {
-      console.warn(`Physical file already deleted or not found: ${targetPath}`);
+      logger.warn(`Physical file already deleted or not found: ${targetPath}`);
     } else {
       // Other errors are concerning - log and track them
       diskDeletionFailed = true;
       diskDeletionError = error;
-      console.error(`Failed to delete physical file: ${targetPath}`, error);
+      logger.error(`Failed to delete physical file: ${targetPath}`, error);
     }
   }