microsoftgraph / microsoftgraph/msgraph-sdk-python-core

Logging to root logger

Ouverte Adaptée aux débutants
#1,078 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

status:waiting-for-triage type:bug
Langage dominant
Python
Étoiles
288
Forks
52
Merge moyen
8 h 10 min
PR mergées (30 j)
1

Description

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.

https://github.com/microsoftgraph/msgraph-sdk-python-core/blob/7c2fe2754ab0b84f390e7f7aaca050c560ff286a/src/msgraph_core/requests/batch_request_builder.py#L70

https://github.com/microsoftgraph/msgraph-sdk-python-core/blob/7c2fe2754ab0b84f390e7f7aaca050c560ff286a/src/msgraph_core/tasks/large_file_upload.py#L133

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par src/msgraph_core/requests/batch_request_builder.py vers la ligne 70 et src/msgraph_core/tasks/large_file_upload.py vers la ligne 133, puis examinez comment leurs loggers sont créés. Reproduisez la requête batch incorrecte avec logging.basicConfig(). Le travail est terminé lorsque les messages de log des deux modules sont envoyés à des loggers spécifiques au module ou au package plutôt qu’au logger racine.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
api
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
Calme
Clarté
Clairement spécifiée
Accessibilité débutants
75/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.