dcos-core-monalisav2-latam 1.0.1 → 1.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dcos-core-monalisav2-latam",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "devDependencies": {
   },
   "scripts": {
@@ -30,6 +30,7 @@
     "csv": "^6.3.0",
     "csv-parser": "^3.0.0",
     "csvtojson": "^2.0.10",
+    "dcos-sls-auth-app": "^1.0.1",
     "dotenv": "^10.0.0",
     "ftp": "^0.3.10",
     "json-2-csv": "^5.5.1",

package/src/functions/clients_peru/updateClients.js

@@ -45,7 +45,7 @@ module.exports.consumer = async (event, context) => {
     try {
         let { body } = event;
         body = JSON.parse(body);
-        await this.handleConsumer(ACCOUNTNAME, { process: processName, params: body });
+        return this.handleConsumer(ACCOUNTNAME, { process: processName, params: body });
     } catch (ex) {
         return ApiResponse.response(400, ex.message ?? 'Error in request');
     }

package/src/functions/multiPresentation/multipres.js

@@ -423,6 +423,7 @@ const producer = async (event) => {
     let statusCode = 200;
     let message;
 
+    Logger.log('Context', event?.requestContext);
     try {
         const clientId = event?.requestContext?.authorizer?.claims?.client_id;
         const scope = event?.requestContext?.authorizer?.claims?.scope;
@@ -431,7 +432,7 @@ const producer = async (event) => {
         if (!accountName) {
             return errorResponse(404, "AccountName not found");
         }
-        if (!scope.includes("admin")) {
+        if (scope && !scope?.includes("admin")) {
             const validateAccount = await ClientData.validateAccount(clientId, accountName);
             if (!validateAccount) {
                 Logger.error("Account not found: ", accountName, ". ClientId: ", clientId);

package/src/functions/orders/orders-status-producer.js

@@ -324,7 +324,7 @@ const validDocuments = async ({ event = null, orderClientVtexResponse = null })
 const validReturnedItems = async (request, orderClientVtexResponse) => {
   const { invoiceValue, returnItems } = request.documents;
 
-  if (returnItems.length == 0) {
+  if (returnItems?.length == 0) {
     return [`Return items must have at least one item`];
   }
 

package/src/functions/security/security-handler.js

@@ -22,7 +22,7 @@ const producer = async (event) => {
   try {
     // Determinar el tipo de operación basado en la ruta
     const operationType = getOperationType(event);
-    
+
     // Conexión a la API de Secret Manager
     const configAuth = await AWSServices.getSecretValue(MANAGER_STORE_KEYS);
     const auth = JSON.parse(configAuth);
@@ -265,19 +265,19 @@ function validateSenderAuthorizationAdmin(email, authorizedUsers, isAvailable) {
 function getFormData(event, availableExtensions) {
   const formData = parse(event, true);
 
-  let errors = validate(formData, {
-    'file': { presence: { allowEmpty: false } },
-    'file.type': { presence: { allowEmpty: false } },
-    'file.filename': { presence: { allowEmpty: false } },
-    'file.contentType': {
+  const { email, file } = formData;
+  const { type, filename, contentType } = file;
+  let errors = validate({ email, type, filename, contentType }, {
+    'email': { presence: { allowEmpty: true }, email: true },
+    'type': { presence: { allowEmpty: false } },
+    'filename': { presence: { allowEmpty: false } },
+    'contentType': {
       presence: { allowEmpty: false },
       inclusion: {
         within: availableExtensions,
         message: `^El archivo debe tener una extensión .csv o .txt. Verifique el formato de su archivo e inténtelo nuevamente.`
       }
-    },
-    'file.content': { presence: { allowEmpty: false } },
-    'email': { presence: { allowEmpty: true }, email: true }
+    }
   });
 
   if (errors) {
@@ -312,18 +312,18 @@ async function getSecretDataAccount(auth) {
     throw new Error(`Information from SM API not defined`);
   }
 
-  const { 
-    isAvailable = false, 
-    authorizedUsers = {}, 
-    availableExtensions = ['text/csv', 'text/plain'], 
-    accounts = [] 
+  const {
+    isAvailable = false,
+    authorizedUsers = {},
+    availableExtensions = ['text/csv', 'text/plain'],
+    accounts = []
   } = smData[0].value;
 
   // 1. Procesar cuentas: Mantener como ARRAY para usar .some()
   const accountsData = accounts
-    .filter(account => 
-      account.accountName && 
-      account.vtexAppkey && 
+    .filter(account =>
+      account.accountName &&
+      account.vtexAppkey &&
       account.vtexAppToken
     );
 
@@ -331,7 +331,7 @@ async function getSecretDataAccount(auth) {
   Object.keys(authorizedUsers).forEach(email => {
     const hasWildcard = authorizedUsers[email].hasOwnProperty('*');
     const { '*': wildcardUser, ...remainPerms } = authorizedUsers[email];
-    
+
     Object.keys(remainPerms).forEach(account => {
       if (remainPerms[account] === (hasWildcard ? wildcardUser : isAvailable)) {
         delete authorizedUsers[email][account];
@@ -339,11 +339,11 @@ async function getSecretDataAccount(auth) {
     });
   });
 
-  return { 
+  return {
     accountsData, // Ahora es un array filtrado y mapeado
-    isAvailable, 
-    authorizedUsers, 
-    availableExtensions 
+    isAvailable,
+    authorizedUsers,
+    availableExtensions
   };
 }
 
@@ -413,14 +413,14 @@ async function processAccountsCsv(fileData, accountsData, auth) {
     }
 
     const csvRows = csvDataString.trim().split(/\r?\n/);
-    
+
     // 2. Validación de headers
     if (csvRows.length === 0) {
       throw new Error("El archivo no contiene datos");
     }
 
     const headerRow = csvRows.shift();
-    
+
     // 3. Validación de existencia de headers
     if (!headerRow) {
       throw new Error("El archivo no contiene encabezados. Debe incluir la primera línea con los nombres de columna.");
@@ -435,7 +435,7 @@ async function processAccountsCsv(fileData, accountsData, auth) {
     const headers = headerRow.split(';').map(h => h.trim().toLowerCase());
     const expectedHeaders = ['action', 'accountname', 'appkey', 'apptoken'];
     const missingHeaders = expectedHeaders.filter(h => !headers.includes(h));
-    
+
     if (missingHeaders.length > 0) {
       throw new Error(`Encabezados incorrectos. Faltan: ${missingHeaders.join(', ')}. Formato esperado: ACTION;ACCOUNTNAME;APPKEY;APPTOKEN`);
     }
@@ -448,7 +448,7 @@ async function processAccountsCsv(fileData, accountsData, auth) {
 
     // Obtener configuración actual del Secret Manager
     const { currentConfig, secretManagerApi } = await getCurrentSecretManagerConfig(auth);
-    
+
     if (!Array.isArray(currentConfig.value.accounts)) {
       currentConfig.value.accounts = [];
     }
@@ -463,7 +463,7 @@ async function processAccountsCsv(fileData, accountsData, auth) {
         }
 
         const rowData = row.split(';');
-        
+
         // 6. Validación de campos vacíos
         const action = (rowData[columnIndices.action]?.trim() || 'add').toLowerCase();
         const accountName = rowData[columnIndices.accountname]?.trim();

package/src/functions/utils/index.js

@@ -50,7 +50,7 @@ module.exports.handler = async (req, res) => {
             });
         }
         break;
-        // route notification new leads
+      // route notification new leads
       case "/new-leads/notification/{an}":
         switch (method) {
           case "POST":
@@ -96,7 +96,7 @@ module.exports.handler = async (req, res) => {
         }
         break;
       case "/errors/{errorCode}":
-          return await AccountErrorsHandler.handler(req, res);
+        return await AccountErrorsHandler.handler(req, res);
         break;
       case "/accounts/properties":
       case "/accounts/properties/required":
@@ -104,15 +104,15 @@ module.exports.handler = async (req, res) => {
         break;
       // Multi presentations INIT
       case "/multipresentation/{accountName}":
-          return await multiPresentation?.producer(req, res);
+        return await multiPresentation?.producer(req, res);
         break;
       // Multi presentations END
       // Roles Security INIT
       case "/batch/users":
-          return await securityHandler?.producer(req, res);
+        return await securityHandler?.producer(req, res);
         break;
       case "/batch/accounts":
-          return await securityHandler?.producer(req, res);
+        return await securityHandler?.producer(req, res);
         break;
       // Roles Security END
       default: