byt3bl33d3r/CrackMapExec

Logging discrepancies ?

Open

#534 ouverte le 6 févr. 2022

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Python (1 650 forks)batch import
help wanted

Métriques du dépôt

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

Description

Hey there, I was working on the source code of the ldap protocol (cme/protocols/ldap.py) and was wondering why some debug logging is made with self.logger.info and why other is made with logging.debug.

https://github.com/byt3bl33d3r/CrackMapExec/blob/6e57e4c3616b361fa44311389af2bbd7bb12839c/cme/protocols/ldap.py#L509

https://github.com/byt3bl33d3r/CrackMapExec/blob/6e57e4c3616b361fa44311389af2bbd7bb12839c/cme/protocols/ldap.py#L533

Is there a best practice to follow, which one should we use?

Guide contributeur

Direction de recherche
Examinez l'utilisation de self.logger.info par rapport à logging.debug dans le fichier ldap.py aux lignes 509 et 533. Passez en revue d'autres fichiers de protocole dans cme/protocols pour voir quel style de journalisation est le plus courant. Si incohérent, proposez une approche cohérente (par exemple, utilisez self.logger pour toute la journalisation de débogage) et mettez à jour le fichier en conséquence. Le changement est localisé et ne doit pas affecter la fonctionnalité.
Stack technique
python
Domaine
developer experience
Type d'issue
Recherche
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.
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
Python basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90