YamuxStream is tagged to Unreadable too early, since there may be data that hasn't been processed
Personne n'a encore pris cette issue.
- Langage dominant
- C++
- Étoiles
- 492
- Forks
- 130
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
From testing of my program, after YamuxStream tagging stream being unreadable in onFINReceived at this line, there may still be remaining data to be processed in onLengthRead.
So I think tagging stream being unreadable in onFINReceived is too early.
I have no better idea for now, so I just comment this line to workaround this issue.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez dans src/muxer/yamux/yamux_stream.cpp, au niveau de onFINReceived et de la ligne référencée, puis suivez la façon dont les données restantes atteignent onLengthRead dans src/protocol/kademlia/impl/session.cpp. Vérifiez que le stream n’est pas marqué comme unreadable avant le traitement des données en attente ; le travail est terminé lorsque l’ordre entre FIN et l’état unreadable ne fait plus perdre ces données.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- networking
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 38/100