getsentry / getsentry/sentry-javascript

[Nestjs] Instrument Controllers

Abierto
#18,485 2 comentarios 0 reacciones 0 asignados Ver en GitHub
Improvement Nest.js
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
1.8k
Merge medio
1 d 17 h
PR fusionados (30 d)
515

Descripción

I noticed that we do not have any tracing for nest [controllers](https://docs.nestjs.com/controllers) atm. I'd assume that many people use nest as a REST service, in which case controllers are the entry points for any incoming requests.

Solution Brainstorm:

* All controllers have a `@Controller` decorator. Individual endpoints also have annotations for the request type (e.g. `@Get`).
* We can hook into one of these annotations as we for instance already do for the `@Injectable` decorator.
* We need to check if all annotations are actually required or if users can omit them (for instance if we omit the request type annotation it might default to a GET endpoint).

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.