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

Wallet crashes if you specify an incorrect port and close wallet. #744

Open
Denis-Asipenko opened this issue Jun 27, 2019 · 2 comments

Comments

2 participants
@Denis-Asipenko
Copy link

commented Jun 27, 2019

If you specify the wrong port in the node address(example: eu-node01.masternet.beam.mw:8104) and close the wallet, the wallet crashes.
Precondition: The wallet is installed and runs on a local node.
Steps to reproduce the behaviour:

  1. Go to the wallet.
  2. Navigate to the Settings screen.
  3. Click the switch from local to remote node.
  4. Enter node address(example: eu-node01.masternet.beam.mw:8104).
  5. Apply the changes and close the wallet.
    Actual: wallet crashes
    checked on win
    version: 2.1.5534
    Beam Wallet Masternet0.dmp.zip

@Denis-Asipenko Denis-Asipenko added the bug label Jun 27, 2019

@Denis-Asipenko Denis-Asipenko added this to To do in Desktop wallet improvements via automation Jun 27, 2019

@Denis-Asipenko Denis-Asipenko added this to To do in Clear Cathode 3.1 via automation Jun 27, 2019

@chapati

This comment has been minimized.

Copy link
Member

commented Jun 28, 2019

Also after you follow the steps above and run wallet again it would be stuck on loading forever and would always crash on close. Will fix this crash and looks like a separate bug for the always stuck wallet should be opened. Reproduced on Fedora Linux v30, should be reproduced on every OS.

chapati added a commit that referenced this issue Jul 1, 2019

@chapati

This comment has been minimized.

Copy link
Member

commented Jul 2, 2019

Graceful network - shutdown - b1244b8

@chapati chapati moved this from To do to In progress in Clear Cathode 3.1 Jul 4, 2019

@anatolse anatolse moved this from In progress to Done in Clear Cathode 3.1 Jul 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.