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

Cannot load Community Rooms if leaving room added #13335

Closed
KazamaSion opened this issue Apr 23, 2020 · 13 comments
Closed

Cannot load Community Rooms if leaving room added #13335

KazamaSion opened this issue Apr 23, 2020 · 13 comments
Labels
A-Spaces Spaces, groups, communities T-Defect

Comments

@KazamaSion
Copy link

KazamaSion commented Apr 23, 2020

Description

If I create a room with no local address, add the room to the community, then leave the room, the community cannot load the room list anymore.
It's not a network issue. A newly created community with no operations can see the room list (even with no rooms, it at least can load).
The only way to solve is to destroy and rebuild the community.

Temporary example of the bugged community: +ftqu8z5yo:matrix.org

Steps to reproduce

  • Create a room with no local address
  • Add the room to the community
  • Leave the room

Logs being sent: yes

Bugged community:

Normal Community:

Version information

  • Platform: web and desktop

For the web app:

  • Browser: Chrome 80
  • OS: Windows 10
  • URL: riot.im/app

For the desktop app:

  • OS: Windows
  • Version: 1.5.15
@prasket
Copy link

prasket commented May 8, 2020

I are also seeing this same behavior in one of the communities on a homeserver I help host. Taking this info I am going to dig into the database to see if there is a way to remove bad room from there to resolve. I will update with info I find.

@prasket
Copy link

prasket commented May 9, 2020

OK I can confirm that if you remove the rooms from the group_rooms table in postrgres the rooms will go back to showing again.

@hlesueur
Copy link

hlesueur commented May 9, 2020

Hello, I'm having the same issue, but my community (epb) is on matrix.org, so I have no access to the db
Anyone maintining the database would be able to help me removing this broken room from the database?
Thanks!

@KazamaSion
Copy link
Author

Since there's no dev replying to this issue, I think this is no the right place to feedback (maybe?)
So here is the question: where to submit this issue?

@hlesueur
Copy link

hlesueur commented May 9, 2020

I also put it in #riot-web:matrix.org, but no answer until now

I may have a suggestion on how to fix this, but that's nothing I can add on my own.
option 1:
upon quitting a room, make a simple test whether the room gets empty when the last user quits it , then if yes remove the room from the community it's attached to
(although I don't know how simple it is to test if it's in a community, unless the flair-badge is automatically set in the room settings when the room is added to the community ... anyways, I never got the meaning/purpose of the flair except for the esthetic of messages)

option2:
test the community in the database regularly for empty rooms , and remove them from the community and/or throw a message to the admin of the community with a link to the room.

@Twi1ightSparkle
Copy link

@hlesueur
Copy link

Hello @Twi1ightSparkle, thanks for the ticket. I've been watching zammad, but it's quite an empty room.
Is anyone looking at this? Should I do something?
thank you

@turt2live turt2live added the A-Spaces Spaces, groups, communities label May 12, 2020
@turt2live
Copy link
Member

Apologies for the delayed response here - we don't reply to every issue, nor do we spend a lot of time looking at issues related to the communities feature these days. The whole feature is slated for rewrite at the moment, which is why we're not spending much time triaging these things.

From a high-level glance it looks like this is probably a Synapse issue, and potentially a recentish regression. Considering their implementation has received just as much attention as ours (ie: very little), I am not all that surprised it has regressed, unfortunately.

Best advice I can give is to be careful about the rooms which get included in a community :(

@Twi1ightSparkle
Copy link

@hlesueur
We only use Zammad internally, which is why you don't see anything. I will check if we think doing the database fix suggested above is safe, and if yes I will do that for you.

@hlesueur
Copy link

Thanks for both your answers.
I understand you may have some load here! thanks for the work by the way.

@Twi1ightSparkle are you suggesting doing for me the fix (eg software fix), or doing the manual removal of empty room just for my broken db?
in any case, thanks

@Twi1ightSparkle
Copy link

Synapse issue matrix-org/synapse#4891

@hlesueur
Copy link

Thank you so much @Twi1ightSparkle it's repaired... !

@t3chguy
Copy link
Member

t3chguy commented May 13, 2021

This is fixed by Spaces landing in Beta in next Release

@t3chguy t3chguy closed this as completed May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Spaces Spaces, groups, communities T-Defect
Projects
None yet
Development

No branches or pull requests

6 participants