facebookarchive/prepack

Bad command-line crashes Prepack

Open

#2.509 aberto em 30 de ago. de 2018

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (14.268 stars) (520 forks)batch import
bughelp wantedlevel 1 (super easy)priority: low

Description

This triggers an invariant violation.

node lib/prepack-cli.js --debugDiagnosticSeverity foo

It shouldn't. The command-line argument processing should rather print a user-friendly error message in such cases and exit, like it already happens for most or all other bad arguments.

Guia do colaborador

Pilha de tecnologia
javascriptnodejs
Domain
clideveloper experience
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
JavaScriptNode.jsbasic CLI knowledge
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65
Direção de pesquisa
Identify where command line arguments are parsed in lib/prepack cli.js or similar. Determine how other invalid arguments produce user friendly errors. Find the specific invariant violation triggered by bad debugDiagnosticSeverity and replace it with a proper error message and exit.