protocolbuffers/protobuf

protoc cli returns success with --fatal_warnings switch when warnings are detected

Open

#10.486 aperta il 1 set 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C++ (16.128 fork)batch import
help wantedprotoc

Metriche repository

Star
 (71.223 star)
Metriche merge PR
 (Merge medio 2g 11h) (185 PR mergiate in 30 g)

Descrizione

Version: libprotoc 3.19.4 Language: NA OS: OSX, installed via brew

Steps to reproduce the behavior:

  1. open a terminal (I use zsh)
  2. decode a protobuf file for which a warning will be emitted using the --fatal_warnings switch
  3. check the command return value
$ cat temp.in.pbuf | protoc  --decode job_definition.job_t --fatal_warnings --proto_path ~/workspace/engine/proto/ job.proto > /dev/null
warning:  Input message is missing required fields:  country[0].id, country[1].id
$ echo $?
0

notice the contract include

message country_t {
	required int32 id = 1;
	required string country = 2;
}

and the encode file does not indeed have an id field, i.e. the warning is correct.

What did you expect to see protoc shall return a non-zero error

What did you see instead? protoc returns 0

Guida contributor

Direzione di ricerca
Esamina il percorso del codice in protoc che gestisce fatal warnings, in particolare come vengono accumulati gli avvisi e come viene impostato il codice di uscita. Traccia la logica da main() al gestore degli avvisi per trovare dove il codice di uscita non viene impostato su un valore diverso da zero.
Tech stack
cpp
Dominio
cliapi
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
C++protoc
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65