Skip to content
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

Token transfer fails when no memo #792

Closed
Tracked by #807
plafer opened this issue Jul 28, 2023 · 0 comments · Fixed by #793
Closed
Tracked by #807

Token transfer fails when no memo #792

plafer opened this issue Jul 28, 2023 · 0 comments · Fixed by #793
Assignees
Labels
A: bug Admin: something isn't working A: critical Admin: critical or Important A: urgent Admin: high priority issue
Milestone

Comments

@plafer
Copy link
Contributor

plafer commented Jul 28, 2023

From @yito88:

Transfers between Namada chains or from Namada to Gaia with the ibc-rs work well.
Receiving the packet failed because the packet data couldn't be decoded in my local test. The ack had the following error:

{"error":"failed to deserialize packet data"}

The packet data from Gaia didn't have the "memo" field as below.

{"amount":"100","denom":"samoleans","receiver":"atest1d9khqw36xgeyxwz9geprxseegv6yvwp5xsmygs35xscng3fjxumnssenxvurwv2y89qnxs2p8yj8f3","sender":"cosmos19hav62pmlg53fxd3lryc768kwhde7l2ffc8rnf"}

Hermes command to transfer from Gaia:

hermes --config config_for_namada.toml tx ft-transfer --dst-chain namada-test.170f420af02f48796c --src-chain gaia-1 --src-port transfer --src-channel channel-0 --amount 100 --receiver atest1d9khqw36xgeyxwz9geprxseegv6yvwp5xsmygs35xscng3fjxumnssenxvurwv2y89qnxs2p8yj8f3 --denom samoleans --timeout-height-offset 1000

When I tried to transfer a token with --memo test from Gaia, receiving the packet worked well.

hermes --config config_for_namada.toml tx ft-transfer --dst-chain namada-test.170f420af02f48796c --src-chain gaia-1 --src-port transfer --src-channel channel-0 --amount 100 --receiver atest1d9khqw36xgeyxwz9geprxseegv6yvwp5xsmygs35xscng3fjxumnssenxvurwv2y89qnxs2p8yj8f3 --denom samoleans --timeout-height-offset 1000 --memo test

I'm using,
ibc-rs: v0.41.0
Hermes: forked based on v1.6.0
Gaia: v9.0.3

@plafer plafer added A: bug Admin: something isn't working A: urgent Admin: high priority issue A: critical Admin: critical or Important labels Jul 28, 2023
@github-project-automation github-project-automation bot moved this to 📥 To Do in ibc-rs Jul 28, 2023
@Farhad-Shabani Farhad-Shabani added this to the v0.43.0 milestone Jul 31, 2023
@github-project-automation github-project-automation bot moved this from 📥 To Do to ✅ Done in ibc-rs Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: bug Admin: something isn't working A: critical Admin: critical or Important A: urgent Admin: high priority issue
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants