jaraco / jaraco/cssutils

CSSUTILS logger overrides existing config on import

Ouverte
#13 1 commentaire 2 réactions 0 personnes assignées Voir sur GitHub
enhancement help wanted
Langage dominant
Python
Étoiles
92
Forks
30
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Lisez les lignes 51-56 de cssutils/errorhandler.py et suivez la manière dont le logger CSSUTILS est configuré lors du premier import. Définissez done comme le fait de préserver une configuration existante du logger, y compris le niveau de journalisation de l’utilisateur, au lieu d’appliquer les valeurs par défaut ; l’issue ne nomme aucun fichier de test, identifiez donc la couverture existante pertinente avant d’effectuer des changements.

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

Évaluation

Stack technique
python
Domaine
developer-experience
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

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