danielgtaylor / danielgtaylor/python-betterproto

Capitalized (Title case) package name causes invalid gRPC code generation

Abierto
#272 0 comentarios 4 reacciones 0 asignados Ver en GitHub
compiler-bug
Lenguaje dominante
Python
Estrellas
1.8k
Forks
234
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Working with gRPC on `v2.0.0b3`, if the `.proto` package name starts with a capital letter, e.g.
``` proto
package Echo;
```
Then the generated file becomes significantly different and invalid. Most obviously there is `from .. import MessageName as _MessageName__` for the message types and it can't be imported.

For example, take the `echo.proto` example from the README.md and generate as suggested with `python -m grpc_tools.protoc -I . --python_betterproto_out=. echo.proto`, it seems fine, but edit it to `package Echo;` and in `Echo/__init__.py` you get, among other changes not limited to `s/echo/Echo`, at the end:

``` py
from .. import EchoEchoRequest as _EchoEchoRequest__
from .. import EchoEchoResponse as _EchoEchoResponse__
from .. import EchoEchoStreamResponse as _EchoEchoStreamResponse__
```

These don't exist, and the package can't be imported.

Found in `v2.0.0b3`. Thanks!

----
For completeness, here's the problematic `.proto`
``` proto
syntax = "proto3";

package Echo;

message EchoRequest {
string value = 1;
// Number of extra times to echo
uint32 extra_times = 2;
}

message EchoResponse {
repeated string values = 1;
}

message EchoStreamResponse {
string value = 1;
}

service Echo {
rpc Echo(EchoRequest) returns (EchoResponse);
rpc EchoStream(EchoRequest) returns (stream EchoStreamResponse);
}
```

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza con el ejemplo de README.md echo.proto y reproduce el problema utilizando el comando documentado python -m grpc_tools.protoc después de cambiar el paquete a Echo. Inspecciona el Echo/__init__.py generado, especialmente los imports mostrados en el informe. Se considera terminado cuando se pueda importar la salida con el paquete escrito en mayúsculas y las referencias a los mensajes generados se resuelvan correctamente.

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

Evaluación

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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.