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

Empty pairings.json file causes app to not start after pairing two devices #12342

Closed
jrainville opened this issue Oct 4, 2023 · 2 comments · Fixed by status-im/status-go#4113
Assignees
Labels
bug Something isn't working E:Desktop Messenger Syncing Issues about syncing Messenger data when recovering an account E:Desktop Wallet Sync Wallet account syncing
Milestone

Comments

@jrainville
Copy link
Member

Bug Report

Description

After doing a successful pairing and closing the app, it can no longer be started again, because the pairings.json file is empty.

Logs:

INF 2023-10-04 11:01:42.919-04:00 starting application controller...         topics="status-app" tid=246112 file=nim_status_client.nim:193
DBG 2023-10-04 11:01:42.919-04:00 init keycard using                         topics="keycard-service" tid=246112 file=service.nim:84 pairingsJson=/home/jonathan/dev/status-desktop/Status/data/keycard/pairings.json
DBG 2023-10-04 11:01:42.919-04:00 initialization response:                   topics="keycard-service" tid=246112 file=service.nim:87 initResp="unexpected end of JSON input"
Traceback (most recent call last)
/home/jonathan/dev/status-desktop/src/nim_status_client.nim(200) nim_status_client
/home/jonathan/dev/status-desktop/src/nim_status_client.nim(194) mainProc
/home/jonathan/dev/status-desktop/src/app/boot/app_controller.nim(402) start
/home/jonathan/dev/status-desktop/src/app/modules/startup/io_interface.nim(22) load
/home/jonathan/dev/status-desktop/src/app/modules/startup/module.nim(142) load
/home/jonathan/dev/status-desktop/src/app/modules/startup/io_interface.nim(112) setSelectedLoginAccount
/home/jonathan/dev/status-desktop/src/app/modules/startup/module.nim(300) setSelectedLoginAccount
/home/jonathan/dev/status-desktop/src/app/modules/startup/controller.nim(536) cancelCurrentFlow
/home/jonathan/dev/status-desktop/src/app_service/service/keycard/service.nim(147) cancelCurrentFlow
/home/jonathan/dev/status-desktop/vendor/nim-keycard-go/keycard_go.nim(21) keycardCancelFlow
SIGSEGV: Illegal storage access. (Attempt to read from nil?)
make: *** [Makefile:794: run-linux] Segmentation fault (core dumped)

Indeed, the file contains nothing.

Steps to reproduce

  1. Have an account ready to pair
  2. Open a new status instance
  3. Pair the two using the sync code
  4. All goes well
  5. Close the second app
  6. Try to reopen it

It doesn't open with the error above

Expected behavior

App opens normally, ready to login

Actual behavior

App doesn't start

Additional Information

  • Status desktop version: master (`c6449f67c622a824a95c9c58ed6a091181963820
  • Operating System: Ubuntu 20
@jrainville jrainville added the bug Something isn't working label Oct 4, 2023
@jrainville jrainville added this to the 0.15 milestone Oct 4, 2023
@jrainville jrainville added E:Desktop Wallet Sync Wallet account syncing E:Desktop Messenger Syncing Issues about syncing Messenger data when recovering an account labels Oct 4, 2023
@jrainville
Copy link
Member Author

@saledjenic does this issue ring a bell for you?

@saledjenic
Copy link
Contributor

@jrainville yes, thanks, it's a bug definitely, if the pairings.json is empty on the sender side we should either send {} (an empty json object as content) or don't send anything (don't create pairings.json file on the receiver side).

If you want me to take this over, just assign it to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working E:Desktop Messenger Syncing Issues about syncing Messenger data when recovering an account E:Desktop Wallet Sync Wallet account syncing
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants