dotnet/tye

Exit code not properly logged in error message

Open

#1138 aperta il 4 ago 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (532 fork)batch import
buggood first issue

Metriche repository

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

Descrizione

Describe the bug

If a watched replica exits due to a non-zero exit code, Tye logs an error message but does not properly include the exit code.

To Reproduce

  1. Start a Tye application in "watch" mode.
  2. Kill one of the replica processes.
  3. Note watch: Exited with error code Microsoft.Tye.ProcessResult message.

Further technical details

The bug is here; only the ExitCode property should be logged, not the whole ProcessResult.

Guida contributor

Direzione di ricerca
Il problema è chiaro e indica direttamente la posizione del bug alla riga 76 di src/Microsoft.Tye.Hosting/Watch/DotNetWatcher.cs. La correzione consiste nel modificare l'istruzione di log in modo che stampi solo la proprietà ExitCode di ProcessResult invece dell'intero oggetto. Non è necessaria alcuna ulteriore indagine.
Tech stack
csharp
Dominio
clideveloper experience
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic C# knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
95