Skip to content

Don't allow multiple people in DMs #320

@opusforlife2

Description

@opusforlife2

Description

I just discovered the horrifying fact that you can invite people to a 1:1 chat. I tested this just now and the 3rd user was indeed able to join after an invite. Why the hell is this allowed by the client/server/spec?

I have never seen this behaviour in any messaging service. This completely ruins the sanctity of personal chats between people, because a 1:1 chat can be converted to a group chat at any point, exposing the chat history.

This means that one can have no expectation of privacy on Matrix, and will always have to be guarded in conversations, because there is the (major or minor, depending on the persons involved) chance that the other user can invite a 3rd party to the chat and let them see the entire chat history between the first 2 users.

There is also the chance that a non-tech-savvy user mistakenly invites a 3rd user to the chat, resulting in the same problem.

Steps to reproduce

  • Start a chat with a single user
  • Create a long chat history over a long time period, including several private/personal messages.
  • Invite a 3rd user and watch them join, exposing the entire chat history in the process irrevocably.

Describe how what happens differs from what you expected.

Neither user in a 1:1 chat should be able to invite another user. This functionality should be disabled for all 1:1 chats.

If a user creates a group chat by inviting at least 2 people, then the invite function could stay enabled.

Version information

  • Platform: web

For the web app:

  • URL: app.element.io (Synapse 1.33.2)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions