openzipkin/zipkin-js

Status code 1xx should not be marked as error

Open

#343 aperta il 11 mar 2019

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (167 fork)auto 404
good first issue

Metriche repository

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

Descrizione

Currently, responses with status code <200 are being marked as errors.

https://github.com/openzipkin/zipkin-js/blob/master/packages/zipkin/src/instrumentation/httpServer.js#L91 https://github.com/openzipkin/zipkin-js/blob/master/packages/zipkin/src/instrumentation/httpClient.js#L43

Comes from https://github.com/openzipkin/brave/issues/874

Ping @adriancole @ghermeto

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.