python / python/cpython

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

Offen
#95,891 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

stdlib type-bug
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
36k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne im TCP-Sendezweig von SysLogHandler um die Zeilen 990-991 und reproduziere den Fehler mit Python 3.10 unter Windows 10 mit FastAPI und Uvicorn. Überprüfe, dass das Senden mehrerer Nachrichten nach dem Schließen des TCP-Sockets nicht fehlschlägt, und ergänze oder aktualisiere die Testabdeckung für das Wiederverbindungsverhalten, falls das Repository einen geeigneten Testbereich enthält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
networking, observability-sre
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

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