13 KiB
MSC2403: Add "knock" feature
Many people are in invite-only rooms. Sometimes, someone wants to join such a room and can't, as they aren't invited. This proposal adds a feature for a user to indicate that they want to join a room.
Proposal
This proposal implements the reserved "knock" membership type for the
m.room.member state event. This state event indicates that when a user
knocks on a room, they are asking for permission to join. It contains an
optional "reason" parameter to specify the reason you want to join. Like
other membership types, the parameters "displayname" and "avatar_url" are
optional. This membership can be set from users who aren't currently in said
room. An example for the membership would look like the following:
{
"membership": "knock",
"displayname": "Alice",
"avatar_url": "mxc://example.org/avatar",
"reason": "I want to join this room as I really love foxes!"
}
After a knock in a room, a member of the room can invite the knocker.
Restrictions
There are restrictions to being able to set this membership.
Current membership
Only users without a current membership or with their current membership being "leave" can knock on a room. This means that a user that is banned, has already knocked or is currently in the room cannot knock on it.
Join Rules
This proposal introduces a new possible value for join_rule in
m.room.join_rules: "knock". The value of join_rule in m.room.join_rules
must be set to "knock" for a knock to succeed. This means that existing rooms
will need to opt into allowing knocks in their rooms. Other than allowing
knocks, "knock" is no different from the "invite" join rule.
As we're updating the join rules, and thus the auth rules, this proposal will need to be introduced as part of a new room version.
Membership changes
Once someone has sent a knock membership into the room, the membership for
that user can be transitioned to the following possible states:
invite: In this case, the knock was accepted by someone inside the room and they are inviting the knocker into the room.leave: In this case, similar to how kicks are handled, the knock has been rejected.ban: In this case, the knock was rejected and the user has been prevented from sending further knocks.
Users are not allowed to change their membership once set to knock, in
order to prevent users from being able to knock multiple times and spam a
room.
XXX: So if you knock on a room that's then abandoned that's in your /sync
forever? Clients should have a way to tell their server to hide and show
knocks.
Client-Server API
Two new endpoints are introduced in the Client-Server API (similarly to
join): POST /_matrix/client/r0/rooms/{roomId}/knock and POST /_matrix/client/r0/knock/{roomIdOrAlias}. These allow the client to state
their intent to knock on a room.
Additionally, additions to the GET /_matrix/client/r0/sync endpoint are
introduced. These allow a client to receive information about the status of
their knock attempt.
POST /_matrix/client/r0/rooms/{roomId}/knock
The path parameter (roomId) is the room on which you want to knock. It is
required. The post body accepts an optional string parameter, reason, which
is the reason you want to join the room. A request could look as follows:
POST /_matrix/client/r0/rooms/%21d41d8cd%3Amatrix.org/knock HTTP/1.1
Content-Type: application/json
{
"reason": "I want to join this room as I really love foxes!"
}
Responses:
Status code 200:
The user knocked successfully. Empty reply:
{}
Status code 400:
This request was invalid, e.g. bad JSON. Example reply:
{
"errcode": "M_UNKNOWN",
"error": "An unknown error occurred"
}
Status code 403:
The user wasn't allowed to knock (e.g. they are banned). Error reply:
{
"errcode": "M_FORBIDDEN",
"error": "The user isn't allowed to knock in this room."
}
Status code 429:
This request was rate-limited. Example reply:
{
"errcode": "M_LIMIT_EXCEEDED",
"error": "Too many requests",
"retry_after_ms": 2000
}
POST /_matrix/client/r0/knock/{roomIdOrAlias}
The path parameter (roomIdOrAlias) is either the room ID or the alias of
the room you want to knock on. Additionally several server_name parameters
can be specified via the query parameters. The post body accepts an optional
string parameter, reason, which is the reason you want to join the room. A
request could look as follows:
POST /_matrix/client/r0/knock/%23foxes%3Amatrix.org?server_name=matrix.org&server_name=elsewhere.ca HTTP/1.1
Content-Type: application/json
{
"reason": "I want to join this room as I really love foxes!"
}
Responses:
The possible responses are the same as for the POST /_matrix/client/r0/rooms/{roomId}/knock endpoint.
Extensions to GET /_matrix/client/r0/sync
In the response to
/sync
is a rooms field. This is a dictionary which currently contains keys
join, invite and leave, which each provide information to the client on
various membership states regarding the user.
It is proposed to add a fourth possible key to rooms, called knock. Its
value is a mapping from room ID to room information. The room information is
a mapping from a key knock_state to another mapping with key events being
a list of StrippedStateEvent. StrippedStateEvents are defined as state
events that only contain the sender, type, state_key and content
keys. This behaviour matches invite_events which already exists to provide
information to the client their current room invites.
These stripped state events contain information about the room, most notably
the room's name and avatar. A client will need this information to show a
nice representation of pending knocked rooms. Only m.room.name,
m.room.avatar, m.room.join_rules and m.room.membership state events
should be included here, rather than all room state event types.
Additionally, only m.room.membership events of the knocking user should be
included.
This prevents unneeded state from the room leaking out, and also speeds things up (think not sending over hundreds of membership events from big rooms).
XXX: Is m.room.canonical_alias worth allowing here for any reason?
The following is an example of knock state coming down /sync.
Request:
GET /_matrix/client/r0/sync HTTP/1.1
Content-Type: application/json
Response:
{
...
"rooms": {
"knock": {
"!abcdefghijklmo:example.com": {
"knock_state": {
events: [
{
"content": {
"join_rule": "knock"
},
"sender": "@room_admin:example.com",
"state_key": "",
"type": "m.room.join_rules"
},
{
"content": {
"name": "Some cool room"
},
"sender": "@room_admin:example.com",
"state_key": "",
"type": "m.room.name"
},
{
"content": {
"url": "mxc://example.com/xyz54321"
},
"sender": "@room_admin:example.com",
"state_key": "",
"type": "m.room.avatar"
},
{
"content": {
"avatar_url": "mxc://example.org/abc1234",
"displayname": "Knocking User",
"membership": "knock"
},
"origin_server_ts": 1598548763903,
"sender": "@knocking_user:example.org",
"state_key": "@knocking_user:example.org",
"type": "m.room.member",
"unsigned": {
"age": 5
},
"event_id": "$12345"
}
]
}
}
}
},
...
}
Once a knock has been made, a user in the room can decide whether they want to accept or deny the knock. If they accept, they will invite the knocker, which the knocker will be notified about through existing flows.
If they deny, then a leave membership event is sent in the room, and the knocking user will be notified through existing flows (matching the behaviour of when an invite is recinded).
TODO: Federation passing certain needed state events from the server that has the room to the server that's knocking. Can we filter the events sent over so that we can avoid the security issue with invites that we found earlier? (And then backport this filter to invites? :)
Server-Server API
Similarly to join and leave over federation, a ping-pong game with two new
endpoints is introduced: make_knock and send_knock. Both endpoints must
be protected via server ACLs.
GET /_matrix/federation/v1/make_knock/{roomId}/{userId}
Asks the receiving server to return information that the sending server will need to prepare a knock event.
Request format:
| Parameter | Type | Description |
|---|---|---|
| Path parameters: | ||
| roomId | string | Required. The room ID that should receive the knock. |
| userId | string | Required. The user ID the knock event will be for. |
| Query Parameters: | ||
| ver | [string] | The room versions the sending server has support for. Defaults to [1]. |
Response Format:
| Parameter | Type | Description |
|---|---|---|
| room_version | string | The version of the room where the server is trying to knock. |
| event | Event Template | An unsigned template event. May differ between room versions. |
Responses
Status code 200:
Returns a template to be used to knock on rooms. May depend on room version.
{
"room_version": "2",
"event": {
"type": "m.room.member",
"room_id": "!somewhere:example.org",
"content": {
"membership": "knock"
},
"state_key": "@someone:example.org",
"origin": "example.org",
"origin_server_ts": 1549041175876,
"sender": "@someone:example.org"
}
}
Status code 400:
This request was invalid, e.g. bad JSON. Example reply:
{
"errcode": "M_INCOMPATIBLE_ROOM_VERSION",
"error": "Your homeserver does not support the features required to join this room",
"room_version": "3"
}
PUT /_matrix/federation/v1/send_knock/{roomId}/{eventId}
Submits a signed knock event to the resident server for it to accept into the room's graph. Note that event format may differ between room versions.
Request format:
| Parameter | Type | Description |
|---|---|---|
| Path parameters: | ||
| roomId | string | Required. The room ID that should receive the knock. |
| eventId | string | Required. The event ID for the knock event. |
The JSON body is expected to be the full event.
Response Format:
| Parameter | Type | Description |
|---|---|---|
<body> |
Empty Object |
A request could look as follows:
PUT /_matrix/federation/v1/send_knock/%21abc123%3Amatrix.org/%24abc123%3Aexample.org HTTP/1.1
Content-Type: application/json
{
"sender": "@someone:example.org",
"origin": "matrix.org",
"origin_server_ts": 1234567890,
"type": "m.room.member",
"state_key": "@someone:example.org",
"content": {
"membership": "knock",
"displayname": "Alice",
"avatar_url": "mxc://example.org/avatar",
"reason": "I want to join this room as I really love foxes!"
}
}
Response:
Status code 200:
The event was successfully accepted into the graph by the receiving homeserver.
{}
Potential issues
This new feature would allow users to send events into rooms that they don't partake in. That is why this proposal adds a new join rule, in order to allow room admins to opt in to this behaviour.
Alternatives
The two endpoints for the Client-Server API seem redundant, this MSC followed
how JOIN is working currently: One "proper" endpoint (/rooms/{roomId}/join)
and one to work properly over federation (/join/{roomIdOrAlias}). They
could both be merged into one, however, as that would also affect the join
endpoint it seems out-of-scope for this MSC.
Client UX recommendations
After a knock is received in a room, it is expected to be displayed in the timeline, similar to other membership changes. Clients can optionally add a way for users of a room to review all current knocks.
Security considerations
Clients must take care when implementing this feature in order to prevent simple abuse vectors that can be accomplished by individual users. For instance, when a knock occurs, clients are advised to hide the reason until the user interacts with the client in some way (e.g. clicking on a "show reason" button). The user should reveal the reason only if they choose to.
It is recommended to not display the reason by default as else this would essentially allow outsiders to send messages into the room.
It is still theoretically possible for a server admin to create many users with different user IDs or display names, all spelling out an abusive message, and then having each of them knock in order. In this case, room admins should employ typical abuse mitigation tools, such as Server ACLs.