-
Notifications
You must be signed in to change notification settings - Fork 11
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
Remove/delete rooms & spaces #466
Comments
This is more of a server concern than the client. The server has the opportunity to delete rooms that have been abandoned, and synapse (at least) doesn't use this opportunity. The best issue I can find that relates is: matrix-org/synapse#2338 (I could have sworn there was an issue for deleting rooms when they are abandoned though) |
So could I. Anyway, matrix-org/synapse#4720, until we track it down. |
Maybe "forcing abandonment" would be a different name then. If the admin of a room wants to make it "gone", they should be able to easily remove all its users, "lock" it so no one can resurrect it and suggest the server to drop all history it knows. |
For everyone who search for a secure solution to delete a room (or something nearby), I create a php script as replacement for the php riot_delete_room.php https://matrix.org "MDA...kYK" "#alias:domain.de" The script only shows all members and the stop and ask if the members should kick. Then ask for a reason and a possible userId which should not kicked. |
It's weird for end-users not to be able to "delete" a room: it does not really matter if the room is left abandonned, but it would make sense for admins to have a "delete" button that kicks everyone, lock and leave. Homeserver GC can take care of the rest, but I think it doesn't matter from the user point of view. It should not be hard to implement in riot. |
It would be hard because as soon as there's multiple admins one admin cannot do this on their own, all admins would have to leave/derank themselves then the final one click this button. Admins cannot kick each other. |
Good point. Issue we got at work today: someone created a duplicated room by error, and invited 5-8 users. It was confusing for them, as they had to do everything by hand. We could add an "Abandon room" button in settings for rooms with a single admin, which should already cover most use cases. Not perfect, but still better than the current situation. Would you be open to such a patch? |
I don't see this as a big enough hurdle to block an expected feature. If there's 2+ admins, just fade out the delete button with a message to reduce to 1 admins in order to delete it. |
My use case for deleting a room was: I created a room with encryption on. Now I want to have no encryption (because I do not want to veryfy every user). This is not possible. So I want to delete the room, but create a new one with the same name(!), then without encryption. Would that be possible? |
Just another voice echoing a use case similar to kollokollo's. I mentioned this in the #matrix channel what I did and brought up the usability issue of letting (at least in Element) a user set encrypt a public room with a published address in the first place. But yes I would like to delete (or have Matrix clear the empty room) so that I can reclaim the published address and do it correctly the second time around. Thanks all. |
In the meantime, I've discovered there seems to be an API in Synapse to do just that: https://github.com/matrix-org/synapse/blob/develop/docs/admin_api/rooms.md#delete-room-api I wonder why this can't somehow be wired to show up as a button for admins in the frontend? Shouldn't be too hard to do... |
Because its a Synapse specific (not Matrix) API and requires a Matrix Access Token of a Synapse Admin and the |
Thanks a lot for the explanation @t3chguy . Understood. But I thought Synapse was the most widespread server a.k.a. "standard implementation"? Couldn't Element detect, if this server feature is available and then add this as an additional menu entry/delete button? |
Sure, but Element strives to be a Matrix client, not a Synapse one. |
It's embarrassing that this is such a longstanding issue - it's a common general complaint against Matrix that "you can't even delete rooms!!!". I've been thinking about it today, and may have a way forward:
As @t3chguy mentioned above, exposing the server admin features is a bit unpleasant as they are currently synapse specific, but better that than not have the feature exposed at all. https://github.com/matrix-org/matrix-doc/issues/1411 tracks speccing server admin functions properly. Finally, a common reason to 'remove a room' is actually to merge it into another one. Therefore a second 'merge room' button could be added in Room Settings which sends a tombstone into the room to point it at a destination. |
As a user my expectation of "remove room" would be just that: the room should not exist anymore. (not having any members, not be join-able by anyone, deleted after some grace-period)
The management of aliases (actually called Addresses in the UI) is a different issue imo and I would not click a "delete room" button when I search for that feature. I would expect a workflow like this:
Should Element-Web really include administration-functionality? There are issues (#1237, matrix-org/synapse#2032) requesting a dedicated interface for such things. |
please, please, please, don't use synapse's admin APIs in clients. Once you do so, other clients are pressurised to do the same, and they become a defacto part of the spec. (Also: synapse's deployment instructions specifically advise against exposing the admin API to the internet, since it gives attackers who get hold of an admin's access token a huge amount of power.) |
Element already uses some Synapse Admin APIs @richvdh |
that is very much a bug, not a pattern to be repeated. |
It was done by product demand for a |
Given https://matrix.org/docs/spec/client_server/r0.6.1#get-matrix-client-r0-admin-whois-userid already exists, the second of those is particularly surprising. |
yes I appreciate there are good reasons for wanting these features, but the way to do it is with specced APIs, not by skirting around the spec process by using unspecced APIs. |
The only thing I really don't understand is: Why is it a non-admin/general-user feature to create a room but there are so many complications around just doing the opposite and reversing the action I might have just done? It seems weird to me in the first place to include one thing in the spec and not the other. |
@rgpublic because a room admin has shared control of the room with other room admins (users of equal permission) - you cannot kick or demote them, this is to prevent a rogue admin/hacked account from doing irreversible damage. The ability for an admin to delete a room would cause irreversible damage. In any case, a discussion for the matrix spec and not Element |
Even using the Admin API to delete a room turns up odd behaviour. It may be useful to track that as a separate issue from a room admin trying to delete the room but I'll put this here for now.
Suggestions that would be possible without spec change:
Possible spec changes:
|
@hex-m that's because rooms don't exist/belong on a single server. They are decentralized amongst all servers which have users in that room, so you're only deleting your own server's copy of that room. Synapse supports operations for preventing reaccessing the room from an invite etc, offtopic for this repo. Your suggestions apply to Synapse specific implementation and the Matrix spec, not for this issue. I suggest spinning up issues in relevant repositories for those. |
* Make double-clicking the PiP take you to the call room ([\element-hq#7142](matrix-org/matrix-react-sdk#7142)). Fixes element-hq#18421 element-hq#15920 and element-hq#18421. Contributed by @SimonBrandner. * Add maximise widget functionality ([\element-hq#7098](matrix-org/matrix-react-sdk#7098)). Fixes element-hq#19619, element-hq#19621 element-hq#19760 and element-hq#19619. * Add rainfall effect ([\element-hq#7086](matrix-org/matrix-react-sdk#7086)). Contributed by @justjosias. * Add root folder to zip file created by export chat feature ([\element-hq#7097](matrix-org/matrix-react-sdk#7097)). Fixes element-hq#19653 and element-hq#19653. Contributed by @aaronraimist. * Improve VoIP UI/UX ([\element-hq#7048](matrix-org/matrix-react-sdk#7048)). Fixes element-hq#19513 and element-hq#19513. Contributed by @SimonBrandner. * Unified room context menus ([\element-hq#7072](matrix-org/matrix-react-sdk#7072)). Fixes element-hq#19527 and element-hq#19527. * In forgot password screen, show validation errors inline in the form, instead of in modals ([\element-hq#7113](matrix-org/matrix-react-sdk#7113)). Contributed by @psrpinto. * Implement more meta-spaces ([\element-hq#7077](matrix-org/matrix-react-sdk#7077)). Fixes element-hq#18634 element-hq#17295 and element-hq#18634. * Expose power level control for m.space.child ([\#7120](matrix-org/matrix-react-sdk#7120)). * Forget member-list query when switching out of a room ([\element-hq#7093](matrix-org/matrix-react-sdk#7093)). Fixes element-hq#19432 and element-hq#19432. Contributed by @SimonBrandner. * Do pre-submit availability check on username during registration ([\#6978](matrix-org/matrix-react-sdk#6978)). Fixes element-hq#9545 and element-hq#9545. * Adjust recovery key button sizes depending on text width ([\element-hq#7134](matrix-org/matrix-react-sdk#7134)). Fixes element-hq#19511 and element-hq#19511. Contributed by @weeman1337. * Fix bulk invite button getting a negative count ([\element-hq#7122](matrix-org/matrix-react-sdk#7122)). Fixes element-hq#19466 and element-hq#19466. Contributed by @renancleyson-dev. * Fix maximised / pinned widget state being loaded correctly ([\element-hq#7146](matrix-org/matrix-react-sdk#7146)). Fixes element-hq#19768 and element-hq#19768. * Don't reload the page when user hits enter when entering ban reason ([\element-hq#7145](matrix-org/matrix-react-sdk#7145)). Fixes element-hq#19763 and element-hq#19763. * Fix timeline text when sharing room layout ([\element-hq#7140](matrix-org/matrix-react-sdk#7140)). Fixes element-hq#19622 and element-hq#19622. * Fix look of emoji verification ([\element-hq#7133](matrix-org/matrix-react-sdk#7133)). Fixes element-hq#19740 and element-hq#19740. Contributed by @SimonBrandner. * Fixes element not remembering widget hidden state per room ([\#7136](matrix-org/matrix-react-sdk#7136)). Fixes element-hq#16672, matrix-org/element-web-rageshakes#4407, element-hq#15718 element-hq#15768 and element-hq#16672. * Don't keep spinning if joining space child failed ([\element-hq#7129](matrix-org/matrix-react-sdk#7129)). Fixes matrix-org/element-web-rageshakes#6813 and matrix-org/element-web-rageshakes#6813. * Guard around SpaceStore onAccountData handler prevEvent ([\element-hq#7123](matrix-org/matrix-react-sdk#7123)). Fixes element-hq#19705 and element-hq#19705. * Fix missing spaces in threads copy ([\element-hq#7119](matrix-org/matrix-react-sdk#7119)). Fixes element-hq#19702 and element-hq#19702. * Fix hover tile border ([\element-hq#7117](matrix-org/matrix-react-sdk#7117)). Fixes element-hq#19698 and element-hq#19698. Contributed by @SimonBrandner. * Fix quote button ([\element-hq#7096](matrix-org/matrix-react-sdk#7096)). Fixes element-hq#19659 and element-hq#19659. Contributed by @SimonBrandner. * Fix space panel layout edge cases ([\#7101](matrix-org/matrix-react-sdk#7101)). Fixes element-hq#19668 and element-hq#19668. * Update powerlevel/role when the user changes in the user info panel ([\element-hq#7099](matrix-org/matrix-react-sdk#7099)). Fixes element-hq#19666 and element-hq#19666. Contributed by @SimonBrandner. * Fix avatar disappearing when setting a room topic ([\element-hq#7092](matrix-org/matrix-react-sdk#7092)). Fixes element-hq#19226 and element-hq#19226. Contributed by @SimonBrandner. * Fix possible infinite loop on widget start ([\#7071](matrix-org/matrix-react-sdk#7071)). Fixes element-hq#15494 and element-hq#15494. * Use device IDs for nameless devices in device list ([\element-hq#7081](matrix-org/matrix-react-sdk#7081)). Fixes element-hq#19608 and element-hq#19608. * Don't re-sort rooms on no-op RoomUpdateCause.PossibleTagChange ([\element-hq#7053](matrix-org/matrix-react-sdk#7053)). Contributed by @bradtgmurray.
* Make double-clicking the PiP take you to the call room ([\element-hq#7142](matrix-org/matrix-react-sdk#7142)). Fixes element-hq#18421 element-hq#15920 and element-hq#18421. Contributed by @SimonBrandner. * Add maximise widget functionality ([\element-hq#7098](matrix-org/matrix-react-sdk#7098)). Fixes element-hq#19619, element-hq#19621 element-hq#19760 and element-hq#19619. * Add rainfall effect ([\element-hq#7086](matrix-org/matrix-react-sdk#7086)). Contributed by @justjosias. * Add root folder to zip file created by export chat feature ([\element-hq#7097](matrix-org/matrix-react-sdk#7097)). Fixes element-hq#19653 and element-hq#19653. Contributed by @aaronraimist. * Improve VoIP UI/UX ([\element-hq#7048](matrix-org/matrix-react-sdk#7048)). Fixes element-hq#19513 and element-hq#19513. Contributed by @SimonBrandner. * Unified room context menus ([\element-hq#7072](matrix-org/matrix-react-sdk#7072)). Fixes element-hq#19527 and element-hq#19527. * In forgot password screen, show validation errors inline in the form, instead of in modals ([\element-hq#7113](matrix-org/matrix-react-sdk#7113)). Contributed by @psrpinto. * Implement more meta-spaces ([\element-hq#7077](matrix-org/matrix-react-sdk#7077)). Fixes element-hq#18634 element-hq#17295 and element-hq#18634. * Expose power level control for m.space.child ([\#7120](matrix-org/matrix-react-sdk#7120)). * Forget member-list query when switching out of a room ([\element-hq#7093](matrix-org/matrix-react-sdk#7093)). Fixes element-hq#19432 and element-hq#19432. Contributed by @SimonBrandner. * Do pre-submit availability check on username during registration ([\#6978](matrix-org/matrix-react-sdk#6978)). Fixes element-hq#9545 and element-hq#9545. * Adjust recovery key button sizes depending on text width ([\element-hq#7134](matrix-org/matrix-react-sdk#7134)). Fixes element-hq#19511 and element-hq#19511. Contributed by @weeman1337. * Fix bulk invite button getting a negative count ([\element-hq#7122](matrix-org/matrix-react-sdk#7122)). Fixes element-hq#19466 and element-hq#19466. Contributed by @renancleyson-dev. * Fix maximised / pinned widget state being loaded correctly ([\element-hq#7146](matrix-org/matrix-react-sdk#7146)). Fixes element-hq#19768 and element-hq#19768. * Don't reload the page when user hits enter when entering ban reason ([\element-hq#7145](matrix-org/matrix-react-sdk#7145)). Fixes element-hq#19763 and element-hq#19763. * Fix timeline text when sharing room layout ([\element-hq#7140](matrix-org/matrix-react-sdk#7140)). Fixes element-hq#19622 and element-hq#19622. * Fix look of emoji verification ([\element-hq#7133](matrix-org/matrix-react-sdk#7133)). Fixes element-hq#19740 and element-hq#19740. Contributed by @SimonBrandner. * Fixes element not remembering widget hidden state per room ([\#7136](matrix-org/matrix-react-sdk#7136)). Fixes element-hq#16672, matrix-org/element-web-rageshakes#4407, element-hq#15718 element-hq#15768 and element-hq#16672. * Don't keep spinning if joining space child failed ([\element-hq#7129](matrix-org/matrix-react-sdk#7129)). Fixes matrix-org/element-web-rageshakes#6813 and matrix-org/element-web-rageshakes#6813. * Guard around SpaceStore onAccountData handler prevEvent ([\element-hq#7123](matrix-org/matrix-react-sdk#7123)). Fixes element-hq#19705 and element-hq#19705. * Fix missing spaces in threads copy ([\element-hq#7119](matrix-org/matrix-react-sdk#7119)). Fixes element-hq#19702 and element-hq#19702. * Fix hover tile border ([\element-hq#7117](matrix-org/matrix-react-sdk#7117)). Fixes element-hq#19698 and element-hq#19698. Contributed by @SimonBrandner. * Fix quote button ([\element-hq#7096](matrix-org/matrix-react-sdk#7096)). Fixes element-hq#19659 and element-hq#19659. Contributed by @SimonBrandner. * Fix space panel layout edge cases ([\#7101](matrix-org/matrix-react-sdk#7101)). Fixes element-hq#19668 and element-hq#19668. * Update powerlevel/role when the user changes in the user info panel ([\element-hq#7099](matrix-org/matrix-react-sdk#7099)). Fixes element-hq#19666 and element-hq#19666. Contributed by @SimonBrandner. * Fix avatar disappearing when setting a room topic ([\element-hq#7092](matrix-org/matrix-react-sdk#7092)). Fixes element-hq#19226 and element-hq#19226. Contributed by @SimonBrandner. * Fix possible infinite loop on widget start ([\#7071](matrix-org/matrix-react-sdk#7071)). Fixes element-hq#15494 and element-hq#15494. * Use device IDs for nameless devices in device list ([\element-hq#7081](matrix-org/matrix-react-sdk#7081)). Fixes element-hq#19608 and element-hq#19608. * Don't re-sort rooms on no-op RoomUpdateCause.PossibleTagChange ([\element-hq#7053](matrix-org/matrix-react-sdk#7053)). Contributed by @bradtgmurray.
Use case: 1:1 room, one user got locked out from the account, so they can't leave the room. There should be an ability to remove such room by the other user (who is also admin). |
If you mean that in an 1:1 room any user should be able to delete the room without any traces, I'd be opposed to that. As a user, I don't want the person I'm chatting with to be able to completely remove our communication history. |
It's kind of already possible for individual messages, including "Remove recent messages" for more bulk deletions. So not conceptually very different, just a bigger scale. |
Still waiting for an update on this. It's very annoying |
Got here from trying to leave a test room and making it disappear. Turns out you can't. That's pretty funny. |
Like matrix-org/matrix-spec-proposals#722, which has been falsely closed as a duplicate of matrix-org/matrix-spec#242, I still think it should be possible to delete rooms without any shell script or other magic. We're using Matrix for some weeks now and are VERY happy with it, but the inability to really delete rooms (and NOT just hide them) is a major annoyance. I dont want to give all my coworkers SSH access to the chat server and tell them to run a shell-script just so they can delete rooms they've created. I'm aware of the fact that when federation is enabled you cannot really guarantee that the room is actually deleted. But: We have already the option during room creation to define a room as server-local. At the very least these rooms should be deletable IMHO.
The text was updated successfully, but these errors were encountered: