microsoftgraph / microsoftgraph/msgraph-sdk-python-core
Logging to root logger
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 288
- Forks
- 52
- Ø Merge
- 8 Std. 10 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit src/msgraph_core/requests/batch_request_builder.py ungefähr in Zeile 70 und src/msgraph_core/tasks/large_file_upload.py ungefähr in Zeile 133 und prüfe anschließend, wie deren Logger erstellt werden. Reproduziere die fehlerhafte Batch-Anfrage mit logging.basicConfig(). Als erledigt gilt die Aufgabe, wenn die Log-Meldungen aus beiden Modulen an modul- oder paketspezifische Logger statt an den Root-Logger gehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 75/100