Skip to content
This repository has been archived by the owner on Oct 19, 2024. It is now read-only.

Cannot spawn futures #77

Closed
gakonst opened this issue Oct 1, 2020 · 0 comments · Fixed by #81
Closed

Cannot spawn futures #77

gakonst opened this issue Oct 1, 2020 · 0 comments · Fixed by #81
Labels
bug Something isn't working

Comments

@gakonst
Copy link
Owner

gakonst commented Oct 1, 2020

Due to async_trait(?Send) being used in Signer, futures cannot be spawned. This is done because Ledger's futures are not Send. We could block_on on the .exchange call in the ledger to get around that, but maybe there's a better way (and it could be problematic? @prestwich)
@gakonst gakonst added the bug Something isn't working label Oct 1, 2020
@gakonst gakonst mentioned this issue Oct 6, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Transaction Gas Price Escalator middleware gakonst/ethers-rs
1 participant
@gakonst