YamuxStream is tagged to Unreadable too early, since there may be data that hasn't been processed
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C++
- Estrellas
- 492
- Forks
- 130
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en src/muxer/yamux/yamux_stream.cpp, en onFINReceived y en la línea referenciada, y luego sigue cómo los datos restantes llegan a onLengthRead en src/protocol/kademlia/impl/session.cpp. Confirma que el stream no se marque como unreadable antes de procesar los datos pendientes; el trabajo estará terminado cuando el orden de FIN y del estado unreadable deje de perder esos datos.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- networking
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100