facebookarchive/prepack

`__evaluatePureFunction` doesn't properly validate arguments

Open

#2385 aperta il 3 ago 2018

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
bughelp wantedlevel 1 (super easy)

Metriche repository

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

Descrizione

Prepacking the following crashes Prepack.

__evaluatePureFunction("boo");

In the implementation, it shouldn't check via an invariant that the argument is a proper value, but instead it should throw a user-level exception or emit a CompilerDiagnostics error.

Guida contributor

Direzione di ricerca
Cerca ` evaluatePureFunction` nella codebase per individuare il controllo di invariante. Sostituiscilo con un errore appropriato utilizzando CompilerDiagnostics o un'eccezione a livello utente, seguendo i modelli di gestione degli errori esistenti nel repository.
Tech stack
javascript
Dominio
tooling
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptPrepack internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65