status-im / status-mobile

a free (libre) open source, mobile OS for Ethereum
https://status.app
Mozilla Public License 2.0
3.88k stars 984 forks source link

Error when creating an account on e2e build #20806

Open yevh-berdnyk opened 2 months ago

yevh-berdnyk commented 2 months ago

Bug Report

Problem

Sometimes when creating an account we get a popup with error:

Screenshot 2024-07-17 at 19 13 02

Expected behavior

No error

Actual behavior

Please shake your phone to report this error... is shown

Reproduction

  1. Create a new account

Additional Information

Logs: logcat_shake_the_phone_error.log

qfrank commented 1 day ago

the version seems old in the screenshot you provided? @yevh-berdnyk

I can't find the usage of "start-using-status" in the cljs file on the latest develop branch. Am I missing something? @Parveshdhull

image
Parveshdhull commented 1 day ago

Yes, the screenshot is old, but I also encountered this error recently, just two days ago. Status-debug-logs.zip

Note: In my case, I might have installed the wrong build while working on the syncing fallback flow, so the logs may not be very helpful and error might have been expected.

Parveshdhull commented 1 day ago

Ok, as per logs shared by @yevh-berdnyk

07-17 19:12:04.124 24559 24667 E ReactNativeJS: 'An error occured while handling the re-frame event:', '[:visibility-status-updates/visibility-status-option-pressed 1]', '\n', 'Within the', ':visibility-status-updates/visibility-status-option-pressed', 'event handler function.'

I removed this call in https://github.com/status-im/status-mobile/pull/21068, so this issue should be fixed

Parveshdhull commented 1 day ago

@yevh-berdnyk @churik are you still facing this issue?

Parveshdhull commented 1 day ago

I removed this call in https://github.com/status-im/status-mobile/pull/21068, so this issue should be fixed

But in my case, error is

'An error occured while handling the re-frame event:', '[:push-notifications/switch false]', '\n', 'Within the', ':push-notifications/switch', 'event handler function.'
Parveshdhull commented 1 day ago

So most probably, problem is with multiaccounts.update/multiaccount-update. As both functions are using this. So we trying to update an no existing multiaccount, and this is causing the crash.

Parveshdhull commented 1 day ago

Ok, real error in my case

node.login error failed to start node: failed to apply migrations: failed to apply status-go/protocol migrations: failed to migrate: file does not exist
Parveshdhull commented 1 day ago
node.login error failed to start node: failed to apply migrations: failed to apply status-go/protocol migrations: failed to migrate: file does not exist

But as I was working on seed phrase fallback and trying different builds, this probably is expected in my case. So if no one else faces this issue, we can close it?

yevh-berdnyk commented 23 hours ago

@yevh-berdnyk @churik are you still facing this issue?

Yes, a lot of. For example, here are the logs from Sep 19 nightly build: requests.log geth.log The pop-up appeared when creating a user

Parveshdhull commented 23 hours ago

thank you @yevh-berdnyk for confirming. Do you access to Status.log file

image

yevh-berdnyk commented 23 hours ago

thank you @yevh-berdnyk for confirming. Do you access to Status.log file

image

Not for now. We gather geth and requests logs by script, if you need it, I can add gathering Status.log as well