Skip to content

adicionado morgan como logger#12

Open
danielmadu wants to merge 2 commits into
sapucaiatech:masterfrom
danielmadu:master
Open

adicionado morgan como logger#12
danielmadu wants to merge 2 commits into
sapucaiatech:masterfrom
danielmadu:master

Conversation

@danielmadu
Copy link
Copy Markdown
Contributor

O pacote morgan possui já uma gama de combinações que podem ser utilizadas para log de acesso além também de ser possível salvar esse log em arquivos.

@filhocodes
Copy link
Copy Markdown

@danielmadu Não utilize process.env.DEV para identificar modo de desenvolvimento por dois motivos:

@filhocodes
Copy link
Copy Markdown

@danielmadu não utilize o modo de desenvolvimento para identificar se devemos registrar ou não. Logs devem sempre existir em uma aplicação (e isso me faz sentir muita falta da utilização do pacote debug neste projeto).

Minha sugestão: process.env.APP_LOG deve indicar se devemos usar ou não logs, inclusive o formato que queremos (process.env.APP_LOG="dev").

Conhecendo o pacote Morgan, também poderíamos configurar em que nível que queremos logs HTTP:

  • Completo - todas as requisições
  • Sensíveis - apenas requisições que modificam recursos (POST, PUT, DELETE, PATCH)
  • Erros - apenas requisições cujas respostas sejam da família 4** ou 5**
  • Sensíveis e erros - you got the point

@filhocodes
Copy link
Copy Markdown

/cc @gabsprates nos dois comentários acima

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants