python / python/cpython

asyncio: ` IocpProactor.recvfrom_into() ` misnames its receive-size parameter as ` flags`

Ouverte
#157,418 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

OS-windows stdlib topic-asyncio type-bug
Langage dominant
Python
Étoiles
77.2k
Forks
35.9k
Métriques de merge des PR
Métriques de PR en attente

Description

Bug report

Bug description:

On Windows, IocpProactor.recvfrom_into() defines its third parameter as flags:

def recvfrom_into(self, conn, buf, flags=0):
    ...
    ov.WSARecvFromInto(conn.fileno(), buf, flags)

However, _overlapped.Overlapped.WSARecvFromInto() expects:

(handle, buf, size, flags=0)

so the value named flags is actually passed as size and used as the receive length (nbytes). The parameter is therefore misnamed and inconsistent with the surrounding APIs.

CPython versions tested on:

CPython main branch

Operating systems tested on:

No response

Linked PRs
  • gh-157422

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 par localiser IocpProactor.recvfrom_into() et comparer son troisième paramètre avec _overlapped.Overlapped.WSARecvFromInto(). Examinez ensuite la PR liée gh-157422, puis vérifiez que le nommage des paramètres et l’utilisation de la taille de réception sont cohérents sans modifier l’argument flags.

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

Évaluation

Stack technique
python
Domaine
networking
Type d'issue
Bug
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
30/100

Recevez les nouvelles issues par e-mail

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