Allow custom message limits in flowkit client
@devbugging đang làm issue này rồi.
Từ ngày 17/7/2023.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Issue To Be Solved
Currently, the flowkit client uses a hardcoded max message size:
https://github.com/onflow/flow-cli/blob/6aa41a27add2d95066fc96072cc33c498f515deb/flowkit/gateway/grpc.go#L39
However, Access nodes can configure custom node specific limits to handle cases where data sizes exceed the default. This has come up several times in the last few months as data sizes have increased. Public ANs currently use a limit of 60MB.
(Optional): Suggest A Solution
We should make this limit configurable. A few potential backwards compatible solutions:
- Add a new constructor
NewCustomGrpcGatewaywhich supports passing in a client and the other fields. This could allow callers to build their own SDK client with the grpc settings they want, as well as pass their own context. - Add optional Option callbacks to allow customizing the grpc client options. This would be less flexible and require some more complex logic in the library to properly manage and pass the options into the SDK instantiation.
- Add an optional Option callback to allow specifying just the max message size. This would be simple to implement and add minimal changes to the code, but provides the least flexibility. However, it would solve the immediate problem in a simple way.
(Optional): Context
This limit is too low for GetTransactionResultByBlockID for some recent blocks, resulting the ResourceExhausted errors.
- Ngôn ngữ chính
- Go
- Star
- 209
- Fork
- 88
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của onflow/flow-cli
-
Simplify MCP contract sorting with slices helpersCó thể đã có người làm @luangucun đã nhận 42 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
`flow cadence lint` lints importsCó thể đã có người làm @holyfuchs đã nhận 181 ngày trước. Đang mởBug Improvement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
-
flow test prints unreadable outputCó thể đã có người làm @holyfuchs đã nhận 179 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
Feature
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
Tất cả issue của onflow/flow-cli
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
praetorian-inc/augustus#334 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
TestTOMLBigFileParsesInLinearTime: a 1s wall-clock bound on a 3 MB parse fails under load (1.27s)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100