nccgroup/ScoutSuite

Improve error handling across all resources

Open

#873 ouverte le 23 sept. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)Python (978 forks)batch import
component-provider-alibabacomponent-provider-awscomponent-provider-azurecomponent-provider-gcpcomponent-provider-ocienhancementgood first issue

Métriques du dépôt

Stars
 (5 904 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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))

Guide contributeur

Direction de recherche
Inspectez toutes les méthodes d'analyse des ressources dans le répertoire providers, identifiez les endroits où try except manque et enveloppez les appels d'analyse avec des blocs try except pour gérer les exceptions de manière appropriée. Assurez vous que fetch all continue en cas d'échec. Suivez le modèle montré dans l'exemple des rôles AWS IAM.
Stack technique
python
Domaine
backendsecuritycloud
Type d'issue
Refactorisation
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.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80