nccgroup/ScoutSuite

Improve error handling across all resources

Open

#873 aberto em 23 de set. de 2020

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)Python (978 forks)batch import
component-provider-alibabacomponent-provider-awscomponent-provider-azurecomponent-provider-gcpcomponent-provider-ocienhancementgood first issue

Métricas do repositório

Stars
 (5.904 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Currently, facades implement good exception handling, but resource parsing does not. That means that for a given resource type, if parsing fails for any given resource, the fetch_all method fails and stops, hence not parsing any additional resources.

All resources should be reviewed and updated, to ensure they handle parsing errors.

e.g., for AWS IAM roles (https://github.com/nccgroup/ScoutSuite/blob/master/ScoutSuite/providers/aws/resources/iam/roles.py#L8) the method should be updated as so:

for raw_role in raw_roles:
    try:
        name, resource = self._parse_role(raw_role)
        self[name] = resource
    except Exception as e:
        print_exception('Failed to parse {} resource: {}'.format(self.__class__.__name__, e))

Guia do colaborador

Direção de pesquisa
Inspecione todos os métodos de análise de recursos no diretório providers, identifique onde falta try except e envolva as chamadas de análise com blocos try except para lidar com exceções de forma adequada. Certifique se de que fetch all continue em caso de falha. Siga o padrão mostrado no exemplo de funções do AWS IAM.
Pilha de tecnologia
python
Domain
backendsecuritycloud
Tipo Issue
Refatoração
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.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Python
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80