MagicStack / MagicStack/uvloop

uvloop is broken for abstract sockets - create_unix_connection does not work for abstract sockets as libuv does not support them

Offen
#528 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Cython
Sterne
11.9k
Forks
616
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

* **uvloop version**: master
* **Python version**: 3.10+
* **Platform**: unix
* **Can you reproduce the bug with `PYTHONASYNCIODEBUG` in env?**: yes
* **Does uvloop behave differently from vanilla asyncio? How?**: yes

uvloop:
tr = UnixTransport.new(self, protocol, None, waiter, context)
tr.connect(path) <--- uses uv_pipe_connect which uses uv__strscpy which assumes zero terminated string

asyncio:
sock_connect -> getsockaddrarg (in socketmodule.c) which uses memcpy


I have raised this with libuv as issue: https://github.com/libuv/libuv/discussions/3911

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

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 bei create_unix_connection und dem gezeigten Pfad UnixTransport.new/tr.connect(path), und vergleiche ihn anschließend mit der socket_connect-Behandlung von vanilla asyncio. Lies die verlinkte libuv-Diskussion über abstrakte Sockets, bevor du entscheidest, ob die Behebung in uvloop oder libuv gehört. Als erledigt gilt die Aufgabe, wenn Verbindungen zu abstrakten Unix-Sockets wie erwartet funktionieren und das Verhalten anhand von asyncio verifiziert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
networking
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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