Allow custom message limits in flowkit client
@devbugging arbeitet bereits daran.
Seit 17.7.2023.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
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.
- Vorherrschende Sprache
- Go
- Sterne
- 209
- Forks
- 88
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus onflow/flow-cli
-
Simplify MCP contract sorting with slices helpersEvtl. vergeben @luangucun hat das vor 43 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
`flow cadence lint` lints importsEvtl. vergeben @holyfuchs hat das vor 182 Tagen übernommen. OffenBug Improvement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
-
flow test prints unreadable outputEvtl. vergeben @holyfuchs hat das vor 181 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
-
Feature
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
Alle Issues in onflow/flow-cli
Ähnliche Issues
-
area: watch bug priority: low
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
mariadb-operator/mariadb-operator#1962 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
eugenioenko/ttt#919 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
QuantumNous/new-api#7839 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs-triage service/costoptimizationhub
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
hashicorp/terraform-provider-aws#50507 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag