python / python/cpython

SysLogHandler closes tcp session after first message (also noted in #66006)

Ouverte
#95,891 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

stdlib type-bug
Langage dominant
Python
Étoiles
77.2k
Forks
36k
Métriques de merge des PR
Métriques de PR en attente

Description

Bug report

Python 3.10.5
While running FastAPI (Uvicorn) and the SysLogHandler. The Socket if using a TCP stream is being closed after the first message, causing all additional messages to fail stating OSError10083. In looking through the SysLogHandler code I was able to add a try/except that solved the issue by opening a new socket if the current is found to be closed.

Lines 990/991:

else:
    self.socket.sendall(msg)

I changed to:

else:
    try:
        self.socket.sendall(msg)
    except OSError:
        with socket.create_connection(self.address) as s:
            s.sendall(msg)

Your environment

Python 3.10

  • CPython versions tested on: 3.10
  • Operating system and architecture: Windows 10

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 dans la branche d’envoi TCP de SysLogHandler autour des lignes 990-991 et reproduisez l’échec avec Python 3.10 sur Windows 10 en utilisant FastAPI et Uvicorn. Vérifiez que l’envoi de plusieurs messages n’échoue pas après la fermeture du socket TCP, et ajoutez ou mettez à jour la couverture du comportement de reconnexion si le dépôt contient un emplacement de test pertinent.

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

Évaluation

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

Recevez les nouvelles issues par e-mail

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