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

app.tornado.cash crashes app on Android with 'java.lang.illegalArgumentException' #11565

Closed
churik opened this issue Dec 21, 2020 · 6 comments · Fixed by #12987
Closed

app.tornado.cash crashes app on Android with 'java.lang.illegalArgumentException' #11565

churik opened this issue Dec 21, 2020 · 6 comments · Fixed by #12987
Assignees
Labels

Comments

@churik
Copy link
Member

churik commented Dec 21, 2020

Bug Report

Problem

Initially reported by a user.
At attempt to deposit ETH via dapp app.tornado.cash status crashes with error:
Screenshot_20201220-105743_Status
Works with MetaMask in Chrome and with Status on IOS.

Expected behavior

can deposit ETH or gracefully handled error

Actual behavior

crash

Reproduction

  • Open Status
  • Restore multiaccount with seed phrase (with at least 0.1 ETH to check the case)
  • Open app.tornado.cash
  • Connect wallet
  • Try to deposit 0.1 ETH

Additional Information

  • Status version: nightly 21/12/20, release 1.9.0
  • Operating System: Android only

Logs

Status-debug-logs.zip

@churik churik changed the title app.tornado.cash crashes status app app.tornado.cash crashes status app on Android Dec 21, 2020
@churik churik changed the title app.tornado.cash crashes status app on Android app.tornado.cash crashes app on Android Dec 21, 2020
@churik churik changed the title app.tornado.cash crashes app on Android app.tornado.cash crashes app on Android with 'java.lang.illegalArgumentException' Dec 21, 2020
@flexsurfer
Copy link
Member

@churik
Copy link
Member Author

churik commented May 11, 2021

still relevant on 1.13

@churik
Copy link
Member Author

churik commented Jul 27, 2021

on 1.15 the crash is happening on "Setup account" stage
IMAGE 2021-07-27 13:08:03

@churik
Copy link
Member Author

churik commented Aug 3, 2021

Reproducible on #12409

@bitgamma
Copy link
Member

bitgamma commented Aug 3, 2021

This is a known problem with Android webview not supporting blob URL's for download. On react-native-webview's side this is being fixed by react-native-webview/react-native-webview#2091

@bitgamma bitgamma self-assigned this Jan 10, 2022
@bitgamma
Copy link
Member

bitgamma commented Jan 10, 2022

the bug in react-native-webview should be fixed upstream. Since we are using a fork and made some changes integrating the fix will probably not be trivial. Ideally, I want to try to avoid using our fork altogether since in the meantime upstream has received many improvements and seems to also implement the functionality we implemented in our fork (although probably is not 1-to-1 compatible, let's see). I updated our fork and applied our changes

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

Successfully merging a pull request may close this issue.

3 participants