danielgtaylor / danielgtaylor/python-betterproto

Empty list cannot be parsed by `repeated`

Abierto
#509 1 comentario 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Python
Estrellas
1.8k
Forks
234
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Versions
```
betterproto[compiler]==2.0.0b6
grpcio-tools
pydantic==1.10.8
```

I get the following error on the server side:
```
023-07-10 15:48:08,980 - 140154842511168 - server.py-server:468 - ERROR: Application error
Traceback (most recent call last):
File "/worker-py/.direnv/python-3.10/lib/python3.10/site-packages/grpclib/server.py", line 446, in request_handler
await method_func(stream)
File "/worker-py/generated/__init__.py", line 157, in __rpc_teamlead_reply
request = await stream.recv_message()
File "/worker-py/.direnv/python-3.10/lib/python3.10/site-packages/grpclib/server.py", line 136, in recv_message
message = await recv_message(self._stream, self._codec, self._recv_type)
File "/worker-py/.direnv/python-3.10/lib/python3.10/site-packages/grpclib/stream.py", line 32, in recv_message
message = codec.decode(message_bin, message_type)
File "/worker-py/.direnv/python-3.10/lib/python3.10/site-packages/grpclib/encoding/proto.py", line 54, in decode
return message_type.FromString(data)
File "/worker-py/.direnv/python-3.10/lib/python3.10/site-packages/betterproto/__init__.py", line 1052, in FromString
return cls().parse(data)
File "pydantic/dataclasses.py", line 286, in pydantic.dataclasses._add_pydantic_validation_attributes.handle_extra_init
File "", line 4, in __init__
File "pydantic/dataclasses.py", line 308, in pydantic.dataclasses._add_pydantic_validation_attributes.new_post_init
File "pydantic/dataclasses.py", line 425, in pydantic.dataclasses._dataclass_validate_values
pydantic.error_wrappers.ValidationError: 1 validation error for GenerateRequest
conversation
value is not a valid list (type=type_error.list)
```

proto defintion
```proto
service ReplyGenerator {
rpc Reply(GenerateRequest) returns (stream StreamReply) {}
}

message GenerateRequest {
repeated Message messages = 2;
}
```

Calling it like this from client:
```python
async for res in service.teamlead_reply(guided_chat.GenerateRequest(messages=[])):
```

Compilation was done with:
```
PROTO_DIR=protos
GEN_DIR=generated
protofiles=$(find "${PROTO_DIR}" -name "*.proto")
python -m grpc_tools.protoc -I "${PROTO_DIR}" --python_betterproto_opt=pydantic_dataclasses --python_betterproto_out="${GEN_DIR}" ${protofiles}
```

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza con la llamada al análisis de mensajes en el __init__.py generado y luego inspecciona betterproto/__init__.py alrededor de FromString y parse, usando el ejemplo proporcionado GenerateRequest(messages=[]) y el comando protoc para reproducir el fallo. Se considera terminado cuando un campo repeated vacío se decodifica correctamente sin el error de validación de Pydantic.

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

Evaluación

Stack tecnológico
grpc, python
Área
api
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.