pinojs/pino

bindings() and setBindings() are not present in the browser version of pino

Aperta

#1177 aperta il 20 ott 2021

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)JavaScript (839 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (13.157 stelle)
Metriche merge PR
 (Merge medio 1h 28m) (1 PR mergiata in 30 g)

Descrizione

Hello!

I was attempting to use pino in the browser today and discovered that .bindings() is undefined. Additionally, .setBindings() is also undefined, but it's also undocumented in general (is it officially part of the API?).

I looked into adding this to browser.js but I'd definitely need some guidance before proceeding. For example, I'm not sure if bindings changes cascade through child loggers or if they are fixed at the time of child creation, for example.

Guida contributor

Direzione di ricerca
Inizia esaminando il file browser.js nel repository per capire come è implementato il logger del browser. Controlla la documentazione per bindings() e setBindings() nella versione Node.js. Osserva come vengono creati i logger figli e se i binding sono mutabili. L'autore del problema ha richiesto una guida, quindi sarebbe utile rivedere eventuali commenti per ulteriore contesto.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70