jaraco / jaraco/cssutils

CSSUTILS logger overrides existing config on import

Offen
#13 1 Kommentar 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement help wanted
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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.