byt3bl33d3r/CrackMapExec

Logging discrepancies ?

Open

#534 aberto em 6 de fev. de 2022

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (1.650 forks)batch import
help wanted

Métricas do repositório

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

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?

Guia do colaborador

Direção de pesquisa
Examine o uso de self.logger.info versus logging.debug no arquivo ldap.py nas linhas 509 e 533. Revise outros arquivos de protocolo em cme/protocols para ver qual estilo de registro é mais comum. Se inconsistente, proponha uma abordagem consistente (por exemplo, use self.logger para todos os registros de depuração) e atualize o arquivo de acordo. A alteração é localizada e não deve afetar a funcionalidade.
Pilha de tecnologia
python
Domain
developer experience
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
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
Python basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90