pugjs/pug

Generated `debug_sources` are node `Buffer`s and fail `pug_rethrow`

Open

#3290 aperta il 18 ago 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (1955 fork)batch import
Good First Issuebug

Metriche repository

Star
 (21.509 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Pug Version: your version number here 3.0.0

Node Version: your version number here 14.7.0

When an included file is read, it's Buffer being stored in debug_sources:

https://github.com/pugjs/pug/blob/06baa525a23049756de9587461d389a12bc12537/packages/pug/lib/index.js#L167

Later, this value is passed to the generator and being serialized as-it in the final render function.

https://github.com/pugjs/pug/blob/06baa525a23049756de9587461d389a12bc12537/packages/pug/lib/index.js#L204

Which fails pug_rethrow's expectation, and makes it throw:

https://github.com/pugjs/pug/blob/f97ebdb48c7c0fdd4ff4b7418dcf4e03b27a1405/packages/pug-runtime/index.js#L256

Guida contributor

Direzione di ricerca
Il bug si trova nel file index.js del pacchetto pug. Nello specifico, la variabile debug sources memorizza oggetti Buffer da readFile (riga 167), ma la funzione pug rethrow si aspetta stringhe (riga 256). La correzione dovrebbe convertire il Buffer in una stringa utilizzando .toString() prima della memorizzazione o prima di passarlo al generatore. Rivedere le righe di codice collegate per la posizione esatta e assicurarsi che la conversione avvenga in modo appropriato.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
Node.jsBuffers
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60