fpavon-ee-shared 1.0.25 → 1.0.26

package/dist/infrastructure/middlewares/middleware.auth.js

@@ -50,17 +50,9 @@ function getSecret() {
     return Buffer.from(secretString, 'utf8');
 }
 const asyncHandlerWrapper = (req, res, next) => {
-    // Asume que 'fn' es tu función asíncrona original (la que tiene el Promise<void>)
-    // Si 'fn' ya está definida en algún lugar, asegúrate de llamarla aquí.
-    // Ejemplo si 'fn' fuera el middleware asíncrono original:
     Promise.resolve(validarToken(req, res, next)).catch(next);
-    // **Importante**: Si el error persiste, el problema es que la firma de 
-    // la función envuelta (fn) no es compatible con el RequestHandler que espera Express.
-    // La solución más segura es usar una librería conocida de tipos compatibles si es posible.
 };
 exports.validarToken = asyncHandlerWrapper;
-// NOTA: Es esencial que getSecret() ahora devuelva un Buffer para jwt.verify/sign
-//       y que dotenv.config() esté en la PRIMERA LÍNEA del cliente.
 const validarToken = (req, res, next) => __awaiter(void 0, void 0, void 0, function* () {
     // --- BLOQUE 1: Extracción y Validación del Token ---
     try {

package/dist/infrastructure/middlewares/middleware.validarRol.d.ts

@@ -1 +1,2 @@
-export declare function validarRol(rolRequerido: string): (req: Request, res: Response, next: any) => any;
+import { RequestHandler } from 'express';
+export declare function validarRol(rolRequerido: string): RequestHandler;

package/dist/infrastructure/middlewares/middleware.validarRol.js

@@ -5,6 +5,10 @@ function validarRol(rolRequerido) {
     if (typeof rolRequerido != 'string')
         throw new Error("Debe enviar un rol requerido");
     return function (req, res, next) {
+        // Aseguramos req.body exista, como hicimos antes
+        if (!req.body || !req.body.usuario || !req.body.usuario.roles) {
+            return res.status(401).json({ "error": "Datos de usuario insuficientes o token inválido." });
+        }
         try {
             if (!req.body.usuario.roles.includes(rolRequerido)) {
                 return res.status(401).json({ "error": "No tiene permisos para usar esta funcionalidad" });
@@ -12,7 +16,9 @@ function validarRol(rolRequerido) {
             next();
         }
         catch (error) {
-            return res.status(500).json({ "error": `${error}` });
+            // Se usa el error.message si existe para un mejor log
+            console.error("FALLO CRÍTICO en validarRol:", error);
+            return res.status(500).json({ "error": `Error interno en el servidor: ${error.message || error}` });
         }
     };
 }

package/infrastructure/middlewares/middleware.auth.ts

@@ -18,20 +18,9 @@ function getSecret(): Buffer {
 }
 
 const asyncHandlerWrapper: RequestHandler = (req: Request, res: Response, next: NextFunction) => {
-    // Asume que 'fn' es tu función asíncrona original (la que tiene el Promise<void>)
-    // Si 'fn' ya está definida en algún lugar, asegúrate de llamarla aquí.
-    
-    // Ejemplo si 'fn' fuera el middleware asíncrono original:
     Promise.resolve(validarToken(req, res, next)).catch(next); 
-    
-    // **Importante**: Si el error persiste, el problema es que la firma de 
-    // la función envuelta (fn) no es compatible con el RequestHandler que espera Express.
-    // La solución más segura es usar una librería conocida de tipos compatibles si es posible.
 };
 
-// NOTA: Es esencial que getSecret() ahora devuelva un Buffer para jwt.verify/sign
-//       y que dotenv.config() esté en la PRIMERA LÍNEA del cliente.
-
 const validarToken = async (req: Request, res: Response, next: NextFunction): Promise<void> => {
     
     // --- BLOQUE 1: Extracción y Validación del Token ---

package/infrastructure/middlewares/middleware.validarRol.ts

@@ -1,15 +1,24 @@
-import { Request, Response } from 'express';
+import { Request, Response, NextFunction, RequestHandler } from 'express';
 
-export function validarRol(rolRequerido:string) {
-    if(typeof rolRequerido != 'string') throw new Error("Debe enviar un rol requerido");    
-    return function(req: Request, res:Response, next) {
+
+export function validarRol(rolRequerido: string): RequestHandler { 
+    if (typeof rolRequerido != 'string') throw new Error("Debe enviar un rol requerido");
+
+    return function(req: Request, res: Response, next: NextFunction) { 
+        // Aseguramos req.body exista, como hicimos antes
+        if (!req.body || !req.body.usuario || !req.body.usuario.roles) {
+             return res.status(401).json({ "error": "Datos de usuario insuficientes o token inválido." });
+        }
+        
         try {
-            if(!req.body.usuario.roles.includes(rolRequerido)){
-                return res.status(401).json({"error":"No tiene permisos para usar esta funcionalidad"});
+            if (!req.body.usuario.roles.includes(rolRequerido)) {
+                return res.status(401).json({ "error": "No tiene permisos para usar esta funcionalidad" });
             }
             next();
-        } catch (error) {
-            return res.status(500).json({"error":`${error}`});
+        } catch (error:any) {
+            // Se usa el error.message si existe para un mejor log
+            console.error("FALLO CRÍTICO en validarRol:", error);
+            return res.status(500).json({ "error": `Error interno en el servidor: ${error.message || error}` });
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fpavon-ee-shared",
-  "version": "1.0.25",
+  "version": "1.0.26",
   "description": "Carpeta compartida entre servicios de Expediente Electronio",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",