grpc / grpc/grpc-java

async server: need to ignore unary response msg if status is not OK

Abierto
#5,969 2 comentarios 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Java
Estrellas
12.1k
Forks
4k
Merge medio
2 d 17 h
PR fusionados (30 d)
37

Descripción

The expectation for unary responses is that the server will respond with an message or an error. If the server responds with a message and an error, the message should be thrown away. The async API currently would send the message. For unary responses, we should delay sending the response until the onComplete()/onError(). If onError() is called we should not send the message.

https://github.com/grpc/grpc-java/blob/b22017851560197a41015acd90f443f7b9519984/stub/src/main/java/io/grpc/stub/ServerCalls.java#L336-L349

See #5968 and grpc/grpc#12824

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza en stub/src/main/java/io/grpc/stub/ServerCalls.java, en las líneas 336-349, y sigue el procesamiento de respuestas unary asíncronas. Comprueba el comportamiento alrededor de onComplete() y onError() y verifica después que no se envíe un mensaje cuando lo acompaña un error; usa #5968 y grpc/grpc#12824 como contexto.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
api, backend
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.