-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JsonRpcSigner.prototype.sendTransaction fails with "invalid key - from" #306
Comments
I’ll look into this today. Probably related to a fix put in yesterday; the from is added internally for calling estimateGas. Can you try explicitly adding a gasLimit to the transaction? |
I found the problem. I will be doing a small re-factor too which will fix this and make it easier for Signer authors. We also need a way to run regression tests against a node with accounts; this should have been caught in a test case. Sorry. :( In the future we will be able to use ethers-vm for better testing. |
getSigner().sendTransaction(...)
appears to be broken
A fix for this (along with tests in |
I'm going to close this now, but if you are still experiencing issues, please feel free to re-open. Thanks! :) |
Can confirm this is fixed. Thanks for doing this promptly! |
The
sendTransaction
method on anJsonRpcSigner
instance (from thegetSigner
method) appears to be broken:Error: invalid key - from (argument="transaction", value=...
The
from
key should not be required, and seems to be set up incorrectly by the signer.Can anyone else reproduce, or alternatively provide a working example?
Example usage:
The text was updated successfully, but these errors were encountered: