-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: Lotus Send CLI: Lotus send should work with ETH address receipients #12319
Conversation
@rvagg Also tested this with a smart contract as a receipient (using both ETH and non-ETH sender) and that also works fine. |
Seems pretty straightforward; in terms of enabling what we want to work properly at least. You followed up with each of those sends and confirmed that the recipient received it? Is this something we could capture in a simple itest, not necessarily using the CLI but using the send logic for the various cases. |
Co-authored-by: Rod Vagg <rod@vagg.org>
|
@@ -128,7 +147,7 @@ var SendCmd = &cli.Command{ | |||
params.Params = decparams | |||
} | |||
|
|||
if ethtypes.IsEthAddress(params.From) { | |||
if ethtypes.IsEthAddress(params.From) || ethtypes.IsEthAddress(params.To) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is going to cause some issues.
- There are almost certainly users using native accounts to send messages to EVM smart contracts, explicitly specifying the method number.
- Implicitly calling
InvokeContract
is the nice thing to do, but that's a significant change of behavior.
Ideally this would have this method number handling if and only if the user passes a literal 0x...
address in the to field. If they pass a normal Filecoin address, they should get the normal Filecoin method handling.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pulled this up into a new issue #12326
This fixes the Lotus send CLI to work with ETH address recipients (both
0xff...
corresponding to Filecoinf0
addresses and0xabcd
corresponding to Filecoinf410f
addresses).Have tested this locally on a local devnet.