StackGuardian/tirith

🐛 cli.py: return non zero code when final evaluation returns false

Ouverte

#89 ouverte le 28 nov. 2022

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Python (41 forks)auto 404
buggood first issuehacktoberfest

Métriques du dépÎt

Stars
 (168 étoiles)
MĂ©triques de merge PR
 (Métriques PR en attente)

Description

Describe the bug tirith always return zero code even when the final evaluator returns false (failed)

To Reproduce I was using these files: plan.json.txt apigateway_state_logging_enabled.tirith.json.txt

Steps to reproduce the behavior:

  1. Download and rename both of the files above, from .txt to .json
  2. Run tirith -policy-path apigateway_state_logging_enabled.tirith.json -input-path plan.json
  3. The final evaluation should be failed
  4. Check the return code by doing echo $?
  5. It will print 0 instead of 1

Expected behavior It should return non-zero code as it will make the caller of the program knows that tirith final evaluation has failed

Guide contributeur

Direction de recherche
Localisez la logique du code de sortie dans cli.py. Le programme renvoie actuellement toujours 0. Modifiez le pour renvoyer 1 lorsque l'Ă©valuation finale Ă©choue. VĂ©rifiez la fonction main et la fonction evaluate.
Stack technique
python
Domaine
backend
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraßche, active, ancienne, bloquée ou en attente d'un mainteneur.
RĂ©cente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critÚres d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80