byt3bl33d3r/CrackMapExec

Logging discrepancies ?

Open

#534 aperta il 6 feb 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (1650 fork)batch import
help wanted

Metriche repository

Star
 (7931 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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?

Guida contributor

Direzione di ricerca
Esamina l'uso di self.logger.info rispetto a logging.debug nel file ldap.py alle righe 509 e 533. Rivedi altri file di protocollo in cme/protocols per vedere quale stile di logging è più comune. Se incoerente, proponi un approccio coerente (ad esempio, usa self.logger per tutto il debug logging) e aggiorna il file di conseguenza. La modifica è localizzata e non dovrebbe influenzare la funzionalità.
Tech stack
python
Dominio
developer experience
Tipo issue
Ricerca
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90