microsoftgraph / microsoftgraph/msgraph-sdk-python-core
Logging to root logger
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- Python
- Estrelas
- 288
- Forks
- 52
- Merge médio
- 8h 10min
- PRs com merge (30d)
- 1
Descrição
Describe the bug
The library logs to the root logger. As that's a widely-accepted bad practice, I guess it qualifies as a bug.
Expected behavior
All log messages should go to module- or package-specific loggers.
How to reproduce
import logging
logging.basicConfig()
# Send a bad batch request
SDK Version
1.3.8
Latest version known to work for scenario above?
No response
Known Workarounds
No response
Debug output
Configuration
No response
Other information
No response
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece por src/msgraph_core/requests/batch_request_builder.py por volta da linha 70 e src/msgraph_core/tasks/large_file_upload.py por volta da linha 133; em seguida, inspecione como os loggers deles são criados. Reproduza a solicitação de batch incorreta com logging.basicConfig(). O trabalho estará concluído quando as mensagens de log de ambos os módulos forem para loggers específicos do módulo ou do pacote, em vez do logger raiz.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- api
- Tipo de issue
- Bug
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Status de atividade
- Pouca atividade
- Clareza
- Claramente especificada
- Facilidade para iniciantes
- 75/100