danielgtaylor / danielgtaylor/python-betterproto

Mixed case request results in missing request parameters in generated code

Đang mở
#271 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
compiler-bug
Ngôn ngữ chính
Python
Star
1.8k
Fork
234
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Version: betterproto==2.0.0b3

Requests with capital letters in the name seem to cause an issue with constructing methods. The request parameters are not added to the method. e.g

proto file:
```
rpc TestCAPITALS(TestCAPITALSRequest) returns (TestCAPITALSResponse){}

message TestCAPITALSRequest{
string name = 1;
}

message TestCAPITALSResponse{}
```

generated class:
```
async def test_capitals(self) -> "TestCapitalsResponse":

request = TestCapitalsRequest()

return await self._unary_unary(
"/tumelo.transparency.v1.InstrumentPortfolioTransparency/TestCAPITALS",
request,
TestCapitalsResponse,
)
```
we would expect a `name` field in the request but it is not there

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu với các định nghĩa RPC và request có kiểu chữ hoa-thường hỗn hợp được hiển thị trong issue, sau đó so sánh chúng với phương thức async được tạo và quá trình xây dựng request. Theo dõi đường dẫn sinh mã biến tên RPC và tên message thành phương thức, đồng thời xác nhận rằng request được tạo vẫn giữ trường name. Được xem là hoàn tất khi phương thức ví dụ có tham số request, đồng thời vẫn giữ đường dẫn RPC và kiểu response.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
grpc, python
Lĩnh vực
devtools
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.