Merge pull request 'FIX: BugFix relacionado ao status de requisições' (#4) from dev into master

Reviewed-on: #4

.gitea/workflows/deploy-production.yml

@@ -54,56 +54,72 @@ jobs:
           mkdir -p "$DEPLOY_PATH"
           printf '%s\n' "$ENV_PRODUCTION" > "$DEPLOY_PATH/.env.production"
           chmod 600 "$DEPLOY_PATH/.env.production"
+          chown desenvolvimento:desenvolvimento "$DEPLOY_PATH/.env.production"
 
       - name: Deploy files and restart PM2
         shell: bash
         run: |
           set -euo pipefail
 
+          # O runner roda como root (compartilhado com outras automacoes deste servidor), mas o
+          # app deve rodar como 'desenvolvimento' - e o dono do PM2/logs/.env.production hoje.
+          # Em vez de mudar o usuario do runner (afetaria outras automacoes), so essa etapa troca
+          # de usuario via sudo -u, escrevendo o script num arquivo temporario pra evitar qualquer
+          # ambiguidade de escaping de aspas dentro do 'run:' do workflow.
           DEPLOY_PATH="${PROD_DEPLOY_PATH:-/opt/sn-glpi-sync-new}"
           LOCK_FILE="/tmp/sn-glpi-sync-new.deploy.lock"
+          DEPLOY_SCRIPT="/tmp/sn-glpi-sync-new.deploy.sh"
 
           mkdir -p "$DEPLOY_PATH"
 
+          cat > "$DEPLOY_SCRIPT" <<'DEPLOY_EOF'
+          #!/usr/bin/env bash
+          set -euo pipefail
+
+          if [ -d "$DEPLOY_PATH" ] && [ -n "$(ls -A "$DEPLOY_PATH" 2>/dev/null)" ]; then
+            rsync -a --delete "$DEPLOY_PATH"/ "$DEPLOY_PATH.previous"/
+          fi
+
+          rsync -az --delete \
+            --exclude='.git/' \
+            --exclude='.gitea/' \
+            --exclude='.env*' \
+            --exclude='node_modules/' \
+            --exclude='logs/' \
+            "$SRC_DIR"/ "$DEPLOY_PATH"/
+
+          cd "$DEPLOY_PATH"
+
+          test -f .env.production || {
+            echo "Missing $DEPLOY_PATH/.env.production"
+            exit 1
+          }
+
+          grep -q '^SERVICENOW_CAOA_TI_GROUP_ID=.\+' .env.production || {
+            echo "SERVICENOW_CAOA_TI_GROUP_ID must be set in $DEPLOY_PATH/.env.production"
+            exit 1
+          }
+
+          npm ci --omit=dev
+
+          find . -path ./node_modules -prune -o -name '*.js' -print | xargs -n1 node --check
+
+          pm2 startOrRestart ecosystem.config.js --env production
+          pm2 save
+          pm2 list
+          DEPLOY_EOF
+          chmod +x "$DEPLOY_SCRIPT"
+
           (
             flock -n 9 || {
               echo "Another deploy is already running."
               exit 1
             }
-
+            sudo -u desenvolvimento -H DEPLOY_PATH="$DEPLOY_PATH" SRC_DIR="$PWD" bash "$DEPLOY_SCRIPT"
-            if [ -d "$DEPLOY_PATH" ] && [ -n "$(ls -A "$DEPLOY_PATH" 2>/dev/null)" ]; then
-              rsync -a --delete "$DEPLOY_PATH"/ "$DEPLOY_PATH.previous"/
-            fi
-
-            rsync -az --delete \
-              --exclude='.git/' \
-              --exclude='.gitea/' \
-              --exclude='.env*' \
-              --exclude='node_modules/' \
-              --exclude='logs/' \
-              ./ "$DEPLOY_PATH"/
-
-            cd "$DEPLOY_PATH"
-
-            test -f .env.production || {
-              echo "Missing $DEPLOY_PATH/.env.production"
-              exit 1
-            }
-
-            grep -q '^SERVICENOW_CAOA_TI_GROUP_ID=.\+' .env.production || {
-              echo "SERVICENOW_CAOA_TI_GROUP_ID must be set in $DEPLOY_PATH/.env.production"
-              exit 1
-            }
-
-            npm ci --omit=dev
-
-            find . -path ./node_modules -prune -o -name '*.js' -print | xargs -n1 node --check
-
-            pm2 startOrRestart ecosystem.config.js --env production
-            pm2 save
-            pm2 list
           ) 9>"$LOCK_FILE"
 
+          rm -f "$DEPLOY_SCRIPT"
+
       - name: Show sync logs
         if: always()
         shell: bash

src/services/servicenowService.js

@@ -437,17 +437,6 @@ const fetchAndProcessClosedTicketsFromSN = async () => {
 };
 
 const updateStatusInServiceNow = async (snTicketId, state) => {
-    // Mapeamento de status de string para codigo numerico do ServiceNow
-    const stateMap = {
-        'Aguardando Atendimento': '1', // New
-        'Em Atendimento': '2',      // In Progress
-        'Em Espera': '3',           // On Hold (confirmado via teste real - '-5' era invalido e o SN ignorava silenciosamente)
-        'Encerrado': '6',           // Resolved
-        'Encerrado - Omitido': '7'  // Closed
-    };
-
-    const stateCode = stateMap[state] || state; // Usa o codigo mapeado ou o valor original se nao encontrar
-
     let sysId = null; // Declarar sysId fora do try para estar disponivel no catch
     try {
         const ticketSn = await TicketSnModel.findById(snTicketId);
@@ -459,6 +448,19 @@ const updateStatusInServiceNow = async (snTicketId, state) => {
         sysId = ticketSn.sys_id; // Atribuir valor
         const ticketType = ticketSn.tipo;
 
+        // Mapeamento de status de string para codigo numerico do ServiceNow. incident e sc_req_item
+        // (requisicao) tem modelos de status diferentes - confirmado via teste real em dev que
+        // 'Em Espera' e '-5' para requisicao, e '3' para incidente (o mesmo codigo que e invalido/
+        // bloqueado por ACL do outro lado - ja aconteceu erro 403 em producao usando '3' numa requisicao).
+        const stateMap = {
+            'Aguardando Atendimento': '1', // New / Open
+            'Em Atendimento': '2',         // In Progress / Work in Progress
+            'Em Espera': ticketType === 'requisicao' ? '-5' : '3', // On Hold
+            'Encerrado': '6',              // Resolved
+            'Encerrado - Omitido': '7'     // Closed
+        };
+        const stateCode = stateMap[state] || state; // Usa o codigo mapeado ou o valor original se nao encontrar
+
         let url = '';
         let payload = {};