dotnet/tye

Exit code not properly logged in error message

Open

#1 138 ouverte le 4 août 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C# (5 310 stars) (532 forks)batch import
buggood first issue

Description

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.

Guide contributeur

Stack technique
csharp
Domaine
clideveloper experience
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic C# knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
95
Direction de recherche
The issue is clear and directly points to the bug location at src/Microsoft.Tye.Hosting/Watch/DotNetWatcher.cs line 76. The fix is to change the logging statement to output only the ExitCode property of ProcessResult instead of the entire object. No further investigation is needed.