openzipkin/zipkin-js

Adds support for http.route based name

Open

#183 aperta il 30 gen 2018

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (167 fork)auto 404
help wanted

Metriche repository

Star
 (569 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Currently http middlewares use the http method for naming spans. This solution is a not so bad low cardinality option, however from user perspective this could be improved by using the path instead of the verb. For that we can override the span name in the onPreResponse (see this in hapi for example) event so instead of having get as span name we get get /user/{user_id}.

Ping @fbaiodias @adriancole

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.