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

Can't swap some tokens on Uniswap due to json: cannot unmarshal number into Go struct field ... when confirming amount of tokens on your account #12413

Closed
churik opened this issue Aug 3, 2021 · 3 comments · Fixed by #12417
Assignees
Milestone

Comments

@churik
Copy link
Member

churik commented Aug 3, 2021

Bug Report

Problem

To swap tokens on Uniswap you need to confirm amount by signing a message.
When signing got an error
json: cannot unmarshal number into Go struct field TypedDataDomain.domain.chainId of type *math.HexOrDecimal256
It happens only to tokens, which require signing of json (i.e. UNI, DAI)

Maybe related to #11742 (not sure)

Expected behavior

can swap tokens

Actual behavior

IMAGE 2021-08-03 15:44:59

Reproduction

  1. Open Status (restore multiaccounts with some assets, in my case it was UNI)
  2. Connect wallet
  3. Try to swap UNI to ETH

Additional Information

  • Status version: nightly 03/08/2020
  • Operating System: Android, iOS
  • Logs: status_logs.zip
@churik churik added bug labels Aug 3, 2021
@bitgamma
Copy link
Member

bitgamma commented Aug 3, 2021

seems to be a regression in our fork of go-ethereum, probably because when upgrading our fork of go-ethereum this commit status-im/go-ethereum@1d0a734 hasn't been applied

@cammellos is it OK to re-apply it by branching off v1.10.4-status.1 or was there a specific reason why this wasn't merged?

@cammellos
Copy link
Contributor

cammellos commented Aug 3, 2021 via email

@bitgamma bitgamma self-assigned this Aug 3, 2021
@bitgamma
Copy link
Member

bitgamma commented Aug 3, 2021

Ok will do

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

Successfully merging a pull request may close this issue.

3 participants