Conversation
| extra = { | ||
| "tags": _logger_tags.get(), | ||
| "contexts": _logger_contexts.get(), | ||
| "trace_id": ensure_current_trace_id(), |
There was a problem hiding this comment.
Acredito que seja interessante o trace_id estar dentro de contexts, para que todos os handlers possam enviar contendo essa informação.
Não somente nos logs.
There was a problem hiding this comment.
Além disso, estar no core do Tracker, mas fazer de uma forma modularizada. Quem está implementando na biblioteca escolhe o que quer implementar.
| @@ -0,0 +1,35 @@ | |||
| import types | |||
There was a problem hiding this comment.
Vale mover os testes para onde o Tracker tem os testes.
Uma outra coisa, nesse projeto tem testes de mutação e uma cobertura 100% nos testes unitários
| from __future__ import annotations | ||
| from typing import Any, Iterable, Optional | ||
|
|
||
| def attach_framework(app: Any, **kwargs): |
There was a problem hiding this comment.
Eu penso que vale o desenvolvedor da aplicação definir do que tentarmos adivinhar.
There was a problem hiding this comment.
Mas é ele que define. Da uma olhada no Readme de implementação para cada framework
| out[kl] = "***REDACTED***" if kl in self.redact_headers else str(v) | ||
| return out | ||
|
|
||
| self.logger.info("http_request", |
There was a problem hiding this comment.
Acredito que essa parte não devesse estar aqui
2 participants
Este pr tem como objetivo inserir id em cada requisição visando rastreamento e criar logs de request e response de forma modular