Allow custom message limits in flowkit client
@devbugging がすでに取り組んでいます。
2023年7月17日 から。
評価
この issue はまだ評価されていません。
説明
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.
- 主要言語
- Go
- スター
- 209
- フォーク
- 88
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
onflow/flow-cli のほかの issue
-
Simplify MCP contract sorting with slices helpers対応中かも @luangucun が 42 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
`flow cadence lint` lints imports対応中かも @holyfuchs が 181 日前に担当しました。 オープンBug Improvement
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
Bug
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
-
flow test prints unreadable output対応中かも @holyfuchs が 180 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
-
Feature
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
onflow/flow-cli の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
limitlessprojects/Go-Hybrid#50 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
vinyldns/go-vinyldns#106 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
praetorian-inc/augustus#334 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ClickHouse/clickhouse-go#2066 ·
メンテナーはふだん 1 日以内に返信