CSSUTILS logger overrides existing config on import
- Vorherrschende Sprache
- Python
- Sterne
- 92
- Forks
- 30
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hi!
On first import of `cssutils`, the `'CSSUTILS'` logger is set up and configured with some default values, as can be seen here:
https://github.com/jaraco/cssutils/blob/main/cssutils/errorhandler.py#L51-L56
This can lead to issues if your import of `cssutils` happens later than your main logging config setup, since it will override whatever is already configured. To avoid it, you must pre-emptively import `cssutils` before your own logging config.
It would be easier if `cssutils` detected if the logger has already been set up, and didn't add it's own default config in that case. At the very least, if it preserved the loglevel the user has selected that would help.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Rechercherichtung
Lies cssutils/errorhandler.py in den Zeilen 51–56 und verfolge, wie der CSSUTILS-Logger beim ersten Import konfiguriert wird. Definiere „done“ als das Beibehalten einer vorhandenen Logger-Konfiguration, einschließlich des Log-Levels des Benutzers, anstatt Defaults anzuwenden; das Issue nennt keine Testdatei, also ermittle vor den Änderungen die relevante vorhandene Testabdeckung.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- developer-experience
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100