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

Delete session bug #4027

Open
waclaw66 opened this issue Feb 13, 2025 · 3 comments · May be fixed by #4067
Open

Delete session bug #4027

waclaw66 opened this issue Feb 13, 2025 · 3 comments · May be fixed by #4067

Comments

@waclaw66
Copy link

waclaw66 commented Feb 13, 2025

Describe the bug
There is a bug that leads to
Error: [{"message":"Database inconsistency on table compat_sessions column device_id row 01HQ3FB21QVAPX9E710P93212F","locations":[{"line":7,"column":7}],"path":["viewer","appSessions"]}]
while listing user account sessions on MAS

To Reproduce
Steps to reproduce the behavior:

  1. Delete sessions from Element Web and MAS simultaneously. Those sessions were created by failed QR code device pairing described here:
    Link device QR code fails with 412 M_CONCURRENT_WRITE synapse#18155

Expected behavior
No crash

Screenshots

Server:

  • OS: Fedora 41
  • Browser: not relevant
  • Version 0.13.0 (docker image)
@sandhose
Copy link
Member

This is likely to be #2724

Could you look at what the device_id is for this session? The query would be

SELECT device_id
FROM compat_sessions
WHERE compat_session_id = '018DC6F5-8837-DAAD-D4B8-E1059231044F';

The compat_session_id is the one logged, converted from ULID to UUID using tools like this one: https://www.ulidtools.com

@waclaw66
Copy link
Author

waclaw66 commented Feb 13, 2025

It's Signal Bridge, what does it mean considering the error message?
Can I do something to prevent such errors in the future when I reintroduce MAS on my Matrix instance?

@sandhose sandhose linked a pull request Feb 19, 2025 that will close this issue
@sandhose
Copy link
Member

So the problem are device IDs that have a space in them. I opened a PR to support those: #4067

It will also require this on the Synapse side: element-hq/synapse#18174

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants