feat: separação de monitoramento online com Datadog e local com Moesif

PauloGoncalvesBH · PauloGoncalvesBH · commit de6ff16f5efd · 2023-09-08T18:16:13.000+01:00
diff --git a/.husky/pre-push b/.husky/pre-push
@@ -1,4 +1,4 @@
 #!/bin/sh
 . "$(dirname "$0")/_/husky.sh"
 
-make test
+make test-pre-push
diff --git a/Makefile b/Makefile
@@ -25,7 +25,9 @@ run-dev:
 test-contract:
 	@docker-compose up --abort-on-container-exit --build test-contract
 
-test: test-unit test-integration
+test: test-unit test-integration test-e2e-localhost
+
+test-pre-push: test-unit test-integration
 
 test-unit:
 	@docker compose up --abort-on-container-exit --build test-unit
diff --git a/README.md b/README.md
@@ -29,7 +29,6 @@ _ServeRest_ permite o estudo de:
 - Autenticação no header
 - Query string
 - Teste de schema json
-- Teste de carga
 
 <b><h2 align="center">Ambientes disponíveis</h2></b>
 
@@ -129,10 +128,9 @@ Você pode encontrar as versões disponíveis na [lista de tags no Docker Hub](h
 
 ### IMPORTANTE
 
-1. É obrigatório enviar o header `monitor: false` em todas as requisições do seu teste de carga.
-2. O teste de carga deve ser executado apenas em ambiente local (disponibilizado via [NPM](#localmente-com-npm) ou [Docker](#localmente-com-docker) e acessível via <http://localhost:3000>).
+**O teste de carga deve ser executado apenas em ambiente local (disponibilizado via [NPM](#localmente-com-npm) ou [Docker](#localmente-com-docker) e acessível via <http://localhost:3000>).**
 
-> O não seguimento dos 2 tópicos acimas vai acarretar em prejuízo para o projeto open source e gratuito e irá impactar o estudo de outras pessoas.
+> O não seguimento vai acarretar em prejuízo para o projeto open source e gratuito e irá impactar o estudo de outras pessoas.
 
 ### Acesso ao status
 
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -64,7 +64,6 @@
     "morgan": "^1.10.0",
     "nedb-promises": "^5.0.0",
     "open": "^8.0.6",
-    "rate-limiter-flexible": "^2.3.1",
     "swagger-ui-express": "^4.1.4",
     "yargs": "^16.2.0"
   },
diff --git a/src/app.js b/src/app.js
@@ -22,7 +22,6 @@ const errorHandler = require('./middlewares/error-handler')
 const logger = require('./utils/logger')
 const { version } = require('../package.json')
 const swaggerDocument = require('../docs/swagger.json')
-const rateLimiter = require('./middlewares/rate-limiter')
 const packageJson = require('../package.json')
 
 const app = express()
@@ -40,11 +39,6 @@ app.use(queryParser())
 app.use(timeout())
 app.use(cors())
 
-/* istanbul ignore next */
-if (!ehAmbienteDeTestes) {
-  app.use(rateLimiter)
-}
-
 app.disable('etag')
 
 /* istanbul ignore else */
diff --git a/src/middlewares/rate-limiter.js b/src/middlewares/rate-limiter.js
diff --git a/src/utils/logger.js b/src/utils/logger.js
@@ -11,13 +11,14 @@ const moesif = require('moesif-nodejs')
 
 const { version } = require('../../package.json')
 const {
+  aplicacaoExecutandoLocalmente,
   formaDeExecucao,
   ehAmbienteDeDesenvolvimento,
   ehAmbienteDeTestes
 } = require('./ambiente')
 
 module.exports = async app => {
-  if (ehAmbienteDeDesenvolvimento || ehAmbienteDeTestes) {
+  if (ehAmbienteDeDesenvolvimento || ehAmbienteDeTestes || !aplicacaoExecutandoLocalmente()) {
     return
   }
   const { porta, timeout, nodoc, nobearer, nosec } = require('../server').argv
@@ -29,16 +30,13 @@ module.exports = async app => {
       if (req.originalUrl === '/__messages__' ||
           req.originalUrl === '/favicon.ico' ||
           req.originalUrl === '/socket.io' ||
-          req.originalUrl === '/version' ||
           req.originalUrl === '/swagger-ui.css' ||
           req.originalUrl === '/swagger-ui.css.map' ||
           req.originalUrl === '/swagger-ui-init.js' ||
           req.originalUrl === '/swagger-ui-standalone-preset.js' ||
           req.originalUrl === '/swagger-ui-standalone-preset.js.map' ||
           req.originalUrl === '/swagger-ui-bundle.js' ||
-          req.originalUrl === '/swagger-ui-bundle.js.map' ||
-          req.rawHeaders.some(header => header.toLowerCase() === 'monitor'.toLowerCase()) ||
-          (formaDeExecucao() === 'serverest.dev' && req.originalUrl === '/')) {
+          req.originalUrl === '/swagger-ui-bundle.js.map') {
         return true
       }
     },
