element-hq / element-web

A glossy Matrix collaboration client for the web.
https://element.io
GNU Affero General Public License v3.0
11.23k stars 2k forks source link

Crashing - dendrite sending a null conditions like in spec #21305

Closed genofire closed 2 years ago

genofire commented 2 years ago

Steps to reproduce

  1. logout on other devices (android app)
  2. reset all site settings for element-wen
  3. login again
  4. .... crash during, maybe auth/recovery

Outcome

What did you expect?

Let's recovery with another device and sync all chats.

What happened instead?

Operating system

nixos

Browser information

Firefox

URL for webapp

https://element.sum7.eu

Application version

No response

Homeserver

dendrite main commit f75169c35

Will you send logs?

Yes

t3chguy commented 2 years ago

Will you send logs? Yes

I'm not seeing your logs.

t3chguy commented 2 years ago

Got them, thanks

genofire commented 2 years ago

Bugreporter of element.io does not work - it put that under:

https://github.com/matrix-org/element-web-rageshakes/issues/11279

Or somewhere else, here is it again (just log, tar is not allowed here):

log-0.log

t3chguy commented 2 years ago

Bugreporter of element.io does not work - it put that under:

It does work, it goes to a private repo on purpose.

t3chguy commented 2 years ago

The crash you are having is related to Dendrite sending a null conditions, the spec doesn't say it is required so the client is wrong to assert its presence

t3chguy commented 2 years ago

@genofire will be fixed by https://github.com/matrix-org/matrix-react-sdk/pull/7973