-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Element displays error that it failed to restore the session when the network is not yet connected when it launches #26967
Comments
Step to reproduce: Put a breakpoint in looks like the errors bubbles up and is finally caught in |
Looks like there might be an issue with getVersions(): There are several places where version is called and could reject:
There are other places where it's check but there is some basic retries, like in DeviceListener > cli.isVersionSupported("v1.1") Maybe there should be some global retry on getVersion, and also throw something specific to be checked against in handleLoadSessionFailure and show something else? |
I saw this this morning; my server was briefly returning 500s in response to I feel like this has been mis-traged. Clearly it's happening semi-frequently; and it's really, really bad because it encourages the user to throw away all their state. |
This would not have been much of a problem if dehydrated devices was a stable core feature of Element from the start. But we don't live in that parallel universe. |
@t3chguy suggests that maybe this was introduced by us |
Regressed in matrix-org/matrix-react-sdk#12154 which adds an await with no explanation. |
Move the server versions check to each time we reconnect to the server rather than the first time,although, as per comment it will still only trigger the first time, but it will avoid us awaiting and mean we know we're connected to the server when we try, and get automatic retries. Fixes element-hq/element-web#26967
Is it possible that during this error another sync handler is started up? If so, then I think this is indirectly the cause of the recent presence load issue. When I saw this error, was I able to click the red button and then 'cancel' on the next window which allowed me to not have to sign out and back in, but did lead to the behavior I saw that lead me to connect the dots. Full client restart mitigated it back to normal. |
* Fix spurious session corruption error Move the server versions check to each time we reconnect to the server rather than the first time,although, as per comment it will still only trigger the first time, but it will avoid us awaiting and mean we know we're connected to the server when we try, and get automatic retries. Fixes element-hq/element-web#26967 * Move test & add regression test * Write some more tests * More comments & catch exceptions in server versions check * Note caching behaviour * Typo Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com> * Remove the bit of the comment that might be wrong --------- Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
* Fix spurious session corruption error Move the server versions check to each time we reconnect to the server rather than the first time,although, as per comment it will still only trigger the first time, but it will avoid us awaiting and mean we know we're connected to the server when we try, and get automatic retries. Fixes element-hq/element-web#26967 * Move test & add regression test * Write some more tests * More comments & catch exceptions in server versions check * Note caching behaviour * Typo Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com> * Remove the bit of the comment that might be wrong --------- Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com> (cherry picked from commit 1403cd8)
Steps to reproduce
Outcome
What did you expect?
A message that tells me that my internet is not connected
What happened instead?
A error screen appeared that suggests me to clear the storage and login again.
Operating system
Windows
Application version
Version von Element: 1.11.57
Krypto-Version: Olm 3.2.15
How did you install the app?
A long tim ago from the official website
Homeserver
local Synapse v1.99.0
Will you send logs?
Yes
The text was updated successfully, but these errors were encountered: