@oondemand/oon-core-back 0.7.2 → 0.7.3

package/README.md

@@ -73,3 +73,21 @@ defineModel({
 ```
 
 Use `populateRefs: true` para todas as referências da model ou informe uma lista explícita. O CRUD devolve somente `_id` e campos usuais de identificação, como nome, razão social, descrição, código, e-mail e campos pesquisáveis da model referenciada. Exportações mantêm os identificadores originais.
+
+## Requisições com corpo sensível
+
+Rotas privadas declaradas com `defineRoutes` que recebem credenciais ou outros segredos devem marcar a operação mutante com `sensitiveBody: true`:
+
+```js
+defineRoutes("/integracao", (router) => {
+  router.private.put("/credencial", {
+    permission: "integracao.config.write",
+    sensitiveBody: true,
+  }, async (req, res) => {
+    // Validar e persistir com segurança; devolver somente metadados sanitizados.
+    res.json({ configured: true });
+  });
+});
+```
+
+O Core omite todo o corpo de entrada e resposta no `requestLog` para esse método e caminho, inclusive se a rota responder erro. Método, URL, status, usuário e correlationId continuam registrados. A opção não cifra o banco nem substitui autorização, validação e respostas sanitizadas. Nunca retorne o segredo pela rota, pelo erro ou pela auditoria da mutação. A opção não se aplica automaticamente a endpoints de CRUD gerados.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oondemand/oon-core-back",
-  "version": "0.7.2",
+  "version": "0.7.3",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/oondemand/oon-platform.git",

package/src/core/define.js

@@ -161,6 +161,9 @@ function defineRoutes(basePath, builder) {
     const required = opts.permission || opts.permissions;
     if (required) chain.push(requirePermission(required));
     else chain.push(rbac(opts.roles || []));
+    if (["post", "put", "patch", "delete"].includes(method) && opts.sensitiveBody === true) {
+      registry.registerSensitiveRequest(method, basePath, path);
+    }
     if (["post", "put", "patch", "delete"].includes(method) && opts.audit) {
       chain.push(mutationAudit(opts.audit));
     }

package/src/core/middlewares/requestLog.js

@@ -1,4 +1,5 @@
 const Log = require("../models/Log");
+const registry = require("../registry");
 
 /**
  * Registra requisições mutantes (não-GET) na model interna Log.
@@ -7,18 +8,19 @@ const Log = require("../models/Log");
 const requestLog = async (req, res, next) => {
   if (req.method === "GET") return next();
 
+  const sensitiveBody = await registry.isSensitiveRequest(req.method, req.url || req.originalUrl);
   const log = new Log({
     usuario: req.usuario || null,
     endpoint: req.originalUrl,
     metodo: req.method,
     ip: req.ip || req.headers["x-forwarded-for"] || req.connection?.remoteAddress,
     correlationId: req.correlationId,
-    dadosRequisicao: req.body,
+    dadosRequisicao: sensitiveBody ? { omitted: true } : req.body,
   });
 
   res.on("finish", () => {
     log.statusResposta = res.statusCode;
-    log.dadosResposta = res.locals.body || null;
+    log.dadosResposta = sensitiveBody ? null : (res.locals.body || null);
     log.save().catch(() => {});
   });
 

package/src/core/registry.js

@@ -1,3 +1,5 @@
+const express = require("express");
+
 /**
  * Model Registry — coração técnico do Core (Seção 17).
  *
@@ -17,6 +19,7 @@ const store = {
   pipelines: new Map(),
   menus: [],
   features: new Map(),
+  sensitiveRequests: [], // Express routers used only to classify sensitive paths
 };
 
 function registerModel(entry) {
@@ -68,6 +71,37 @@ const simpleRegister = (mapName) => (key, value) => {
 };
 const simpleList = (mapName) => () => [...store[mapName].values()];
 
+function registerSensitiveRequest(method, basePath, path) {
+  // Use the same mount and route semantics as defineRoutes/createApp.
+  const route = express.Router();
+  route[method.toLowerCase()](path, (req) => req.sensitiveMatch());
+  const mounted = express.Router();
+  mounted.use(basePath, route);
+  store.sensitiveRequests.push(mounted);
+}
+
+async function isSensitiveRequest(method, url) {
+  for (const router of store.sensitiveRequests) {
+    const matched = await new Promise((resolve) => {
+      // A separate request prevents classification from modifying the real request.
+      const request = {
+        method: String(method).toUpperCase(),
+        url: String(url || '/'),
+        headers: {},
+        sensitiveMatch: () => resolve(true),
+      };
+      // Routing errors fail closed: omit bodies instead of risking disclosure.
+      try {
+        router.handle(request, {}, (error) => resolve(Boolean(error)));
+      } catch {
+        resolve(true);
+      }
+    });
+    if (matched) return true;
+  }
+  return false;
+}
+
 function registerMenu(item) {
   store.menus.push(item);
 }
@@ -82,6 +116,7 @@ function reset() {
   store.pipelines.clear();
   store.menus.length = 0;
   store.features.clear();
+  store.sensitiveRequests.length = 0;
 }
 
 module.exports = {
@@ -104,6 +139,8 @@ module.exports = {
   registerFeature: simpleRegister("features"),
   listFeatures: simpleList("features"),
   registerMenu,
+  registerSensitiveRequest,
+  isSensitiveRequest,
   listMenus: () => store.menus,
   reset,
 };