Allow custom message limits in flowkit client
@devbugging ci sta già lavorando.
Dal 17/7/2023.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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.
- Lingua principale
- Go
- Stelle
- 209
- Fork
- 88
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di onflow/flow-cli
-
Simplify MCP contract sorting with slices helpersForse già presa @luangucun l’ha presa 42 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
`flow cadence lint` lints importsForse già presa @holyfuchs l’ha presa 181 giorni fa. ApertaBug Improvement
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
-
flow test prints unreadable outputForse già presa @holyfuchs l’ha presa 180 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
-
Feature
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
Tutte le issue di onflow/flow-cli
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
limitlessprojects/Go-Hybrid#50 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
vinyldns/go-vinyldns#106 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
praetorian-inc/augustus#334 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ClickHouse/clickhouse-go#2066 ·
I maintainer di solito rispondono entro 1 giorno