StackGuardian/tirith

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

Aberta

#89 aberto em 28 de nov. de 2022

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)Python (41 forks)auto 404
buggood first issuehacktoberfest

Métricas do repositório

Stars
 (168 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Localize a lógica do código de saída em cli.py. O programa atualmente sempre retorna 0. Modifique para retornar 1 quando a avaliação final falhar. Verifique a função main e a função evaluate.
Pilha de tecnologia
python
Domain
backend
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80