pinojs/pino-http

Bug: `customErrorMessage` actually got passed 4 arguments, but it's ts declaration only has 3 arguments

Chiusa

#359 aperta il 25 nov 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (128 fork)github user discovery
buggood first issue

Metriche repository

Star
 (694 stelle)
Metriche merge PR
 (Merge medio 1m) (2 PR mergiate in 30 g)

Descrizione

customErrorMessage actually got passed 4 arguments: https://github.com/pinojs/pino-http/blob/8baaeec6fa2b53a7fe1c6bb948b3f46b7dbb6bdf/logger.js#L123 , but its ts declaration only has 3 arguments: https://github.com/pinojs/pino-http/blob/8baaeec6fa2b53a7fe1c6bb948b3f46b7dbb6bdf/index.d.ts#L34

I'd like to make use of the 4th argument "responseTime" in my logging message.

Guida contributor

Direzione di ricerca
Aggiorna il file di dichiarazione TypeScript per includere un quarto parametro opzionale per responseTime nel callback customErrorMessage.
Tech stack
javascripttypescriptnodejs
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70