GenesysGo / shadow-drive-cli

Shadow Drive CLI
Apache License 2.0
13 stars 10 forks source link

BUG: upload-file sends too many get-storage-account-info request at once #19

Closed tracy-codes closed 1 year ago

tracy-codes commented 1 year ago

Currently it's just an await Promise.all() with no rate limiting. This is okay for wallets with not very many storage accounts. This is not good for wallets with hundreds of storage accounts. Depending on the system defaults on the client-side (at the OS level, which we can't control) the system will time out these requests as we've observed.

We should implement a form of rate limiting that limits the number of in-flight requests from the CLI to resolve this.