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

Cannot IBC 100+ USDY to Penumbra #404

Closed
UnityChaos opened this issue Sep 3, 2024 · 6 comments
Closed

Cannot IBC 100+ USDY to Penumbra #404

UnityChaos opened this issue Sep 3, 2024 · 6 comments

Comments

@UnityChaos
Copy link

Appears to be a bug once total raw amount goes over 10^20 due to 18 decimals on USDY.

Screenshot from 2024-09-03 13-45-27

@johnletey
Copy link
Member

What message are you attempting to execute?

@zmanian
Copy link
Contributor

zmanian commented Sep 4, 2024

I'm 90% sure that this happening during a call to simulate tx here.

https://github.com/penumbra-zone/web/blob/1954cea252daf6c9472598c4fc5aaec761c23247/apps/minifront/src/state/ibc-in/index.tsx#L173-L176

@grod220
Copy link

grod220 commented Sep 4, 2024

Here is an example message:

{
    "typeUrl": "/ibc.applications.transfer.v1.MsgTransfer",
    "value": {
        "sourcePort": "transfer",
        "sourceChannel": "channel-89",
        "sender": "noble1tnhf37dg646zlr4g54hsp5n2qrchgadk3dzelq",
        "receiver": "penumbracompat11tawd9ru9kvja9wp604zjyq9fu37kh5tmx964deeff0clu35zr69k6k3agmznxz3smszrdcyydpkuepjng06z7arc9akke6aa6njkzrry0car67h5u0chykyjlvwg520e0j83ug",
        "token": {
            "denom": "ausdy",
            "amount": "1.12e+20"
        },
        "timeoutHeight": {
            "revisionNumber": "1",
            "revisionHeight": "944693"
        },
        "timeoutTimestamp": "1725627000000000000",
        "memo": ""
    }
}

The cosmjs SigningStargateClient is not able to simulate the fee for this transfer 🤔 . It throws precisely here.

@grod220
Copy link

grod220 commented Sep 4, 2024

Here is the bug, it's on the penumbra frontend side:

   "token": {
            "denom": "ausdy",
            "amount": "1.12e+20"
        },

Amount should not be formatted in that way. Can close this here. Will fix on the other end.

==

EDIT: PR to fix penumbra-zone/web#1766

@zmanian
Copy link
Contributor

zmanian commented Sep 12, 2024

This can be closed.

@johnletey
Copy link
Member

Thanks everyone for trouble shooting, especially @grod220 🙌🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants