YamuxStream is tagged to Unreadable too early, since there may be data that hasn't been processed
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- networking
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.
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.
- Lenguaje dominante
- C++
- Estrellas
- 492
- Forks
- 130
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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.
Más de libp2p/cpp-libp2p
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
libp2p/cpp-libp2p#351 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
libp2p/cpp-libp2p#347 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
libp2p/cpp-libp2p#341 ·
-
Lack of signedPeerRecord Validation in C++ IdentifyMessageProcessor Leads to Address Injection Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
libp2p/cpp-libp2p#332 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
libp2p/cpp-libp2p#324 · 1 comentario · 1 reacción ·
Todos los issues de libp2p/cpp-libp2p
Issues similares
-
[CI] Nightly Clang build broken: structured binding captured in a lambda in FileDataSource.cpp Abiertobug build
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
facebookincubator/velox#19194 ·
-
JIT-compiled number -> Decimal conversion silently overflows instead of raising DECIMAL_OVERFLOW Abiertofuzz
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
ClickHouse/ClickHouse#122114 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
module/agent platform/macos type/bug/regression
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
enhancement PyCDE
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100