mirror of
https://github.com/element-hq/synapse.git
synced 2026-11-04 15:37:49 +00:00
The rooms row for a new room is inserted by the persist transaction, Rust or Python, when the create event's context is flagged, instead of by a store_room transaction of its own beforehand. The insert also claims the room ID: a row already there fails the batch with RoomIdCollisionError, and create_room runs the whole creation again under a fresh ID, perturbing the create event's timestamp for v12+ rooms as the old loop did. The room version is prefilled on the upgrade path too, as the row is no longer there to read. The chain cover check skips its rooms read when the create event is among the events, since the row may not be stored yet, and the createRoom bench advances the fake clock per room so that rooms do not collide.
6162 lines
230 KiB
Python
6162 lines
230 KiB
Python
#
|
|
# This file is licensed under the Affero General Public License (AGPL) version 3.
|
|
#
|
|
# Copyright 2019 The Matrix.org Foundation C.I.C.
|
|
# Copyright 2017 Vector Creations Ltd
|
|
# Copyright 2014-2016 OpenMarket Ltd
|
|
# Copyright (C) 2023-2024 New Vector, Ltd
|
|
#
|
|
# This program is free software: you can redistribute it and/or modify
|
|
# it under the terms of the GNU Affero General Public License as
|
|
# published by the Free Software Foundation, either version 3 of the
|
|
# License, or (at your option) any later version.
|
|
#
|
|
# See the GNU Affero General Public License for more details:
|
|
# <https://www.gnu.org/licenses/agpl-3.0.html>.
|
|
#
|
|
# Originally licensed under the Apache License, Version 2.0:
|
|
# <http://www.apache.org/licenses/LICENSE-2.0>.
|
|
#
|
|
# [This file includes modifications made by New Vector Limited]
|
|
#
|
|
#
|
|
|
|
"""Tests REST events for /rooms paths."""
|
|
|
|
import json
|
|
import time
|
|
from http import HTTPStatus
|
|
from typing import Any, Iterable, Literal
|
|
from unittest.mock import AsyncMock, Mock, call, create_autospec, patch
|
|
from urllib import parse as urlparse
|
|
|
|
from parameterized import param, parameterized
|
|
|
|
from twisted.internet import defer
|
|
from twisted.internet.testing import MemoryReactor
|
|
from twisted.web.client import Agent
|
|
|
|
import synapse.rest.admin
|
|
from synapse.api.constants import (
|
|
EduTypes,
|
|
EventContentFields,
|
|
EventTypes,
|
|
Membership,
|
|
PublicRoomsFilterFields,
|
|
RoomTypes,
|
|
)
|
|
from synapse.api.errors import Codes, HttpResponseException
|
|
from synapse.appservice import ApplicationService
|
|
from synapse.events import EventBase, make_event_from_dict
|
|
from synapse.events.snapshot import EventContext
|
|
from synapse.rest import admin
|
|
from synapse.rest.client import (
|
|
account,
|
|
directory,
|
|
knock,
|
|
login,
|
|
profile,
|
|
register,
|
|
room,
|
|
sync,
|
|
)
|
|
from synapse.server import HomeServer
|
|
from synapse.types import JsonDict, JsonMapping, RoomAlias, UserID, create_requester
|
|
from synapse.util.clock import Clock
|
|
from synapse.util.duration import Duration
|
|
from synapse.util.stringutils import random_string
|
|
|
|
from tests import unittest
|
|
from tests.http.server._base import make_request_with_cancellation_test
|
|
from tests.server import FakeChannel
|
|
from tests.storage.test_stream import PaginationTestCase
|
|
from tests.test_utils import FakeResponse
|
|
from tests.test_utils.event_injection import (
|
|
create_event,
|
|
inject_event,
|
|
)
|
|
from tests.unittest import override_config
|
|
from tests.utils import default_config
|
|
|
|
PATH_PREFIX = b"/_matrix/client/api/v1"
|
|
|
|
|
|
class RoomBase(unittest.HomeserverTestCase):
|
|
rmcreator_id: str | None = None
|
|
|
|
servlets = [room.register_servlets, room.register_deprecated_servlets]
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
self.hs = self.setup_test_homeserver(
|
|
"red",
|
|
)
|
|
|
|
self.hs.get_federation_handler = Mock() # type: ignore[method-assign]
|
|
self.hs.get_federation_handler.return_value.maybe_backfill = AsyncMock(
|
|
return_value=None
|
|
)
|
|
|
|
async def _insert_client_ip(*args: Any, **kwargs: Any) -> None:
|
|
return None
|
|
|
|
self.hs.get_datastores().main.insert_client_ip = _insert_client_ip # type: ignore[method-assign]
|
|
|
|
return self.hs
|
|
|
|
|
|
class RoomPermissionsTestCase(RoomBase):
|
|
"""Tests room permissions."""
|
|
|
|
user_id = "@sid1:red"
|
|
rmcreator_id = "@notme:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.store_controllers = hs.get_storage_controllers()
|
|
self.helper.auth_user_id = self.rmcreator_id
|
|
# create some rooms under the name rmcreator_id
|
|
self.uncreated_rmid = "!aa:test"
|
|
self.created_rmid = self.helper.create_room_as(
|
|
self.rmcreator_id, is_public=False
|
|
)
|
|
self.created_public_rmid = self.helper.create_room_as(
|
|
self.rmcreator_id, is_public=True
|
|
)
|
|
|
|
# send a message in one of the rooms
|
|
self.created_rmid_msg_path = (
|
|
"rooms/%s/send/m.room.message/a1" % (self.created_rmid)
|
|
).encode("ascii")
|
|
channel = self.make_request(
|
|
"PUT", self.created_rmid_msg_path, b'{"msgtype":"m.text","body":"test msg"}'
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
# set topic for public room
|
|
channel = self.make_request(
|
|
"PUT",
|
|
("rooms/%s/state/m.room.topic" % self.created_public_rmid).encode("ascii"),
|
|
b'{"topic":"Public Room Topic"}',
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
# auth as user_id now
|
|
self.helper.auth_user_id = self.user_id
|
|
|
|
def test_can_do_action(self) -> None:
|
|
msg_content = b'{"msgtype":"m.text","body":"hello"}'
|
|
|
|
seq = iter(range(100))
|
|
|
|
def send_msg_path() -> str:
|
|
return "/rooms/%s/send/m.room.message/mid%s" % (
|
|
self.created_rmid,
|
|
str(next(seq)),
|
|
)
|
|
|
|
# send message in uncreated room, expect 403
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"/rooms/%s/send/m.room.message/mid2" % (self.uncreated_rmid,),
|
|
msg_content,
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# send message in created room not joined (no state), expect 403
|
|
channel = self.make_request("PUT", send_msg_path(), msg_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# send message in created room and invited, expect 403
|
|
self.helper.invite(
|
|
room=self.created_rmid, src=self.rmcreator_id, targ=self.user_id
|
|
)
|
|
channel = self.make_request("PUT", send_msg_path(), msg_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# send message in created room and joined, expect 200
|
|
self.helper.join(room=self.created_rmid, user=self.user_id)
|
|
channel = self.make_request("PUT", send_msg_path(), msg_content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# send message in created room and left, expect 403
|
|
self.helper.leave(room=self.created_rmid, user=self.user_id)
|
|
channel = self.make_request("PUT", send_msg_path(), msg_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_topic_perms(self) -> None:
|
|
topic_content = b'{"topic":"My Topic Name"}'
|
|
topic_path = "/rooms/%s/state/m.room.topic" % self.created_rmid
|
|
|
|
# set/get topic in uncreated room, expect 403
|
|
channel = self.make_request(
|
|
"PUT", "/rooms/%s/state/m.room.topic" % self.uncreated_rmid, topic_content
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/state/m.room.topic" % self.uncreated_rmid
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# set/get topic in created PRIVATE room not joined, expect 403
|
|
channel = self.make_request("PUT", topic_path, topic_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
channel = self.make_request("GET", topic_path)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# set topic in created PRIVATE room and invited, expect 403
|
|
self.helper.invite(
|
|
room=self.created_rmid, src=self.rmcreator_id, targ=self.user_id
|
|
)
|
|
channel = self.make_request("PUT", topic_path, topic_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# get topic in created PRIVATE room and invited, expect 403
|
|
channel = self.make_request("GET", topic_path)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# set/get topic in created PRIVATE room and joined, expect 200
|
|
self.helper.join(room=self.created_rmid, user=self.user_id)
|
|
|
|
# Only room ops can set topic by default
|
|
self.helper.auth_user_id = self.rmcreator_id
|
|
channel = self.make_request("PUT", topic_path, topic_content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.helper.auth_user_id = self.user_id
|
|
|
|
channel = self.make_request("GET", topic_path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assert_dict(json.loads(topic_content.decode("utf8")), channel.json_body)
|
|
|
|
# set/get topic in created PRIVATE room and left, expect 403
|
|
self.helper.leave(room=self.created_rmid, user=self.user_id)
|
|
channel = self.make_request("PUT", topic_path, topic_content)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
channel = self.make_request("GET", topic_path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# get topic in PUBLIC room, not joined, expect 403
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/state/m.room.topic" % self.created_public_rmid
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
# set topic in PUBLIC room, not joined, expect 403
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"/rooms/%s/state/m.room.topic" % self.created_public_rmid,
|
|
topic_content,
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def _test_get_membership(
|
|
self, room: str, members: Iterable = frozenset(), expect_code: int = 200
|
|
) -> None:
|
|
for member in members:
|
|
path = "/rooms/%s/state/m.room.member/%s" % (room, member)
|
|
channel = self.make_request("GET", path)
|
|
self.assertEqual(expect_code, channel.code)
|
|
|
|
def test_membership_basic_room_perms(self) -> None:
|
|
# === room does not exist ===
|
|
room = self.uncreated_rmid
|
|
# get membership of self, get membership of other, uncreated room
|
|
# expect all 403s
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=403
|
|
)
|
|
|
|
# trying to invite people to this room should 403
|
|
self.helper.invite(
|
|
room=room, src=self.user_id, targ=self.rmcreator_id, expect_code=403
|
|
)
|
|
|
|
# set [invite/join/left] of self, set [invite/join/left] of other,
|
|
# expect all 404s because room doesn't exist on any server
|
|
for usr in [self.user_id, self.rmcreator_id]:
|
|
self.helper.join(room=room, user=usr, expect_code=404)
|
|
self.helper.leave(room=room, user=usr, expect_code=404)
|
|
|
|
def test_membership_private_room_perms(self) -> None:
|
|
room = self.created_rmid
|
|
# get membership of self, get membership of other, private room + invite
|
|
# expect all 403s
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=403
|
|
)
|
|
|
|
# get membership of self, get membership of other, private room + joined
|
|
# expect all 200s
|
|
self.helper.join(room=room, user=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=200
|
|
)
|
|
|
|
# get membership of self, get membership of other, private room + left
|
|
# expect all 200s
|
|
self.helper.leave(room=room, user=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=200
|
|
)
|
|
|
|
def test_membership_public_room_perms(self) -> None:
|
|
room = self.created_public_rmid
|
|
# get membership of self, get membership of other, public room + invite
|
|
# expect 403
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=403
|
|
)
|
|
|
|
# get membership of self, get membership of other, public room + joined
|
|
# expect all 200s
|
|
self.helper.join(room=room, user=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=200
|
|
)
|
|
|
|
# get membership of self, get membership of other, public room + left
|
|
# expect 200.
|
|
self.helper.leave(room=room, user=self.user_id)
|
|
self._test_get_membership(
|
|
members=[self.user_id, self.rmcreator_id], room=room, expect_code=200
|
|
)
|
|
|
|
def test_invited_permissions(self) -> None:
|
|
room = self.created_rmid
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
|
|
# set [invite/join/left] of other user, expect 403s
|
|
self.helper.invite(
|
|
room=room, src=self.user_id, targ=self.rmcreator_id, expect_code=403
|
|
)
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=self.rmcreator_id,
|
|
membership=Membership.JOIN,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=self.rmcreator_id,
|
|
membership=Membership.LEAVE,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
def test_joined_permissions(self) -> None:
|
|
room = self.created_rmid
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
self.helper.join(room=room, user=self.user_id)
|
|
|
|
# set invited of self, expect 403
|
|
self.helper.invite(
|
|
room=room, src=self.user_id, targ=self.user_id, expect_code=403
|
|
)
|
|
|
|
# set joined of self, expect 200 (NOOP)
|
|
self.helper.join(room=room, user=self.user_id)
|
|
|
|
other = "@burgundy:red"
|
|
# set invited of other, expect 200
|
|
self.helper.invite(room=room, src=self.user_id, targ=other, expect_code=200)
|
|
|
|
# set joined of other, expect 403
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=other,
|
|
membership=Membership.JOIN,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
# set left of other, expect 403
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=other,
|
|
membership=Membership.LEAVE,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
# set left of self, expect 200
|
|
self.helper.leave(room=room, user=self.user_id)
|
|
|
|
def test_leave_permissions(self) -> None:
|
|
room = self.created_rmid
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
self.helper.join(room=room, user=self.user_id)
|
|
self.helper.leave(room=room, user=self.user_id)
|
|
|
|
# set [invite/join/left] of self, set [invite/join/left] of other,
|
|
# expect all 403s
|
|
for usr in [self.user_id, self.rmcreator_id]:
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=usr,
|
|
membership=Membership.INVITE,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=usr,
|
|
membership=Membership.JOIN,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
# It is always valid to LEAVE if you've already left (currently.)
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=self.rmcreator_id,
|
|
membership=Membership.LEAVE,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
)
|
|
|
|
# tests the "from banned" line from the table in https://spec.matrix.org/unstable/client-server-api/#mroommember
|
|
def test_member_event_from_ban(self) -> None:
|
|
room = self.created_rmid
|
|
self.helper.invite(room=room, src=self.rmcreator_id, targ=self.user_id)
|
|
self.helper.join(room=room, user=self.user_id)
|
|
|
|
other = "@burgundy:red"
|
|
|
|
# User cannot ban other since they do not have required power level
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=other,
|
|
membership=Membership.BAN,
|
|
expect_code=HTTPStatus.FORBIDDEN, # expect failure
|
|
expect_errcode=Codes.FORBIDDEN,
|
|
)
|
|
|
|
# Admin bans other
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.rmcreator_id,
|
|
targ=other,
|
|
membership=Membership.BAN,
|
|
expect_code=HTTPStatus.OK,
|
|
)
|
|
|
|
# from ban to invite: Must never happen.
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.rmcreator_id,
|
|
targ=other,
|
|
membership=Membership.INVITE,
|
|
expect_code=HTTPStatus.FORBIDDEN, # expect failure
|
|
expect_errcode=Codes.BAD_STATE,
|
|
)
|
|
|
|
# from ban to join: Must never happen.
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=other,
|
|
targ=other,
|
|
membership=Membership.JOIN,
|
|
expect_code=HTTPStatus.FORBIDDEN, # expect failure
|
|
expect_errcode=Codes.BAD_STATE,
|
|
)
|
|
|
|
# from ban to ban: No change.
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.rmcreator_id,
|
|
targ=other,
|
|
membership=Membership.BAN,
|
|
expect_code=HTTPStatus.OK,
|
|
)
|
|
|
|
# from ban to knock: Must never happen.
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.rmcreator_id,
|
|
targ=other,
|
|
membership=Membership.KNOCK,
|
|
expect_code=HTTPStatus.FORBIDDEN, # expect failure
|
|
expect_errcode=Codes.BAD_STATE,
|
|
)
|
|
|
|
# User cannot unban other since they do not have required power level
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.user_id,
|
|
targ=other,
|
|
membership=Membership.LEAVE,
|
|
expect_code=HTTPStatus.FORBIDDEN, # expect failure
|
|
expect_errcode=Codes.FORBIDDEN,
|
|
)
|
|
|
|
# from ban to leave: User was unbanned.
|
|
self.helper.change_membership(
|
|
room=room,
|
|
src=self.rmcreator_id,
|
|
targ=other,
|
|
membership=Membership.LEAVE,
|
|
expect_code=HTTPStatus.OK,
|
|
)
|
|
|
|
def test_default_call_invite_power_level(self) -> None:
|
|
pl_event = self.get_success(
|
|
self.store_controllers.state.get_current_state_event(
|
|
self.created_public_rmid, EventTypes.PowerLevels, ""
|
|
)
|
|
)
|
|
assert pl_event is not None
|
|
self.assertEqual(50, pl_event.content.get("events", {}).get("m.call.invite"))
|
|
self.assertEqual(50, pl_event.content.get("events", {}).get("m.room.name"))
|
|
|
|
private_pl_event = self.get_success(
|
|
self.store_controllers.state.get_current_state_event(
|
|
self.created_rmid, EventTypes.PowerLevels, ""
|
|
)
|
|
)
|
|
assert private_pl_event is not None
|
|
self.assertEqual(
|
|
None, private_pl_event.content.get("events", {}).get("m.call.invite")
|
|
)
|
|
|
|
|
|
class RoomStateTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/state."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def test_get_state_cancellation(self) -> None:
|
|
"""Test cancellation of a `/rooms/$room_id/state` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel = make_request_with_cancellation_test(
|
|
"test_state_cancellation",
|
|
self.reactor,
|
|
self.site,
|
|
"GET",
|
|
"/rooms/%s/state" % room_id,
|
|
)
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertCountEqual(
|
|
[state_event["type"] for state_event in channel.json_list],
|
|
{
|
|
"m.room.create",
|
|
"m.room.power_levels",
|
|
"m.room.join_rules",
|
|
"m.room.member",
|
|
"m.room.history_visibility",
|
|
},
|
|
)
|
|
|
|
def test_get_state_event_cancellation(self) -> None:
|
|
"""Test cancellation of a `/rooms/$room_id/state/$event_type` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel = make_request_with_cancellation_test(
|
|
"test_state_cancellation",
|
|
self.reactor,
|
|
self.site,
|
|
"GET",
|
|
"/rooms/%s/state/m.room.member/%s" % (room_id, self.user_id),
|
|
)
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(channel.json_body, {"membership": "join"})
|
|
|
|
def test_get_state_format_content(self) -> None:
|
|
"""Test response of a `/rooms/$room_id/state/$event_type?format=content` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel1 = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/state/m.room.member/%s?format=content"
|
|
% (
|
|
room_id,
|
|
self.user_id,
|
|
),
|
|
)
|
|
self.assertEqual(channel1.code, HTTPStatus.OK, channel1.json_body)
|
|
self.assertEqual(channel1.json_body, {"membership": "join"})
|
|
channel2 = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/state/m.room.member/%s"
|
|
% (
|
|
room_id,
|
|
self.user_id,
|
|
),
|
|
)
|
|
self.assertEqual(channel2.code, HTTPStatus.OK, channel2.json_body)
|
|
# "content" is the default format.
|
|
self.assertEqual(channel1.json_body, channel2.json_body)
|
|
|
|
def test_get_state_format_event(self) -> None:
|
|
"""Test response of a `/rooms/$room_id/state/$event_type?format=event` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/state/m.room.member/%s?format=event"
|
|
% (
|
|
room_id,
|
|
self.user_id,
|
|
),
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
self.assertEqual(channel.json_body["content"], {"membership": "join"})
|
|
self.assertEqual(channel.json_body["room_id"], room_id)
|
|
self.assertRegex(channel.json_body["event_id"], r"\$.+")
|
|
self.assertEqual(channel.json_body["type"], "m.room.member")
|
|
self.assertEqual(channel.json_body["sender"], self.user_id)
|
|
self.assertEqual(channel.json_body["state_key"], self.user_id)
|
|
self.assertTrue(type(channel.json_body["origin_server_ts"]) is int)
|
|
|
|
|
|
class RoomsMemberListTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/members/list REST events."""
|
|
|
|
servlets = RoomBase.servlets + [sync.register_servlets]
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def test_get_member_list(self) -> None:
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel = self.make_request("GET", "/rooms/%s/members" % room_id)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_no_room(self) -> None:
|
|
channel = self.make_request("GET", "/rooms/roomdoesnotexist/members")
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_no_permission(self) -> None:
|
|
room_id = self.helper.create_room_as("@some_other_guy:red")
|
|
channel = self.make_request("GET", "/rooms/%s/members" % room_id)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_no_permission_with_at_token(self) -> None:
|
|
"""
|
|
Tests that a stranger to the room cannot get the member list
|
|
(in the case that they use an at token).
|
|
"""
|
|
room_id = self.helper.create_room_as("@someone.else:red")
|
|
|
|
# first sync to get an at token
|
|
channel = self.make_request("GET", "/sync")
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
sync_token = channel.json_body["next_batch"]
|
|
|
|
# check that permission is denied for @sid1:red to get the
|
|
# memberships of @someone.else:red's room.
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{room_id}/members?at={sync_token}",
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_no_permission_former_member(self) -> None:
|
|
"""
|
|
Tests that a former member of the room can not get the member list.
|
|
"""
|
|
# create a room, invite the user and the user joins
|
|
room_id = self.helper.create_room_as("@alice:red")
|
|
self.helper.invite(room_id, "@alice:red", self.user_id)
|
|
self.helper.join(room_id, self.user_id)
|
|
|
|
# check that the user can see the member list to start with
|
|
channel = self.make_request("GET", "/rooms/%s/members" % room_id)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# ban the user
|
|
self.helper.change_membership(room_id, "@alice:red", self.user_id, "ban")
|
|
|
|
# check the user can no longer see the member list
|
|
channel = self.make_request("GET", "/rooms/%s/members" % room_id)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_no_permission_former_member_with_at_token(self) -> None:
|
|
"""
|
|
Tests that a former member of the room can not get the member list
|
|
(in the case that they use an at token).
|
|
"""
|
|
# create a room, invite the user and the user joins
|
|
room_id = self.helper.create_room_as("@alice:red")
|
|
self.helper.invite(room_id, "@alice:red", self.user_id)
|
|
self.helper.join(room_id, self.user_id)
|
|
|
|
# sync to get an at token
|
|
channel = self.make_request("GET", "/sync")
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
sync_token = channel.json_body["next_batch"]
|
|
|
|
# check that the user can see the member list to start with
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/members?at=%s" % (room_id, sync_token)
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# ban the user (Note: the user is actually allowed to see this event and
|
|
# state so that they know they're banned!)
|
|
self.helper.change_membership(room_id, "@alice:red", self.user_id, "ban")
|
|
|
|
# invite a third user and let them join
|
|
self.helper.invite(room_id, "@alice:red", "@bob:red")
|
|
self.helper.join(room_id, "@bob:red")
|
|
|
|
# now, with the original user, sync again to get a new at token
|
|
channel = self.make_request("GET", "/sync")
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
sync_token = channel.json_body["next_batch"]
|
|
|
|
# check the user can no longer see the updated member list
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/members?at=%s" % (room_id, sync_token)
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_mixed_memberships(self) -> None:
|
|
room_creator = "@some_other_guy:red"
|
|
room_id = self.helper.create_room_as(room_creator)
|
|
room_path = "/rooms/%s/members" % room_id
|
|
self.helper.invite(room=room_id, src=room_creator, targ=self.user_id)
|
|
# can't see list if you're just invited.
|
|
channel = self.make_request("GET", room_path)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
# can see list now joined
|
|
channel = self.make_request("GET", room_path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
self.helper.leave(room=room_id, user=self.user_id)
|
|
# can see old list once left
|
|
channel = self.make_request("GET", room_path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
def test_get_member_list_cancellation(self) -> None:
|
|
"""Test cancellation of a `/rooms/$room_id/members` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
channel = make_request_with_cancellation_test(
|
|
"test_get_member_list_cancellation",
|
|
self.reactor,
|
|
self.site,
|
|
"GET",
|
|
"/rooms/%s/members" % room_id,
|
|
)
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(len(channel.json_body["chunk"]), 1)
|
|
self.assertLessEqual(
|
|
{
|
|
"content": {"membership": "join"},
|
|
"room_id": room_id,
|
|
"sender": self.user_id,
|
|
"state_key": self.user_id,
|
|
"type": "m.room.member",
|
|
"user_id": self.user_id,
|
|
}.items(),
|
|
channel.json_body["chunk"][0].items(),
|
|
)
|
|
|
|
def test_get_member_list_with_at_token_cancellation(self) -> None:
|
|
"""Test cancellation of a `/rooms/$room_id/members?at=<sync token>` request."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
# first sync to get an at token
|
|
channel = self.make_request("GET", "/sync")
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
sync_token = channel.json_body["next_batch"]
|
|
|
|
channel = make_request_with_cancellation_test(
|
|
"test_get_member_list_with_at_token_cancellation",
|
|
self.reactor,
|
|
self.site,
|
|
"GET",
|
|
"/rooms/%s/members?at=%s" % (room_id, sync_token),
|
|
)
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(len(channel.json_body["chunk"]), 1)
|
|
self.assertLessEqual(
|
|
{
|
|
"content": {"membership": "join"},
|
|
"room_id": room_id,
|
|
"sender": self.user_id,
|
|
"state_key": self.user_id,
|
|
"type": "m.room.member",
|
|
"user_id": self.user_id,
|
|
}.items(),
|
|
channel.json_body["chunk"][0].items(),
|
|
)
|
|
|
|
|
|
class RoomsCreateTestCase(RoomBase):
|
|
"""Tests /rooms and /rooms/$room_id REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def _expected_create_room_txn_count(self) -> int:
|
|
"""The Python transactions a plain room creation runs. The state groups
|
|
are written in Rust on the native driver, outside them."""
|
|
count = 7
|
|
if self.hs.get_datastores().state._rust_state_group_writer is not None:
|
|
count -= 1
|
|
return count
|
|
|
|
def test_post_room_no_keys(self) -> None:
|
|
# POST with no config keys, expect new room id
|
|
channel = self.make_request("POST", "/createRoom", "{}")
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
assert channel.resource_usage is not None
|
|
self.assertEqual(
|
|
self._expected_create_room_txn_count(), channel.resource_usage.db_txn_count
|
|
)
|
|
|
|
def test_post_room_initial_state(self) -> None:
|
|
# POST with initial_state config key, expect new room id
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
b'{"initial_state":[{"type": "m.bridge", "content": {}}]}',
|
|
)
|
|
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
assert channel.resource_usage is not None
|
|
self.assertEqual(
|
|
self._expected_create_room_txn_count(), channel.resource_usage.db_txn_count
|
|
)
|
|
|
|
def test_post_room_topic(self) -> None:
|
|
# POST with topic key, expect new room id
|
|
channel = self.make_request("POST", "/createRoom", b'{"topic":"shenanigans"}')
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
room_id = channel.json_body["room_id"]
|
|
|
|
# GET topic event, expect content from topic key
|
|
channel = self.make_request("GET", "/rooms/%s/state/m.room.topic" % (room_id,))
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertEqual(
|
|
{"topic": "shenanigans", "m.topic": {"m.text": [{"body": "shenanigans"}]}},
|
|
channel.json_body,
|
|
)
|
|
|
|
def test_post_room_topic_initial_state(self) -> None:
|
|
# POST with m.room.topic in initial state, expect new room id
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
b'{"initial_state":[{"type": "m.room.topic", "content": {"topic": "foobar"}}]}',
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
room_id = channel.json_body["room_id"]
|
|
|
|
# GET topic event, expect content from initial state
|
|
channel = self.make_request("GET", "/rooms/%s/state/m.room.topic" % (room_id,))
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertEqual(
|
|
{"topic": "foobar"},
|
|
channel.json_body,
|
|
)
|
|
|
|
def test_post_room_topic_overriding_initial_state(self) -> None:
|
|
# POST with m.room.topic in initial state and topic key, expect new room id
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
b'{"initial_state":[{"type": "m.room.topic", "content": {"topic": "foobar"}}], "topic":"shenanigans"}',
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
room_id = channel.json_body["room_id"]
|
|
|
|
# GET topic event, expect content from topic key
|
|
channel = self.make_request("GET", "/rooms/%s/state/m.room.topic" % (room_id,))
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertEqual(
|
|
{"topic": "shenanigans", "m.topic": {"m.text": [{"body": "shenanigans"}]}},
|
|
channel.json_body,
|
|
)
|
|
|
|
def test_post_room_visibility_key(self) -> None:
|
|
# POST with visibility config key, expect new room id
|
|
channel = self.make_request("POST", "/createRoom", b'{"visibility":"private"}')
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
|
|
def test_post_room_custom_key(self) -> None:
|
|
# POST with custom config keys, expect new room id
|
|
channel = self.make_request("POST", "/createRoom", b'{"custom":"stuff"}')
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
|
|
def test_post_room_known_and_unknown_keys(self) -> None:
|
|
# POST with custom + known config keys, expect new room id
|
|
channel = self.make_request(
|
|
"POST", "/createRoom", b'{"visibility":"private","custom":"things"}'
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("room_id" in channel.json_body)
|
|
|
|
def test_post_room_invalid_content(self) -> None:
|
|
# POST with invalid content / paths, expect 400
|
|
channel = self.make_request("POST", "/createRoom", b'{"visibili')
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code)
|
|
|
|
channel = self.make_request("POST", "/createRoom", b'["hello"]')
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code)
|
|
|
|
def test_post_room_invitees_invalid_mxid(self) -> None:
|
|
# POST with invalid invitee, see https://github.com/matrix-org/synapse/issues/4088
|
|
# Note the trailing space in the MXID here!
|
|
channel = self.make_request(
|
|
"POST", "/createRoom", b'{"invite":["@alice:example.com "]}'
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code)
|
|
|
|
@unittest.override_config({"rc_invites": {"per_room": {"burst_count": 3}}})
|
|
def test_post_room_invitees_ratelimit(self) -> None:
|
|
"""Test that invites sent when creating a room are ratelimited by a RateLimiter,
|
|
which ratelimits them correctly, including by not limiting when the requester is
|
|
exempt from ratelimiting.
|
|
"""
|
|
|
|
# Build the request's content. We use local MXIDs because invites over federation
|
|
# are more difficult to mock.
|
|
content = {
|
|
"invite": [
|
|
"@alice1:red",
|
|
"@alice2:red",
|
|
"@alice3:red",
|
|
"@alice4:red",
|
|
]
|
|
}
|
|
|
|
# Test that the invites are correctly ratelimited.
|
|
channel = self.make_request("POST", "/createRoom", content)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code)
|
|
self.assertEqual(
|
|
"Cannot invite so many users at once",
|
|
channel.json_body["error"],
|
|
)
|
|
|
|
# Add the current user to the ratelimit overrides, allowing them no ratelimiting.
|
|
self.get_success(
|
|
self.hs.get_datastores().main.set_ratelimit_for_user(self.user_id, 0, 0)
|
|
)
|
|
|
|
# Test that the invites aren't ratelimited anymore.
|
|
channel = self.make_request("POST", "/createRoom", content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
|
|
def test_spam_checker_may_join_room_deprecated(self) -> None:
|
|
"""Tests that the user_may_join_room spam checker callback is correctly bypassed
|
|
when creating a new room.
|
|
|
|
In this test, we use the deprecated API in which callbacks return a bool.
|
|
"""
|
|
|
|
async def user_may_join_room(
|
|
mxid: str,
|
|
room_id: str,
|
|
is_invite: bool,
|
|
) -> bool:
|
|
return False
|
|
|
|
join_mock = Mock(side_effect=user_may_join_room)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_join_room_callbacks.append(
|
|
join_mock
|
|
)
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
{},
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
self.assertEqual(join_mock.call_count, 0)
|
|
|
|
def test_spam_checker_may_join_room(self) -> None:
|
|
"""Tests that the user_may_join_room spam checker callback is correctly bypassed
|
|
when creating a new room.
|
|
|
|
In this test, we use the more recent API in which callbacks return a `Codes | Literal["NOT_SPAM"]`.
|
|
"""
|
|
|
|
async def user_may_join_room_codes(
|
|
mxid: str,
|
|
room_id: str,
|
|
is_invite: bool,
|
|
) -> Codes:
|
|
return Codes.CONSENT_NOT_GIVEN
|
|
|
|
join_mock = Mock(side_effect=user_may_join_room_codes)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_join_room_callbacks.append(
|
|
join_mock
|
|
)
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
{},
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
self.assertEqual(join_mock.call_count, 0)
|
|
|
|
# Now change the return value of the callback to deny any join. Since we're
|
|
# creating the room, despite the return value, we should be able to join.
|
|
async def user_may_join_room_tuple(
|
|
mxid: str,
|
|
room_id: str,
|
|
is_invite: bool,
|
|
) -> tuple[Codes, dict]:
|
|
return Codes.INCOMPATIBLE_ROOM_VERSION, {}
|
|
|
|
join_mock.side_effect = user_may_join_room_tuple
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
{},
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
self.assertEqual(join_mock.call_count, 0)
|
|
|
|
def _create_basic_room(self) -> tuple[int, object]:
|
|
"""
|
|
Tries to create a basic room and returns the response code.
|
|
"""
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
{},
|
|
)
|
|
return channel.code, channel.json_body
|
|
|
|
@override_config(
|
|
{
|
|
"rc_message": {"per_second": 0.2, "burst_count": 10},
|
|
}
|
|
)
|
|
def test_room_creation_ratelimiting(self) -> None:
|
|
"""
|
|
Regression test for https://github.com/matrix-org/synapse/issues/14312,
|
|
where ratelimiting was made too strict.
|
|
Clients should be able to create 10 rooms in a row
|
|
without hitting rate limits, using default rate limit config.
|
|
(We override rate limiting config back to its default value.)
|
|
|
|
To ensure we don't make ratelimiting too generous accidentally,
|
|
also check that we can't create an 11th room.
|
|
"""
|
|
|
|
for _ in range(10):
|
|
code, json_body = self._create_basic_room()
|
|
self.assertEqual(code, HTTPStatus.OK, json_body)
|
|
|
|
# The 6th room hits the rate limit.
|
|
code, json_body = self._create_basic_room()
|
|
self.assertEqual(code, HTTPStatus.TOO_MANY_REQUESTS, json_body)
|
|
|
|
|
|
class RoomTopicTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/topic REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
# create the room
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
self.path = "/rooms/%s/state/m.room.topic" % (self.room_id,)
|
|
|
|
def test_invalid_puts(self) -> None:
|
|
# missing keys or invalid json
|
|
channel = self.make_request("PUT", self.path, "{}")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", self.path, '{"_name":"bo"}')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", self.path, '{"nao')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request(
|
|
"PUT", self.path, '[{"_name":"bo"},{"_name":"jill"}]'
|
|
)
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", self.path, "text only")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", self.path, "")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
# valid key, wrong type
|
|
content = '{"topic":["Topic name"]}'
|
|
channel = self.make_request("PUT", self.path, content)
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
def test_rooms_topic(self) -> None:
|
|
# nothing should be there
|
|
channel = self.make_request("GET", self.path)
|
|
self.assertEqual(HTTPStatus.NOT_FOUND, channel.code, msg=channel.result["body"])
|
|
|
|
# valid put
|
|
content = '{"topic":"Topic name"}'
|
|
channel = self.make_request("PUT", self.path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# valid get
|
|
channel = self.make_request("GET", self.path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assert_dict(json.loads(content), channel.json_body)
|
|
|
|
def test_rooms_topic_with_extra_keys(self) -> None:
|
|
# valid put with extra keys
|
|
content = '{"topic":"Seasons","subtopic":"Summer"}'
|
|
channel = self.make_request("PUT", self.path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# valid get
|
|
channel = self.make_request("GET", self.path)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assert_dict(json.loads(content), channel.json_body)
|
|
|
|
|
|
class RoomMemberStateTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/members/$user_id/state REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
def test_invalid_puts(self) -> None:
|
|
path = "/rooms/%s/state/m.room.member/%s" % (self.room_id, self.user_id)
|
|
# missing keys or invalid json
|
|
channel = self.make_request("PUT", path, "{}")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, '{"_name":"bo"}')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, '{"nao')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b'[{"_name":"bo"},{"_name":"jill"}]')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, "text only")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, "")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
# valid keys, wrong types
|
|
content = '{"membership":["%s","%s","%s"]}' % (
|
|
Membership.INVITE,
|
|
Membership.JOIN,
|
|
Membership.LEAVE,
|
|
)
|
|
channel = self.make_request("PUT", path, content.encode("ascii"))
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
def test_rooms_members_self(self) -> None:
|
|
path = "/rooms/%s/state/m.room.member/%s" % (
|
|
urlparse.quote(self.room_id),
|
|
self.user_id,
|
|
)
|
|
|
|
# valid join message (NOOP since we made the room)
|
|
content = '{"membership":"%s"}' % Membership.JOIN
|
|
channel = self.make_request("PUT", path, content.encode("ascii"))
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
channel = self.make_request("GET", path, content=b"")
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
expected_response = {"membership": Membership.JOIN}
|
|
self.assertEqual(expected_response, channel.json_body)
|
|
|
|
def test_rooms_members_other(self) -> None:
|
|
self.other_id = "@zzsid1:red"
|
|
path = "/rooms/%s/state/m.room.member/%s" % (
|
|
urlparse.quote(self.room_id),
|
|
self.other_id,
|
|
)
|
|
|
|
# valid invite message
|
|
content = '{"membership":"%s"}' % Membership.INVITE
|
|
channel = self.make_request("PUT", path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
channel = self.make_request("GET", path, content=b"")
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(json.loads(content), channel.json_body)
|
|
|
|
def test_rooms_members_other_custom_keys(self) -> None:
|
|
self.other_id = "@zzsid1:red"
|
|
path = "/rooms/%s/state/m.room.member/%s" % (
|
|
urlparse.quote(self.room_id),
|
|
self.other_id,
|
|
)
|
|
|
|
# valid invite message with custom key
|
|
content = '{"membership":"%s","invite_text":"%s"}' % (
|
|
Membership.INVITE,
|
|
"Join us!",
|
|
)
|
|
channel = self.make_request("PUT", path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
channel = self.make_request("GET", path, content=b"")
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(json.loads(content), channel.json_body)
|
|
|
|
|
|
class RoomInviteRatelimitTestCase(RoomBase):
|
|
user_id = "@sid1:red"
|
|
|
|
servlets = [
|
|
admin.register_servlets,
|
|
profile.register_servlets,
|
|
room.register_servlets,
|
|
]
|
|
|
|
@unittest.override_config(
|
|
{"rc_invites": {"per_room": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_invites_by_rooms_ratelimit(self) -> None:
|
|
"""Tests that invites in a room are actually rate-limited."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
for i in range(3):
|
|
self.helper.invite(room_id, self.user_id, "@user-%s:red" % (i,))
|
|
|
|
self.helper.invite(room_id, self.user_id, "@user-4:red", expect_code=429)
|
|
|
|
@unittest.override_config(
|
|
{"rc_invites": {"per_user": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_invites_by_users_ratelimit(self) -> None:
|
|
"""Tests that invites to a specific user are actually rate-limited."""
|
|
|
|
for _ in range(3):
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
self.helper.invite(room_id, self.user_id, "@other-users:red")
|
|
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
self.helper.invite(room_id, self.user_id, "@other-users:red", expect_code=429)
|
|
|
|
|
|
class RoomJoinTestCase(RoomBase):
|
|
servlets = [
|
|
admin.register_servlets,
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
knock.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user1 = self.register_user("thomas", "hackme")
|
|
self.tok1 = self.login("thomas", "hackme")
|
|
|
|
self.user2 = self.register_user("teresa", "hackme")
|
|
self.tok2 = self.login("teresa", "hackme")
|
|
|
|
self.room1 = self.helper.create_room_as(room_creator=self.user1, tok=self.tok1)
|
|
self.room2 = self.helper.create_room_as(room_creator=self.user1, tok=self.tok1)
|
|
self.room3 = self.helper.create_room_as(room_creator=self.user1, tok=self.tok1)
|
|
|
|
self.store = hs.get_datastores().main
|
|
|
|
def test_spam_checker_may_join_room_deprecated(self) -> None:
|
|
"""Tests that the user_may_join_room spam checker callback is correctly called
|
|
and blocks room joins when needed.
|
|
|
|
This test uses the deprecated API, in which callbacks return booleans.
|
|
"""
|
|
|
|
# Register a dummy callback. Make it allow all room joins for now.
|
|
return_value = True
|
|
|
|
async def user_may_join_room(
|
|
userid: str,
|
|
room_id: str,
|
|
is_invited: bool,
|
|
) -> bool:
|
|
return return_value
|
|
|
|
# `spec` argument is needed for this function mock to have `__qualname__`, which
|
|
# is needed for `Measure` metrics buried in SpamChecker.
|
|
callback_mock = Mock(side_effect=user_may_join_room, spec=lambda *x: None)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_join_room_callbacks.append(
|
|
callback_mock
|
|
)
|
|
|
|
# Join a first room, without being invited to it.
|
|
self.helper.join(self.room1, self.user2, tok=self.tok2)
|
|
|
|
# Check that the callback was called with the right arguments.
|
|
expected_call_args = (
|
|
(
|
|
self.user2,
|
|
self.room1,
|
|
False,
|
|
),
|
|
)
|
|
self.assertEqual(
|
|
callback_mock.call_args,
|
|
expected_call_args,
|
|
callback_mock.call_args,
|
|
)
|
|
|
|
# Join a second room, this time with an invite for it.
|
|
self.helper.invite(self.room2, self.user1, self.user2, tok=self.tok1)
|
|
self.helper.join(self.room2, self.user2, tok=self.tok2)
|
|
|
|
# Check that the callback was called with the right arguments.
|
|
expected_call_args = (
|
|
(
|
|
self.user2,
|
|
self.room2,
|
|
True,
|
|
),
|
|
)
|
|
self.assertEqual(
|
|
callback_mock.call_args,
|
|
expected_call_args,
|
|
callback_mock.call_args,
|
|
)
|
|
|
|
# Now make the callback deny all room joins, and check that a join actually fails.
|
|
return_value = False
|
|
self.helper.join(
|
|
self.room3, self.user2, expect_code=HTTPStatus.FORBIDDEN, tok=self.tok2
|
|
)
|
|
|
|
def test_spam_checker_may_join_room(self) -> None:
|
|
"""Tests that the user_may_join_room spam checker callback is correctly called
|
|
and blocks room joins when needed.
|
|
|
|
This test uses the latest API to this day, in which callbacks return `NOT_SPAM` or `Codes`.
|
|
"""
|
|
|
|
# Register a dummy callback. Make it allow all room joins for now.
|
|
return_value: Literal["NOT_SPAM"] | tuple[Codes, dict] | Codes = (
|
|
synapse.module_api.NOT_SPAM
|
|
)
|
|
|
|
async def user_may_join_room(
|
|
userid: str,
|
|
room_id: str,
|
|
is_invited: bool,
|
|
) -> Literal["NOT_SPAM"] | tuple[Codes, dict] | Codes:
|
|
return return_value
|
|
|
|
# `spec` argument is needed for this function mock to have `__qualname__`, which
|
|
# is needed for `Measure` metrics buried in SpamChecker.
|
|
callback_mock = Mock(side_effect=user_may_join_room, spec=lambda *x: None)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_join_room_callbacks.append(
|
|
callback_mock
|
|
)
|
|
|
|
# Join a first room, without being invited to it.
|
|
self.helper.join(self.room1, self.user2, tok=self.tok2)
|
|
|
|
# Check that the callback was called with the right arguments.
|
|
expected_call_args = (
|
|
(
|
|
self.user2,
|
|
self.room1,
|
|
False,
|
|
),
|
|
)
|
|
self.assertEqual(
|
|
callback_mock.call_args,
|
|
expected_call_args,
|
|
callback_mock.call_args,
|
|
)
|
|
|
|
# Join a second room, this time with an invite for it.
|
|
self.helper.invite(self.room2, self.user1, self.user2, tok=self.tok1)
|
|
self.helper.join(self.room2, self.user2, tok=self.tok2)
|
|
|
|
# Check that the callback was called with the right arguments.
|
|
expected_call_args = (
|
|
(
|
|
self.user2,
|
|
self.room2,
|
|
True,
|
|
),
|
|
)
|
|
self.assertEqual(
|
|
callback_mock.call_args,
|
|
expected_call_args,
|
|
callback_mock.call_args,
|
|
)
|
|
|
|
# Now make the callback deny all room joins, and check that a join actually fails.
|
|
# We pick an arbitrary Codes rather than the default `Codes.FORBIDDEN`.
|
|
return_value = Codes.CONSENT_NOT_GIVEN
|
|
self.helper.invite(self.room3, self.user1, self.user2, tok=self.tok1)
|
|
self.helper.join(
|
|
self.room3,
|
|
self.user2,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
expect_errcode=return_value,
|
|
tok=self.tok2,
|
|
)
|
|
|
|
# Now make the callback deny all room joins, and check that a join actually fails.
|
|
# As above, with the experimental extension that lets us return dictionaries.
|
|
return_value = (Codes.BAD_ALIAS, {"another_field": "12345"})
|
|
self.helper.join(
|
|
self.room3,
|
|
self.user2,
|
|
expect_code=HTTPStatus.FORBIDDEN,
|
|
expect_errcode=return_value[0],
|
|
tok=self.tok2,
|
|
expect_additional_fields=return_value[1],
|
|
)
|
|
|
|
def test_suspended_user_cannot_join_room(self) -> None:
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user2, True))
|
|
|
|
channel = self.make_request(
|
|
"POST", f"/join/{self.room1}", access_token=self.tok2
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
channel = self.make_request(
|
|
"POST", f"/rooms/{self.room1}/join", access_token=self.tok2
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_cannot_knock_on_room(self) -> None:
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user2, True))
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/v3/knock/{self.room1}",
|
|
access_token=self.tok2,
|
|
content={},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_cannot_invite_to_room(self) -> None:
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
# first user invites second user
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/rooms/{self.room1}/invite",
|
|
access_token=self.tok1,
|
|
content={"user_id": self.user2},
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_can_leave_room(self) -> None:
|
|
channel = self.make_request(
|
|
"POST", f"/join/{self.room1}", access_token=self.tok1
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
# leave room
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/rooms/{self.room1}/leave",
|
|
access_token=self.tok1,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
|
|
class RoomAppserviceTsParamTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
room.register_servlets,
|
|
synapse.rest.admin.register_servlets,
|
|
register.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.appservice_user, _ = self.register_appservice_user(
|
|
"as_user_potato", self.appservice.token
|
|
)
|
|
|
|
# Create a room as the appservice user.
|
|
args = {
|
|
"access_token": self.appservice.token,
|
|
"user_id": self.appservice_user,
|
|
}
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/createRoom?{urlparse.urlencode(args)}",
|
|
content={"visibility": "public"},
|
|
)
|
|
|
|
assert channel.code == 200
|
|
self.room = channel.json_body["room_id"]
|
|
|
|
self.main_store = self.hs.get_datastores().main
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
config = self.default_config()
|
|
|
|
self.appservice = ApplicationService(
|
|
token="i_am_an_app_service",
|
|
id="1234",
|
|
namespaces={"users": [{"regex": r"@as_user.*", "exclusive": True}]},
|
|
# Note: this user does not have to match the regex above
|
|
sender=UserID.from_string("@as_main:test"),
|
|
)
|
|
|
|
mock_load_appservices = Mock(return_value=[self.appservice])
|
|
with patch(
|
|
"synapse.storage.databases.main.appservice.load_appservices",
|
|
mock_load_appservices,
|
|
):
|
|
hs = self.setup_test_homeserver(config=config)
|
|
return hs
|
|
|
|
def test_send_event_ts(self) -> None:
|
|
"""Test sending a non-state event with a custom timestamp."""
|
|
ts = 1
|
|
|
|
url_params = {
|
|
"user_id": self.appservice_user,
|
|
"ts": ts,
|
|
}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
path=f"/_matrix/client/r0/rooms/{self.room}/send/m.room.message/1234?"
|
|
+ urlparse.urlencode(url_params),
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
access_token=self.appservice.token,
|
|
)
|
|
self.assertEqual(channel.code, 200, channel.json_body)
|
|
event_id = channel.json_body["event_id"]
|
|
|
|
# Ensure the event was persisted with the correct timestamp.
|
|
res = self.get_success(self.main_store.get_event(event_id))
|
|
self.assertEqual(ts, res.origin_server_ts)
|
|
|
|
def test_send_state_event_ts(self) -> None:
|
|
"""Test sending a state event with a custom timestamp."""
|
|
ts = 1
|
|
|
|
url_params = {
|
|
"user_id": self.appservice_user,
|
|
"ts": ts,
|
|
}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
path=f"/_matrix/client/r0/rooms/{self.room}/state/m.room.name?"
|
|
+ urlparse.urlencode(url_params),
|
|
content={"name": "test"},
|
|
access_token=self.appservice.token,
|
|
)
|
|
self.assertEqual(channel.code, 200, channel.json_body)
|
|
event_id = channel.json_body["event_id"]
|
|
|
|
# Ensure the event was persisted with the correct timestamp.
|
|
res = self.get_success(self.main_store.get_event(event_id))
|
|
self.assertEqual(ts, res.origin_server_ts)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_event_ts(self) -> None:
|
|
"""Test scheduling a delayed event with a custom timestamp."""
|
|
ts = 1
|
|
|
|
url_params = {
|
|
"user_id": self.appservice_user,
|
|
"ts": ts,
|
|
}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
path=f"/_matrix/client/unstable/org.matrix.msc4140/rooms/{self.room}/delayed_event/m.room.message/1234?"
|
|
+ urlparse.urlencode(url_params),
|
|
content={
|
|
"delay_ms": 2000,
|
|
"content": {"body": "test", "msgtype": "m.text"},
|
|
},
|
|
access_token=self.appservice.token,
|
|
)
|
|
self.assertEqual(channel.code, 200, channel.json_body)
|
|
|
|
# Wait for the delayed event to be sent, then ensure it was persisted
|
|
# with the custom timestamp.
|
|
self.reactor.advance(Duration(seconds=2).as_secs())
|
|
(event_id,) = self.get_success(
|
|
self.main_store.get_latest_event_ids_in_room(self.room)
|
|
)
|
|
res = self.get_success(self.main_store.get_event(event_id))
|
|
self.assertEqual("m.room.message", res.type, res)
|
|
self.assertEqual(ts, res.origin_server_ts)
|
|
|
|
def test_send_membership_event_ts(self) -> None:
|
|
"""Test sending a membership event with a custom timestamp."""
|
|
ts = 1
|
|
|
|
url_params = {
|
|
"user_id": self.appservice_user,
|
|
"ts": ts,
|
|
}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
path=f"/_matrix/client/r0/rooms/{self.room}/state/m.room.member/{self.appservice_user}?"
|
|
+ urlparse.urlencode(url_params),
|
|
content={"membership": "join", "display_name": "test"},
|
|
access_token=self.appservice.token,
|
|
)
|
|
self.assertEqual(channel.code, 200, channel.json_body)
|
|
event_id = channel.json_body["event_id"]
|
|
|
|
# Ensure the event was persisted with the correct timestamp.
|
|
res = self.get_success(self.main_store.get_event(event_id))
|
|
self.assertEqual(ts, res.origin_server_ts)
|
|
|
|
|
|
class RoomJoinRatelimitTestCase(RoomBase):
|
|
user_id = "@sid1:red"
|
|
|
|
servlets = [
|
|
admin.register_servlets,
|
|
profile.register_servlets,
|
|
room.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
super().prepare(reactor, clock, hs)
|
|
# profile changes expect that the user is actually registered
|
|
user = UserID.from_string(self.user_id)
|
|
self.register_user(user.localpart, "supersecretpassword")
|
|
|
|
@unittest.override_config(
|
|
{"rc_joins": {"local": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_join_local_ratelimit(self) -> None:
|
|
"""Tests that local joins are actually rate-limited."""
|
|
# Create 4 rooms
|
|
room_ids = [
|
|
self.helper.create_room_as(self.user_id, is_public=True) for _ in range(4)
|
|
]
|
|
|
|
joiner_user_id = self.register_user("joiner", "secret")
|
|
# Now make a new user try to join some of them.
|
|
|
|
# The user can join 3 rooms
|
|
for room_id in room_ids[0:3]:
|
|
self.helper.join(room_id, joiner_user_id)
|
|
|
|
# But the user cannot join a 4th room
|
|
self.helper.join(
|
|
room_ids[3], joiner_user_id, expect_code=HTTPStatus.TOO_MANY_REQUESTS
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{"rc_joins": {"local": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_join_attempts_local_ratelimit(self) -> None:
|
|
"""Tests that unsuccessful joins that end up being denied are rate-limited."""
|
|
# Create 4 rooms
|
|
room_ids = [
|
|
self.helper.create_room_as(self.user_id, is_public=True) for _ in range(4)
|
|
]
|
|
# Pre-emptively ban the user who will attempt to join.
|
|
joiner_user_id = self.register_user("joiner", "secret")
|
|
for room_id in room_ids:
|
|
self.helper.ban(room_id, self.user_id, joiner_user_id)
|
|
|
|
# Now make a new user try to join some of them.
|
|
# The user can make 3 requests, each of which should be denied.
|
|
for room_id in room_ids[0:3]:
|
|
self.helper.join(room_id, joiner_user_id, expect_code=HTTPStatus.FORBIDDEN)
|
|
|
|
# The fourth attempt should be rate limited.
|
|
self.helper.join(
|
|
room_ids[3], joiner_user_id, expect_code=HTTPStatus.TOO_MANY_REQUESTS
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{"rc_joins": {"local": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_join_local_ratelimit_profile_change(self) -> None:
|
|
"""Tests that sending a profile update into all of the user's joined rooms isn't
|
|
rate-limited by the rate-limiter on joins."""
|
|
|
|
# Create and join as many rooms as the rate-limiting config allows in a second.
|
|
room_ids = [
|
|
self.helper.create_room_as(self.user_id),
|
|
self.helper.create_room_as(self.user_id),
|
|
self.helper.create_room_as(self.user_id),
|
|
]
|
|
# Let some time for the rate-limiter to forget about our multi-join.
|
|
self.reactor.advance(2)
|
|
# Add one to make sure we're joined to more rooms than the config allows us to
|
|
# join in a second.
|
|
room_ids.append(self.helper.create_room_as(self.user_id))
|
|
|
|
# Update the display name for the user.
|
|
path = "/_matrix/client/r0/profile/%s/displayname" % self.user_id
|
|
channel = self.make_request("PUT", path, {"displayname": "John Doe"})
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# Check that all the rooms have been sent a profile update into.
|
|
for room_id in room_ids:
|
|
path = "/_matrix/client/r0/rooms/%s/state/m.room.member/%s" % (
|
|
room_id,
|
|
self.user_id,
|
|
)
|
|
|
|
channel = self.make_request("GET", path)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
self.assertIn("displayname", channel.json_body)
|
|
self.assertEqual(channel.json_body["displayname"], "John Doe")
|
|
|
|
@unittest.override_config(
|
|
{"rc_joins": {"local": {"per_second": 0.5, "burst_count": 3}}}
|
|
)
|
|
def test_join_local_ratelimit_idempotent(self) -> None:
|
|
"""Tests that the room join endpoints remain idempotent despite rate-limiting
|
|
on room joins."""
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
# Let's test both paths to be sure.
|
|
paths_to_test = [
|
|
"/_matrix/client/r0/rooms/%s/join",
|
|
"/_matrix/client/r0/join/%s",
|
|
]
|
|
|
|
for path in paths_to_test:
|
|
# Make sure we send more requests than the rate-limiting config would allow
|
|
# if all of these requests ended up joining the user to a room.
|
|
for _ in range(4):
|
|
channel = self.make_request("POST", path % room_id, {})
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"rc_joins": {"local": {"per_second": 0.5, "burst_count": 3}},
|
|
"auto_join_rooms": ["#room:red", "#room2:red", "#room3:red", "#room4:red"],
|
|
"autocreate_auto_join_rooms": True,
|
|
},
|
|
)
|
|
def test_autojoin_rooms(self) -> None:
|
|
user_id = self.register_user("testuser", "password")
|
|
|
|
# Check that the new user successfully joined the four rooms
|
|
rooms = self.get_success(
|
|
self.hs.get_datastores().main.get_rooms_for_user(user_id)
|
|
)
|
|
self.assertEqual(len(rooms), 4)
|
|
|
|
|
|
class RoomMessagesTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/messages/$user_id/$msg_id REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
def test_invalid_puts(self) -> None:
|
|
path = "/rooms/%s/send/m.room.message/mid1" % (urlparse.quote(self.room_id))
|
|
# missing keys or invalid json
|
|
channel = self.make_request("PUT", path, b"{}")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b'{"_name":"bo"}')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b'{"nao')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b'[{"_name":"bo"},{"_name":"jill"}]')
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b"text only")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
channel = self.make_request("PUT", path, b"")
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
def test_rooms_messages_sent(self) -> None:
|
|
path = "/rooms/%s/send/m.room.message/mid1" % (urlparse.quote(self.room_id))
|
|
|
|
content = b'{"body":"test","msgtype":{"type":"a"}}'
|
|
channel = self.make_request("PUT", path, content)
|
|
self.assertEqual(
|
|
HTTPStatus.BAD_REQUEST, channel.code, msg=channel.result["body"]
|
|
)
|
|
|
|
# custom message types
|
|
content = b'{"body":"test","msgtype":"test.custom.text"}'
|
|
channel = self.make_request("PUT", path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
# m.text message type
|
|
path = "/rooms/%s/send/m.room.message/mid2" % (urlparse.quote(self.room_id))
|
|
content = b'{"body":"test2","msgtype":"m.text"}'
|
|
channel = self.make_request("PUT", path, content)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
@parameterized.expand(
|
|
[
|
|
# Allow
|
|
param(
|
|
name="NOT_SPAM",
|
|
value="NOT_SPAM",
|
|
expected_code=HTTPStatus.OK,
|
|
expected_fields={},
|
|
),
|
|
param(
|
|
name="False",
|
|
value=False,
|
|
expected_code=HTTPStatus.OK,
|
|
expected_fields={},
|
|
),
|
|
# Block
|
|
param(
|
|
name="scalene string",
|
|
value="ANY OTHER STRING",
|
|
expected_code=HTTPStatus.FORBIDDEN,
|
|
expected_fields={"errcode": "M_FORBIDDEN"},
|
|
),
|
|
param(
|
|
name="True",
|
|
value=True,
|
|
expected_code=HTTPStatus.FORBIDDEN,
|
|
expected_fields={"errcode": "M_FORBIDDEN"},
|
|
),
|
|
param(
|
|
name="Code",
|
|
value=Codes.LIMIT_EXCEEDED,
|
|
expected_code=HTTPStatus.FORBIDDEN,
|
|
expected_fields={"errcode": "M_LIMIT_EXCEEDED"},
|
|
),
|
|
param(
|
|
name="Tuple",
|
|
value=(Codes.SERVER_NOT_TRUSTED, {"additional_field": "12345"}),
|
|
expected_code=HTTPStatus.FORBIDDEN,
|
|
expected_fields={
|
|
"errcode": "M_SERVER_NOT_TRUSTED",
|
|
"additional_field": "12345",
|
|
},
|
|
),
|
|
]
|
|
)
|
|
def test_spam_checker_check_event_for_spam(
|
|
self,
|
|
name: str,
|
|
value: str | bool | Codes | tuple[Codes, JsonDict],
|
|
expected_code: int,
|
|
expected_fields: dict,
|
|
) -> None:
|
|
class SpamCheck:
|
|
mock_return_value: str | bool | Codes | tuple[Codes, JsonDict] | bool = (
|
|
"NOT_SPAM"
|
|
)
|
|
mock_content: JsonMapping | None = None
|
|
|
|
async def check_event_for_spam(
|
|
self,
|
|
event: synapse.events.EventBase,
|
|
) -> str | Codes | tuple[Codes, JsonDict] | bool:
|
|
self.mock_content = event.content
|
|
return self.mock_return_value
|
|
|
|
spam_checker = SpamCheck()
|
|
|
|
self.hs.get_module_api_callbacks().spam_checker._check_event_for_spam_callbacks.append(
|
|
spam_checker.check_event_for_spam
|
|
)
|
|
|
|
# Inject `value` as mock_return_value
|
|
spam_checker.mock_return_value = value
|
|
path = "/rooms/%s/send/m.room.message/check_event_for_spam_%s" % (
|
|
urlparse.quote(self.room_id),
|
|
urlparse.quote(name),
|
|
)
|
|
body = "test-%s" % name
|
|
content = '{"body":"%s","msgtype":"m.text"}' % body
|
|
channel = self.make_request("PUT", path, content)
|
|
|
|
# Check that the callback has witnessed the correct event.
|
|
self.assertIsNotNone(spam_checker.mock_content)
|
|
if (
|
|
spam_checker.mock_content is not None
|
|
): # Checked just above, but mypy doesn't know about that.
|
|
self.assertEqual(
|
|
spam_checker.mock_content["body"], body, spam_checker.mock_content
|
|
)
|
|
|
|
# Check that we have the correct result.
|
|
self.assertEqual(expected_code, channel.code, msg=channel.result["body"])
|
|
for expected_key, expected_value in expected_fields.items():
|
|
self.assertEqual(
|
|
channel.json_body.get(expected_key, None),
|
|
expected_value,
|
|
"Field %s absent or invalid " % expected_key,
|
|
)
|
|
|
|
|
|
class RoomPowerLevelOverridesTestCase(RoomBase):
|
|
"""Tests that the power levels can be overridden with server config."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
servlets = [
|
|
admin.register_servlets,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.admin_user_id = self.register_user("admin", "pass")
|
|
self.admin_access_token = self.login("admin", "pass")
|
|
|
|
def power_levels(self, room_id: str) -> dict[str, Any]:
|
|
return self.helper.get_state(
|
|
room_id, "m.room.power_levels", self.admin_access_token
|
|
)
|
|
|
|
def test_default_power_levels_with_room_override(self) -> None:
|
|
"""
|
|
Create a room, providing power level overrides.
|
|
When the `power_level_content_override` was provided, it should replace the
|
|
default power levels.
|
|
|
|
See https://github.com/matrix-org/matrix-spec/issues/492
|
|
- currently we overwrite each key of power_level_content_override
|
|
completely.
|
|
"""
|
|
|
|
room_id = self.helper.create_room_as(
|
|
self.user_id,
|
|
extra_content={
|
|
"power_level_content_override": {"events": {"custom.event": 0}}
|
|
},
|
|
)
|
|
self.assertEqual(
|
|
{
|
|
"custom.event": 0,
|
|
},
|
|
self.power_levels(room_id)["events"],
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {"events": {"custom.event": 0}},
|
|
}
|
|
},
|
|
)
|
|
def test_power_levels_with_server_override(self) -> None:
|
|
"""
|
|
With a server configured `default_power_level_content_override`, creating a room
|
|
without `power_level_content_override` should result in the server-level overrides
|
|
being applied.
|
|
|
|
Similar to https://github.com/matrix-org/matrix-spec/issues/492,
|
|
we overwrite each key of power_level_content_override completely.
|
|
"""
|
|
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
self.assertEqual(
|
|
{
|
|
"custom.event": 0,
|
|
},
|
|
self.power_levels(room_id)["events"],
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {
|
|
"events": {"server.event": 0},
|
|
"ban": 13,
|
|
},
|
|
}
|
|
},
|
|
)
|
|
def test_power_levels_with_server_and_room_overrides(self) -> None:
|
|
"""
|
|
With a server configured to modify the room-level defaults,
|
|
create a room, providing different overrides.
|
|
Confirm that the room's power levels reflect both overrides, and
|
|
choose the room overrides where they clash.
|
|
"""
|
|
|
|
room_id = self.helper.create_room_as(
|
|
self.user_id,
|
|
extra_content={
|
|
"power_level_content_override": {"events": {"room.event": 0}}
|
|
},
|
|
)
|
|
|
|
# Room override wins over server config
|
|
self.assertEqual(
|
|
{"room.event": 0},
|
|
self.power_levels(room_id)["events"],
|
|
)
|
|
|
|
# But where there is no room override, server config wins
|
|
self.assertEqual(13, self.power_levels(room_id)["ban"])
|
|
|
|
|
|
class RoomPowerLevelOverridesInPracticeTestCase(RoomBase):
|
|
"""
|
|
Tests that we can really do various otherwise-prohibited actions
|
|
based on overriding the power levels in config.
|
|
"""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def test_creator_can_post_state_event(self) -> None:
|
|
# Given I am the creator of a room
|
|
room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am allowed
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
def test_normal_user_can_not_post_state_event(self) -> None:
|
|
# Given I am a normal member of a room
|
|
room_id = self.helper.create_room_as("@some_other_guy:red")
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am not allowed because state events require PL>=50
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(
|
|
"You don't have permission to post that to the room. "
|
|
"user_level (0) < send_level (50)",
|
|
channel.json_body["error"],
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {"events": {"custom.event": 0}},
|
|
}
|
|
},
|
|
)
|
|
def test_with_config_override_normal_user_can_post_state_event(self) -> None:
|
|
# Given the server has config allowing normal users to post my event type,
|
|
# and I am a normal member of a room
|
|
room_id = self.helper.create_room_as("@some_other_guy:red")
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am allowed
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.result["body"])
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {"events": {"custom.event": 0}},
|
|
}
|
|
},
|
|
)
|
|
def test_any_room_override_defeats_config_override(self) -> None:
|
|
# Given the server has config allowing normal users to post my event type
|
|
# And I am a normal member of a room
|
|
# But the room was created with special permissions
|
|
extra_content: dict[str, Any] = {
|
|
"power_level_content_override": {"events": {}},
|
|
}
|
|
room_id = self.helper.create_room_as(
|
|
"@some_other_guy:red", extra_content=extra_content
|
|
)
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am not allowed
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {"events": {"custom.event": 0}},
|
|
}
|
|
},
|
|
)
|
|
def test_specific_room_override_defeats_config_override(self) -> None:
|
|
# Given the server has config allowing normal users to post my event type,
|
|
# and I am a normal member of a room,
|
|
# but the room was created with special permissions for this event type
|
|
extra_content = {
|
|
"power_level_content_override": {"events": {"custom.event": 1}},
|
|
}
|
|
room_id = self.helper.create_room_as(
|
|
"@some_other_guy:red", extra_content=extra_content
|
|
)
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am not allowed
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(
|
|
"You don't have permission to post that to the room. "
|
|
+ "user_level (0) < send_level (1)",
|
|
channel.json_body["error"],
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"public_chat": {"events": {"custom.event": 0}},
|
|
"private_chat": None,
|
|
"trusted_private_chat": None,
|
|
}
|
|
},
|
|
)
|
|
def test_config_override_applies_only_to_specific_preset(self) -> None:
|
|
# Given the server has config for public_chats,
|
|
# and I am a normal member of a private_chat room
|
|
room_id = self.helper.create_room_as("@some_other_guy:red", is_public=False)
|
|
self.helper.invite(room=room_id, src="@some_other_guy:red", targ=self.user_id)
|
|
self.helper.join(room=room_id, user=self.user_id)
|
|
|
|
# When I send a state event
|
|
path = "/rooms/{room_id}/state/custom.event/my_state_key".format(
|
|
room_id=urlparse.quote(room_id),
|
|
)
|
|
channel = self.make_request("PUT", path, "{}")
|
|
|
|
# Then I am not allowed because the public_chat config does not
|
|
# affect this room, because this room is a private_chat
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(
|
|
"You don't have permission to post that to the room. "
|
|
+ "user_level (0) < send_level (50)",
|
|
channel.json_body["error"],
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"default_power_level_content_override": {
|
|
"private_chat": {
|
|
"events": {
|
|
"m.room.avatar": 50,
|
|
"m.room.canonical_alias": 50,
|
|
"m.room.encryption": 999,
|
|
"m.room.history_visibility": 100,
|
|
"m.room.name": 50,
|
|
"m.room.power_levels": 100,
|
|
"m.room.server_acl": 100,
|
|
"m.room.tombstone": 100,
|
|
},
|
|
"events_default": 0,
|
|
},
|
|
}
|
|
},
|
|
)
|
|
def test_config_override_blocks_encrypted_room(self) -> None:
|
|
# Given the server has config for private_chats,
|
|
|
|
# When I attempt to create an encrypted private_chat room
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
'{"creation_content": {"m.federate": false},"name": "Secret Private Room","preset": "private_chat","initial_state": [{"type": "m.room.encryption","state_key": "","content": {"algorithm": "m.megolm.v1.aes-sha2"}}]}',
|
|
)
|
|
|
|
# Then I am not allowed because the required power level is unattainable
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, msg=channel.result["body"])
|
|
self.assertEqual(
|
|
"You cannot create an encrypted room. "
|
|
+ "user_level (100) < send_level (999)",
|
|
channel.json_body["error"],
|
|
)
|
|
|
|
|
|
class RoomInitialSyncTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/initialSync."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
# create the room
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
def test_initial_sync(self) -> None:
|
|
channel = self.make_request("GET", "/rooms/%s/initialSync" % self.room_id)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
|
|
self.assertEqual(self.room_id, channel.json_body["room_id"])
|
|
self.assertEqual("join", channel.json_body["membership"])
|
|
|
|
# Room state is easier to assert on if we unpack it into a dict
|
|
state: JsonDict = {}
|
|
for event in channel.json_body["state"]:
|
|
if "state_key" not in event:
|
|
continue
|
|
t = event["type"]
|
|
if t not in state:
|
|
state[t] = []
|
|
state[t].append(event)
|
|
|
|
self.assertTrue("m.room.create" in state)
|
|
|
|
self.assertTrue("messages" in channel.json_body)
|
|
self.assertTrue("chunk" in channel.json_body["messages"])
|
|
self.assertTrue("end" in channel.json_body["messages"])
|
|
|
|
self.assertTrue("presence" in channel.json_body)
|
|
|
|
presence_by_user = {
|
|
e["content"]["user_id"]: e for e in channel.json_body["presence"]
|
|
}
|
|
self.assertTrue(self.user_id in presence_by_user)
|
|
self.assertEqual(EduTypes.PRESENCE, presence_by_user[self.user_id]["type"])
|
|
|
|
|
|
class RoomMessageListTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/messages REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
def test_topo_token_is_accepted(self) -> None:
|
|
token = "t1-0_0_0_0_0_0_0_0_0_0_0_0_0_0"
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/messages?access_token=x&from=%s" % (self.room_id, token)
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("start" in channel.json_body)
|
|
self.assertEqual(token, channel.json_body["start"])
|
|
self.assertTrue("chunk" in channel.json_body)
|
|
self.assertTrue("end" in channel.json_body)
|
|
|
|
def test_stream_token_is_accepted_for_fwd_pagianation(self) -> None:
|
|
token = "s0_0_0_0_0_0_0_0_0_0_0_0_0_0"
|
|
channel = self.make_request(
|
|
"GET", "/rooms/%s/messages?access_token=x&from=%s" % (self.room_id, token)
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code)
|
|
self.assertTrue("start" in channel.json_body)
|
|
self.assertEqual(token, channel.json_body["start"])
|
|
self.assertTrue("chunk" in channel.json_body)
|
|
self.assertTrue("end" in channel.json_body)
|
|
|
|
def test_room_messages_purge(self) -> None:
|
|
store = self.hs.get_datastores().main
|
|
pagination_handler = self.hs.get_pagination_handler()
|
|
|
|
# Send a first message in the room, which will be removed by the purge.
|
|
first_event_id = self.helper.send(self.room_id, "message 1")["event_id"]
|
|
first_token = self.get_success(
|
|
store.get_topological_token_for_event(first_event_id)
|
|
)
|
|
first_token_str = self.get_success(first_token.to_string(store))
|
|
|
|
# Send a second message in the room, which won't be removed, and which we'll
|
|
# use as the marker to purge events before.
|
|
second_event_id = self.helper.send(self.room_id, "message 2")["event_id"]
|
|
second_token = self.get_success(
|
|
store.get_topological_token_for_event(second_event_id)
|
|
)
|
|
second_token_str = self.get_success(second_token.to_string(store))
|
|
|
|
# Send a third event in the room to ensure we don't fall under any edge case
|
|
# due to our marker being the latest forward extremity in the room.
|
|
self.helper.send(self.room_id, "message 3")
|
|
|
|
# Check that we get the first and second message when querying /messages.
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=x&from=%s&dir=b&filter=%s"
|
|
% (
|
|
self.room_id,
|
|
second_token_str,
|
|
json.dumps({"types": [EventTypes.Message]}),
|
|
),
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
chunk = channel.json_body["chunk"]
|
|
self.assertEqual(len(chunk), 2, [event["content"] for event in chunk])
|
|
|
|
# Purge every event before the second event.
|
|
self.get_success(
|
|
pagination_handler.purge_history(
|
|
room_id=self.room_id,
|
|
token=second_token_str,
|
|
delete_local_events=True,
|
|
)
|
|
)
|
|
|
|
# Check that we only get the second message through /message now that the first
|
|
# has been purged.
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=x&from=%s&dir=b&filter=%s"
|
|
% (
|
|
self.room_id,
|
|
second_token_str,
|
|
json.dumps({"types": [EventTypes.Message]}),
|
|
),
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
chunk = channel.json_body["chunk"]
|
|
self.assertEqual(len(chunk), 1, [event["content"] for event in chunk])
|
|
|
|
# Check that we get no event, but also no error, when querying /messages with
|
|
# the token that was pointing at the first event, because we don't have it
|
|
# anymore.
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=x&from=%s&dir=b&filter=%s"
|
|
% (
|
|
self.room_id,
|
|
first_token_str,
|
|
json.dumps({"types": [EventTypes.Message]}),
|
|
),
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
chunk = channel.json_body["chunk"]
|
|
self.assertEqual(len(chunk), 0, [event["content"] for event in chunk])
|
|
|
|
def test_room_message_filter_query_validation(self) -> None:
|
|
# Test json validation in (filter) query parameter.
|
|
# Does not test the validity of the filter, only the json validation.
|
|
|
|
# Check Get with valid json filter parameter, expect 200.
|
|
valid_filter_str = '{"types": ["m.room.message"]}'
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{self.room_id}/messages?access_token=x&dir=b&filter={valid_filter_str}",
|
|
)
|
|
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# Check Get with invalid json filter parameter, expect 400 NOT_JSON.
|
|
invalid_filter_str = "}}}{}"
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{self.room_id}/messages?access_token=x&dir=b&filter={invalid_filter_str}",
|
|
)
|
|
|
|
self.assertEqual(channel.code, HTTPStatus.BAD_REQUEST, channel.json_body)
|
|
self.assertEqual(
|
|
channel.json_body["errcode"], Codes.NOT_JSON, channel.json_body
|
|
)
|
|
|
|
def test_room_messages_paginate_through_rejected_events(
|
|
self,
|
|
) -> None:
|
|
"""Test that pagination continues past a batch of rejected events.
|
|
|
|
Regression test for https://github.com/element-hq/synapse/security/advisories/GHSA-6qf2-7x63-mm6v
|
|
|
|
Synapse before 1.152.1 had a bug meaning that a batch full of only
|
|
rejected events would cause `/messages` to not return any more
|
|
pagination tokens, falsely signalling the end of backpagination.
|
|
"""
|
|
# Send an early message that should not be filtered.
|
|
early_event_id = self.helper.send(self.room_id, "early message")["event_id"]
|
|
|
|
# Inject a batch of events and mark them as rejected in the database.
|
|
# We create more events than a single pagination request would fetch,
|
|
# so that one page of backward pagination request would only see rejected events.
|
|
for _ in range(3):
|
|
event = self.get_success(
|
|
inject_event(
|
|
self.hs,
|
|
room_id=self.room_id,
|
|
sender=self.user_id,
|
|
type=EventTypes.Message,
|
|
content={"body": "filtered event", "msgtype": "m.text"},
|
|
)
|
|
)
|
|
self.get_success(
|
|
self.hs.get_datastores().main.db_pool.runInteraction(
|
|
"mark_rejected",
|
|
self.hs.get_datastores().main.mark_event_rejected_txn,
|
|
event.event_id,
|
|
"testing",
|
|
)
|
|
)
|
|
|
|
# Send a message after all the rejected events.
|
|
latest_event_id = self.helper.send(self.room_id, "latest message")["event_id"]
|
|
|
|
# Start backpaginating.
|
|
channel = self.make_request(
|
|
"GET", f"/rooms/{self.room_id}/messages?dir=b&limit=2"
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
events_in_page = [e["event_id"] for e in channel.json_body["chunk"]]
|
|
end_token: str | None = channel.json_body["end"]
|
|
|
|
self.assertEqual(
|
|
events_in_page,
|
|
[latest_event_id],
|
|
"The latest event should be included in the first page we see whilst backpaginating",
|
|
)
|
|
|
|
event_ids_in_pages: list[list[str]] = [events_in_page]
|
|
|
|
# Bound the number of backpagination attempts to 2
|
|
for _ in range(2):
|
|
channel = self.make_request(
|
|
"GET", f"/rooms/{self.room_id}/messages?from={end_token}&dir=b&limit=2"
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
events_in_page = [e["event_id"] for e in channel.json_body["chunk"]]
|
|
event_ids_in_pages.append(events_in_page)
|
|
|
|
if early_event_id in events_in_page:
|
|
# We have found the event we were looking for
|
|
return
|
|
|
|
self.assertIn(
|
|
"end",
|
|
channel.json_body,
|
|
f"No `end` token received. Did not find {early_event_id} whilst backpaginating ({latest_event_id = }, {event_ids_in_pages = })",
|
|
)
|
|
# Use the end_token in the next iteration
|
|
end_token = channel.json_body["end"]
|
|
|
|
self.fail(
|
|
f"Exhausted backpagination attempts. Did not find {early_event_id} whilst backpaginating ({latest_event_id = }, {event_ids_in_pages = })"
|
|
)
|
|
|
|
|
|
class RoomMessageFilterTestCase(RoomBase):
|
|
"""Tests /rooms/$room_id/messages REST events."""
|
|
|
|
user_id = "@sid1:red"
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
def test_room_message_filter_wildcard(self) -> None:
|
|
# Send a first message in the room, which will be removed by the purge.
|
|
self.helper.send(self.room_id, "message 1", type="f.message.1")
|
|
self.helper.send(self.room_id, "message 1", type="f.message.2")
|
|
self.helper.send(self.room_id, "not returned in filter")
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=x&dir=b&filter=%s"
|
|
% (
|
|
self.room_id,
|
|
json.dumps({"types": ["f.message.*"]}),
|
|
),
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
chunk = channel.json_body["chunk"]
|
|
self.assertEqual(len(chunk), 2, [event["content"] for event in chunk])
|
|
|
|
|
|
class RoomDelayedEventTestCase(RoomBase):
|
|
"""Tests delayed events."""
|
|
|
|
user_id = "@sid1:red"
|
|
invalid_delay_error_type = Codes.INVALID_PARAM
|
|
|
|
def make_delayed_event_request(
|
|
self,
|
|
*,
|
|
room_id: str,
|
|
delay: Duration,
|
|
event_type: str,
|
|
state_key: str | None = None,
|
|
content: JsonDict,
|
|
txn_id: str | None = None,
|
|
) -> FakeChannel:
|
|
"""
|
|
Build and send a request for scheduling a delayed event.
|
|
|
|
Args:
|
|
room_id: The room to send the event to.
|
|
delay: How long to wait before sending the event.
|
|
event_type: The type of the event to send.
|
|
state_key: The state key of the event, or None for a message event.
|
|
content: The content of the event.
|
|
txn_id: The transaction ID for a delayed message event request.
|
|
Ignored for a delayed state event request.
|
|
Otherwise, generated if not given.
|
|
|
|
Returns:
|
|
The `FakeChannel` object which stores the result of the request.
|
|
"""
|
|
if state_key is not None:
|
|
path = f"rooms/{room_id}/state/{event_type}/{state_key}"
|
|
else:
|
|
if txn_id is None:
|
|
txn_id = "m%s" % (str(time.time()))
|
|
path = f"rooms/{room_id}/send/{event_type}/{txn_id}"
|
|
path += f"?org.matrix.msc4140.delay={delay.as_millis()}"
|
|
return self.make_request("PUT", path, content)
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_id = self.helper.create_room_as(self.user_id)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_invalid_event(self) -> None:
|
|
"""Test sending a delayed event with invalid content."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={},
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
# Assert that the standard error response uses a valid errcode.
|
|
# The specific errcode is irrelevant for the purpose of this test.
|
|
self.assertIsInstance(
|
|
channel.json_body.get("errcode"),
|
|
str,
|
|
)
|
|
|
|
def test_delayed_event_unsupported_by_default(self) -> None:
|
|
"""Test that sending a delayed event is unsupported with the default config."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.FORBIDDEN,
|
|
channel.json_body.get("errcode"),
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"max_event_delay_duration": "24h",
|
|
"experimental_features": {
|
|
"msc4140_max_delayed_events_per_user": 0,
|
|
},
|
|
}
|
|
)
|
|
def test_delayed_event_disabled_by_limit(self) -> None:
|
|
"""Test that delayed events are disabled by configuring the per-user limit to 0."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.FORBIDDEN,
|
|
channel.json_body.get("errcode"),
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "1000"})
|
|
def test_delayed_event_exceeds_max_delay(self) -> None:
|
|
"""Test that sending a delayed event fails if its delay is longer than allowed."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.DELAY_TOO_LARGE,
|
|
channel.json_body.get("errcode"),
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"max_event_delay_duration": "24h",
|
|
"experimental_features": {
|
|
"msc4140_max_delayed_events_per_user": 1,
|
|
},
|
|
}
|
|
)
|
|
def test_delayed_event_user_limit_reached(self) -> None:
|
|
"""Test that users cannot have more delayed events scheduled at once than allowed."""
|
|
# Disable rate-limits for this user. We want to specifically test the storage-based limit, not the request limits
|
|
self.get_success(
|
|
self.hs.get_datastores().main.set_ratelimit_for_user(self.user_id, 0, 0)
|
|
)
|
|
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=15000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
# Send a delayed event to eat up the limit
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
# Try to send another delayed event (we expect to hit the limit on the max number of delayed events that can be scheduled at once)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.TOO_MANY_REQUESTS, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.LIMIT_EXCEEDED,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
# Confirm that the response includes the time remaining until the next of the user's
|
|
# delayed events to be sent, at which point another delayed event may be scheduled
|
|
# without exceeding the limit
|
|
retry_after_headers = channel.headers.getRawHeaders("Retry-After")
|
|
assert retry_after_headers
|
|
retry_after_sec = int(retry_after_headers[0])
|
|
self.assertGreater(retry_after_sec, 0)
|
|
# Confirm that there is only a single value to the Retry-After header, as per RFC9110
|
|
self.assertEqual(1, len(retry_after_headers))
|
|
|
|
# Wait until we're able to retry again (the retry time from the error response)
|
|
self.reactor.advance(retry_after_sec)
|
|
|
|
# We should be able to send another delayed event again
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"max_event_delay_duration": "24h",
|
|
"experimental_features": {
|
|
"msc4140_max_delayed_events_per_user": 1,
|
|
},
|
|
}
|
|
)
|
|
def test_delayed_event_processed_user_limit_reached(self) -> None:
|
|
"""
|
|
Test that delayed events in the midst of being sent still count towards the limit of
|
|
how many delayed events a user may have scheduled at once.
|
|
"""
|
|
send_after = Duration(seconds=1)
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=send_after,
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
# Simulate the server taking a long time to persist delayed events
|
|
simulated_send_lag = Duration(seconds=5)
|
|
event_creation_handler = self.hs.get_event_creation_handler()
|
|
orig_send_fn = event_creation_handler.create_and_send_nonmember_event
|
|
|
|
async def slow_send_fn(*args: Any, **kwargs: Any) -> Any:
|
|
await self.clock.sleep(simulated_send_lag)
|
|
return await orig_send_fn(*args, **kwargs)
|
|
|
|
with patch.object(event_creation_handler, orig_send_fn.__name__, slow_send_fn):
|
|
self.reactor.advance(send_after.as_secs())
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.TOO_MANY_REQUESTS, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.LIMIT_EXCEEDED,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
# Confirm that the response lacks a Retry-After header, because the reason for this limit
|
|
# is the server taking an indeterminitely long time to process a delayed event, and the
|
|
# server doesn't know how much longer the client should wait before sending more requests
|
|
retry_after_headers = channel.headers.getRawHeaders("Retry-After")
|
|
assert not retry_after_headers
|
|
|
|
# Wait until the delayed event gets persisted
|
|
self.reactor.advance(simulated_send_lag.as_secs())
|
|
|
|
# We should be able to send another delayed event again
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"max_event_delay_duration": "24h",
|
|
"experimental_features": {
|
|
"msc4140_max_delayed_events_per_user": 5,
|
|
},
|
|
}
|
|
)
|
|
def test_delayed_event_user_limit_exceeded(self) -> None:
|
|
"""
|
|
Test that delayed event limits work properly when
|
|
the number of already scheduled events exceeds the configured limit.
|
|
|
|
This can be invoked by the server admin lowering the configured limit & restarting the server
|
|
while a user has fewer scheduled delayed events than the old limit, but more than the new limit.
|
|
"""
|
|
send_after: Duration
|
|
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=send_after,
|
|
event_type="m.room.message",
|
|
content={
|
|
"body": f"test (send after {send_after.as_secs()}s)",
|
|
"msgtype": "m.text",
|
|
},
|
|
)
|
|
|
|
for i in range(1, 5):
|
|
send_after = Duration(seconds=i)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
# Simulate restarting the server after having reconfigured the limit
|
|
# to be lower than the number of delayed events we just scheduled.
|
|
#
|
|
# Set the limit > 1 to test not having to wait for _all_ delayed events
|
|
# to be sent before being able to schedule a new one.
|
|
self.hs.config.server.max_delayed_events_per_user = 2
|
|
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.TOO_MANY_REQUESTS, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.LIMIT_EXCEEDED,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
retry_after_header = channel.headers.getRawHeaders("Retry-After")
|
|
assert retry_after_header
|
|
retry_after_sec = int(retry_after_header[0])
|
|
assert retry_after_sec > 0
|
|
|
|
# Wait until we're able to retry again (the retry time from the error response)
|
|
self.reactor.advance(retry_after_sec)
|
|
|
|
# We should be able to send another delayed event again
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
@parameterized.expand(
|
|
(
|
|
# A negative delay would put the send time in the past.
|
|
Duration(milliseconds=-2000),
|
|
# A zero delay is no delay at all; the regular send endpoints are for that.
|
|
Duration(),
|
|
)
|
|
)
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_delayed_event_with_invalid_delay(self, invalid_delay: Duration) -> None:
|
|
"""Test that sending a delayed event fails if its delay is not positive."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=invalid_delay,
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
self.invalid_delay_error_type,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@parameterized.expand(
|
|
(
|
|
# A negative delay would put the send time in the past.
|
|
Duration(milliseconds=-2000),
|
|
# A zero delay is no delay at all; the regular send endpoints are for that.
|
|
Duration(),
|
|
)
|
|
)
|
|
@unittest.override_config({"max_event_delay_duration": None})
|
|
def test_delayed_event_unsupported_with_invalid_delay(
|
|
self,
|
|
invalid_delay: Duration,
|
|
) -> None:
|
|
"""Test that delayed events being unsupported takes precedence over an invalid delay."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=invalid_delay,
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.FORBIDDEN, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.FORBIDDEN,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_message_event(self) -> None:
|
|
"""Test sending a valid delayed message event."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_message_event_idempotent(self) -> None:
|
|
"""
|
|
Test that repeated requests to schedule a delayed message event
|
|
with the same transaction ID are idempotent.
|
|
"""
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
txn_id="txn_id_0",
|
|
)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
delay_id = channel.json_body["delay_id"]
|
|
|
|
# Repeat the request, which reuses its txn_id
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
self.assertEqual(delay_id, channel.json_body.get("delay_id"))
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_state_event(self) -> None:
|
|
"""Test sending a valid delayed state event."""
|
|
channel = self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.topic",
|
|
state_key="",
|
|
content={"topic": "This is a topic"},
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
@unittest.override_config(
|
|
{
|
|
"max_event_delay_duration": "24h",
|
|
"rc_message": {"per_second": 1, "burst_count": 2},
|
|
}
|
|
)
|
|
def test_add_delayed_event_ratelimit(self) -> None:
|
|
"""Test that requests to schedule new delayed events are ratelimited by a RateLimiter,
|
|
which ratelimits them correctly, including by not limiting when the requester is
|
|
exempt from ratelimiting.
|
|
"""
|
|
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.message",
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
# Test that new delayed events are correctly ratelimited.
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.TOO_MANY_REQUESTS, channel.code, channel.result)
|
|
|
|
# Add the current user to the ratelimit overrides, allowing them no ratelimiting.
|
|
self.get_success(
|
|
self.hs.get_datastores().main.set_ratelimit_for_user(self.user_id, 0, 0)
|
|
)
|
|
|
|
# Test that the new delayed events aren't ratelimited anymore.
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
|
|
class RoomDelayedEventDedicatedEndpointTestCase(RoomDelayedEventTestCase):
|
|
"""Tests the dedicated request endpoint for scheduling delayed events."""
|
|
|
|
invalid_delay_error_type = Codes.BAD_JSON
|
|
|
|
def make_delayed_event_request(
|
|
self,
|
|
*,
|
|
room_id: str,
|
|
delay: Duration,
|
|
event_type: str,
|
|
state_key: str | None = None,
|
|
content: JsonDict,
|
|
txn_id: str | None = None,
|
|
) -> FakeChannel:
|
|
return self.make_request(
|
|
"PUT",
|
|
self.helper.build_delayed_event_request_path(
|
|
room_id=room_id,
|
|
event_type=event_type,
|
|
txn_id=txn_id,
|
|
),
|
|
self.helper.build_delayed_event_request_body(
|
|
delay=delay,
|
|
state_key=state_key,
|
|
content=content,
|
|
),
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_delayed_event_with_content_at_top(self) -> None:
|
|
"""Test that the dedicated endpoint fails with event content keys at the top level of the body."""
|
|
channel = self.make_request(
|
|
"PUT",
|
|
self.helper.build_delayed_event_request_path(
|
|
room_id=self.room_id,
|
|
event_type="m.room.message",
|
|
),
|
|
content={
|
|
"delay_ms": 2000,
|
|
"body": "test",
|
|
"msgtype": "m.text",
|
|
},
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.MISSING_PARAM,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@parameterized.expand(("delay_ms", "content"))
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_delayed_event_with_missing_key(self, missing_key: str) -> None:
|
|
"""Test that the dedicated endpoint fails with a body missing a required key."""
|
|
body = self.helper.build_delayed_event_request_body(
|
|
delay=Duration(milliseconds=2000),
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
del body[missing_key]
|
|
channel = self.make_request(
|
|
"PUT",
|
|
self.helper.build_delayed_event_request_path(
|
|
room_id=self.room_id,
|
|
event_type="m.room.message",
|
|
),
|
|
content=body,
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.MISSING_PARAM,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_delayed_event_with_no_keys(self) -> None:
|
|
"""Test that the dedicated endpoint fails with a body missing all required keys."""
|
|
channel = self.make_request(
|
|
"PUT",
|
|
self.helper.build_delayed_event_request_path(
|
|
room_id=self.room_id,
|
|
event_type="m.room.topic",
|
|
),
|
|
{},
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.BAD_JSON,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_delayed_event_with_null_state_key(self) -> None:
|
|
"""
|
|
Test that the dedicated endpoint fails to schedule a message event
|
|
with a "state_key" of `None` in the request body.
|
|
"""
|
|
body = self.helper.build_delayed_event_request_body(
|
|
delay=Duration(milliseconds=2000),
|
|
content={"body": "test", "msgtype": "m.text"},
|
|
)
|
|
body["state_key"] = None
|
|
channel = self.make_request(
|
|
"PUT",
|
|
self.helper.build_delayed_event_request_path(
|
|
room_id=self.room_id,
|
|
event_type="m.room.message",
|
|
),
|
|
body,
|
|
)
|
|
self.assertEqual(HTTPStatus.BAD_REQUEST, channel.code, channel.result)
|
|
self.assertEqual(
|
|
Codes.BAD_JSON,
|
|
channel.json_body["errcode"],
|
|
channel.json_body,
|
|
)
|
|
|
|
@unittest.override_config({"max_event_delay_duration": "24h"})
|
|
def test_send_delayed_state_event_idempotent(self) -> None:
|
|
"""
|
|
Test that repeated requests to schedule a delayed state event
|
|
with the same transaction ID are idempotent.
|
|
Note that only the dedicated endpoint can test this, as the one based on /state
|
|
does not support idempotent requests by using a transaction ID.
|
|
"""
|
|
make_delayed_event_request = lambda: self.make_delayed_event_request(
|
|
room_id=self.room_id,
|
|
delay=Duration(milliseconds=2000),
|
|
event_type="m.room.topic",
|
|
state_key="",
|
|
content={"topic": "This is a topic"},
|
|
txn_id="txn_id_0",
|
|
)
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
|
|
delay_id = channel.json_body["delay_id"]
|
|
|
|
# Repeat the request, which reuses its txn_id
|
|
channel = make_delayed_event_request()
|
|
self.assertEqual(HTTPStatus.OK, channel.code, channel.result)
|
|
self.assertEqual(delay_id, channel.json_body.get("delay_id"))
|
|
|
|
|
|
class RoomSearchTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
user_id = True
|
|
hijack_auth = False
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
# Register the user who does the searching
|
|
self.user_id2 = self.register_user("user", "pass")
|
|
self.access_token = self.login("user", "pass")
|
|
|
|
# Register the user who sends the message
|
|
self.other_user_id = self.register_user("otheruser", "pass")
|
|
self.other_access_token = self.login("otheruser", "pass")
|
|
|
|
# Create a room
|
|
self.room = self.helper.create_room_as(self.user_id2, tok=self.access_token)
|
|
|
|
# Invite the other person
|
|
self.helper.invite(
|
|
room=self.room,
|
|
src=self.user_id2,
|
|
tok=self.access_token,
|
|
targ=self.other_user_id,
|
|
)
|
|
|
|
# The other user joins
|
|
self.helper.join(
|
|
room=self.room, user=self.other_user_id, tok=self.other_access_token
|
|
)
|
|
|
|
def test_finds_message(self) -> None:
|
|
"""
|
|
The search functionality will search for content in messages if asked to
|
|
do so.
|
|
"""
|
|
# The other user sends some messages
|
|
self.helper.send(self.room, body="Hi!", tok=self.other_access_token)
|
|
self.helper.send(self.room, body="There!", tok=self.other_access_token)
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/search?access_token=%s" % (self.access_token,),
|
|
{
|
|
"search_categories": {
|
|
"room_events": {"keys": ["content.body"], "search_term": "Hi"}
|
|
}
|
|
},
|
|
)
|
|
|
|
# Check we get the results we expect -- one search result, of the sent
|
|
# messages
|
|
self.assertEqual(channel.code, 200)
|
|
results = channel.json_body["search_categories"]["room_events"]
|
|
self.assertEqual(results["count"], 1)
|
|
self.assertEqual(results["results"][0]["result"]["content"]["body"], "Hi!")
|
|
|
|
# No context was requested, so we should get none.
|
|
self.assertEqual(results["results"][0]["context"], {})
|
|
|
|
def test_include_context(self) -> None:
|
|
"""
|
|
When event_context includes include_profile, profile information will be
|
|
included in the search response.
|
|
"""
|
|
# The other user sends some messages
|
|
self.helper.send(self.room, body="Hi!", tok=self.other_access_token)
|
|
self.helper.send(self.room, body="There!", tok=self.other_access_token)
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/search?access_token=%s" % (self.access_token,),
|
|
{
|
|
"search_categories": {
|
|
"room_events": {
|
|
"keys": ["content.body"],
|
|
"search_term": "Hi",
|
|
"event_context": {"include_profile": True},
|
|
}
|
|
}
|
|
},
|
|
)
|
|
|
|
# Check we get the results we expect -- one search result, of the sent
|
|
# messages
|
|
self.assertEqual(channel.code, 200)
|
|
results = channel.json_body["search_categories"]["room_events"]
|
|
self.assertEqual(results["count"], 1)
|
|
self.assertEqual(results["results"][0]["result"]["content"]["body"], "Hi!")
|
|
|
|
# We should get context info, like the two users, and the display names.
|
|
context = results["results"][0]["context"]
|
|
self.assertEqual(len(context["profile_info"].keys()), 2)
|
|
self.assertEqual(
|
|
context["profile_info"][self.other_user_id]["displayname"], "otheruser"
|
|
)
|
|
|
|
|
|
class PublicRoomsRestrictedTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
self.url = b"/_matrix/client/r0/publicRooms"
|
|
|
|
config = self.default_config()
|
|
config["allow_public_rooms_without_auth"] = False
|
|
self.hs = self.setup_test_homeserver(config=config)
|
|
|
|
return self.hs
|
|
|
|
def test_restricted_no_auth(self) -> None:
|
|
channel = self.make_request("GET", self.url)
|
|
self.assertEqual(channel.code, HTTPStatus.UNAUTHORIZED, channel.result)
|
|
|
|
def test_restricted_auth(self) -> None:
|
|
self.register_user("user", "pass")
|
|
tok = self.login("user", "pass")
|
|
|
|
channel = self.make_request("GET", self.url, access_token=tok)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
|
|
class PublicRoomsRoomTypeFilterTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
config = self.default_config()
|
|
config["allow_public_rooms_without_auth"] = True
|
|
self.hs = self.setup_test_homeserver(config=config)
|
|
self.url = b"/_matrix/client/r0/publicRooms"
|
|
|
|
return self.hs
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
user = self.register_user("alice", "pass")
|
|
self.token = self.login(user, "pass")
|
|
|
|
# Create a room
|
|
self.helper.create_room_as(
|
|
user,
|
|
is_public=True,
|
|
extra_content={"visibility": "public"},
|
|
tok=self.token,
|
|
)
|
|
# Create a space
|
|
self.helper.create_room_as(
|
|
user,
|
|
is_public=True,
|
|
extra_content={
|
|
"visibility": "public",
|
|
"creation_content": {EventContentFields.ROOM_TYPE: RoomTypes.SPACE},
|
|
},
|
|
tok=self.token,
|
|
)
|
|
|
|
def default_config(self) -> JsonDict:
|
|
config = default_config(server_name="test")
|
|
config["room_list_publication_rules"] = [{"action": "allow"}]
|
|
return config
|
|
|
|
def make_public_rooms_request(
|
|
self,
|
|
room_types: list[str | None] | None,
|
|
instance_id: str | None = None,
|
|
) -> tuple[list[dict[str, Any]], int]:
|
|
body: JsonDict = {"filter": {PublicRoomsFilterFields.ROOM_TYPES: room_types}}
|
|
if instance_id:
|
|
body["third_party_instance_id"] = "test|test"
|
|
|
|
channel = self.make_request("POST", self.url, body, self.token)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
chunk = channel.json_body["chunk"]
|
|
count = channel.json_body["total_room_count_estimate"]
|
|
|
|
self.assertEqual(len(chunk), count)
|
|
|
|
return chunk, count
|
|
|
|
def test_returns_both_rooms_and_spaces_if_no_filter(self) -> None:
|
|
chunk, count = self.make_public_rooms_request(None)
|
|
self.assertEqual(count, 2)
|
|
|
|
# Also check if there's no filter property at all in the body.
|
|
channel = self.make_request("POST", self.url, {}, self.token)
|
|
self.assertEqual(channel.code, 200)
|
|
self.assertEqual(len(channel.json_body["chunk"]), 2)
|
|
self.assertEqual(channel.json_body["total_room_count_estimate"], 2)
|
|
|
|
chunk, count = self.make_public_rooms_request(None, "test|test")
|
|
self.assertEqual(count, 0)
|
|
|
|
def test_returns_only_rooms_based_on_filter(self) -> None:
|
|
chunk, count = self.make_public_rooms_request([None])
|
|
|
|
self.assertEqual(count, 1)
|
|
self.assertEqual(chunk[0].get("room_type", None), None)
|
|
|
|
chunk, count = self.make_public_rooms_request([None], "test|test")
|
|
self.assertEqual(count, 0)
|
|
|
|
def test_returns_only_space_based_on_filter(self) -> None:
|
|
chunk, count = self.make_public_rooms_request(["m.space"])
|
|
|
|
self.assertEqual(count, 1)
|
|
self.assertEqual(chunk[0].get("room_type", None), "m.space")
|
|
|
|
chunk, count = self.make_public_rooms_request(["m.space"], "test|test")
|
|
self.assertEqual(count, 0)
|
|
|
|
def test_returns_both_rooms_and_space_based_on_filter(self) -> None:
|
|
chunk, count = self.make_public_rooms_request(["m.space", None])
|
|
self.assertEqual(count, 2)
|
|
|
|
chunk, count = self.make_public_rooms_request(["m.space", None], "test|test")
|
|
self.assertEqual(count, 0)
|
|
|
|
def test_returns_both_rooms_and_spaces_if_array_is_empty(self) -> None:
|
|
chunk, count = self.make_public_rooms_request([])
|
|
self.assertEqual(count, 2)
|
|
|
|
chunk, count = self.make_public_rooms_request([], "test|test")
|
|
self.assertEqual(count, 0)
|
|
|
|
|
|
class PublicRoomsTestRemoteSearchFallbackTestCase(unittest.HomeserverTestCase):
|
|
"""Test that we correctly fallback to local filtering if a remote server
|
|
doesn't support search.
|
|
"""
|
|
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
return self.setup_test_homeserver(federation_client=AsyncMock())
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.register_user("user", "pass")
|
|
self.token = self.login("user", "pass")
|
|
|
|
self.federation_client = hs.get_federation_client()
|
|
|
|
def test_simple(self) -> None:
|
|
"Simple test for searching rooms over federation"
|
|
self.federation_client.get_public_rooms.return_value = {} # type: ignore[attr-defined]
|
|
|
|
search_filter = {PublicRoomsFilterFields.GENERIC_SEARCH_TERM: "foobar"}
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
b"/_matrix/client/r0/publicRooms?server=testserv",
|
|
content={"filter": search_filter},
|
|
access_token=self.token,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self.federation_client.get_public_rooms.assert_called_once_with( # type: ignore[attr-defined]
|
|
"testserv",
|
|
limit=100,
|
|
since_token=None,
|
|
search_filter=search_filter,
|
|
include_all_networks=False,
|
|
third_party_instance_id=None,
|
|
)
|
|
|
|
def test_fallback(self) -> None:
|
|
"Test that searching public rooms over federation falls back if it gets a 404"
|
|
|
|
# The `get_public_rooms` should be called again if the first call fails
|
|
# with a 404, when using search filters.
|
|
self.federation_client.get_public_rooms.side_effect = ( # type: ignore[attr-defined]
|
|
HttpResponseException(HTTPStatus.NOT_FOUND, "Not Found", b""),
|
|
{},
|
|
)
|
|
|
|
search_filter = {PublicRoomsFilterFields.GENERIC_SEARCH_TERM: "foobar"}
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
b"/_matrix/client/r0/publicRooms?server=testserv",
|
|
content={"filter": search_filter},
|
|
access_token=self.token,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self.federation_client.get_public_rooms.assert_has_calls( # type: ignore[attr-defined]
|
|
[
|
|
call(
|
|
"testserv",
|
|
limit=100,
|
|
since_token=None,
|
|
search_filter=search_filter,
|
|
include_all_networks=False,
|
|
third_party_instance_id=None,
|
|
),
|
|
call(
|
|
"testserv",
|
|
limit=None,
|
|
since_token=None,
|
|
search_filter=None,
|
|
include_all_networks=False,
|
|
third_party_instance_id=None,
|
|
),
|
|
]
|
|
)
|
|
|
|
|
|
class PerRoomProfilesForbiddenTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
profile.register_servlets,
|
|
]
|
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
|
config = self.default_config()
|
|
config["allow_per_room_profiles"] = False
|
|
self.hs = self.setup_test_homeserver(config=config)
|
|
|
|
return self.hs
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user_id = self.register_user("test", "test")
|
|
self.tok = self.login("test", "test")
|
|
|
|
# Set a profile for the test user
|
|
self.displayname = "test user"
|
|
request_data = {"displayname": self.displayname}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"/_matrix/client/r0/profile/%s/displayname" % (self.user_id,),
|
|
request_data,
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self.room_id = self.helper.create_room_as(self.user_id, tok=self.tok)
|
|
|
|
def test_join_has_the_global_profile(self) -> None:
|
|
"""A join without profile data gets the user's display name and
|
|
avatar."""
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"/_matrix/client/r0/profile/%s/avatar_url" % (self.user_id,),
|
|
{"avatar_url": "mxc://test/avatar"},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
room_id = self.helper.create_room_as(self.user_id, tok=self.tok)
|
|
member = self.helper.get_state(
|
|
room_id, "m.room.member", self.tok, state_key=self.user_id
|
|
)
|
|
self.assertEqual(member["displayname"], self.displayname)
|
|
self.assertEqual(member["avatar_url"], "mxc://test/avatar")
|
|
|
|
def test_per_room_profile_forbidden(self) -> None:
|
|
request_data = {"membership": "join", "displayname": "other test user"}
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"/_matrix/client/r0/rooms/%s/state/m.room.member/%s"
|
|
% (self.room_id, self.user_id),
|
|
request_data,
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
event_id = channel.json_body["event_id"]
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/_matrix/client/r0/rooms/%s/event/%s" % (self.room_id, event_id),
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
res_displayname = channel.json_body["content"]["displayname"]
|
|
self.assertEqual(res_displayname, self.displayname, channel.result)
|
|
|
|
|
|
class RoomMembershipReasonTestCase(unittest.HomeserverTestCase):
|
|
"""Tests that clients can add a "reason" field to membership events and
|
|
that they get correctly added to the generated events and propagated.
|
|
"""
|
|
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.creator = self.register_user("creator", "test")
|
|
self.creator_tok = self.login("creator", "test")
|
|
|
|
self.second_user_id = self.register_user("second", "test")
|
|
self.second_tok = self.login("second", "test")
|
|
|
|
self.room_id = self.helper.create_room_as(self.creator, tok=self.creator_tok)
|
|
|
|
def test_join_reason(self) -> None:
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/join",
|
|
content={"reason": reason},
|
|
access_token=self.second_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_leave_reason(self) -> None:
|
|
self.helper.join(self.room_id, user=self.second_user_id, tok=self.second_tok)
|
|
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/leave",
|
|
content={"reason": reason},
|
|
access_token=self.second_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_kick_reason(self) -> None:
|
|
self.helper.join(self.room_id, user=self.second_user_id, tok=self.second_tok)
|
|
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/kick",
|
|
content={"reason": reason, "user_id": self.second_user_id},
|
|
access_token=self.second_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_ban_reason(self) -> None:
|
|
self.helper.join(self.room_id, user=self.second_user_id, tok=self.second_tok)
|
|
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/ban",
|
|
content={"reason": reason, "user_id": self.second_user_id},
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_unban_reason(self) -> None:
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/unban",
|
|
content={"reason": reason, "user_id": self.second_user_id},
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_invite_reason(self) -> None:
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/invite",
|
|
content={"reason": reason, "user_id": self.second_user_id},
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def test_reject_invite_reason(self) -> None:
|
|
self.helper.invite(
|
|
self.room_id,
|
|
src=self.creator,
|
|
targ=self.second_user_id,
|
|
tok=self.creator_tok,
|
|
)
|
|
|
|
reason = "hello"
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/r0/rooms/{self.room_id}/leave",
|
|
content={"reason": reason},
|
|
access_token=self.second_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
self._check_for_reason(reason)
|
|
|
|
def _check_for_reason(self, reason: str) -> None:
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/_matrix/client/r0/rooms/{}/state/m.room.member/{}".format(
|
|
self.room_id, self.second_user_id
|
|
),
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
event_content = channel.json_body
|
|
|
|
self.assertEqual(event_content.get("reason"), reason, channel.result)
|
|
|
|
|
|
class RoomForgottenTestCase(unittest.HomeserverTestCase):
|
|
"""
|
|
Test forget/forgotten rooms
|
|
"""
|
|
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.store = hs.get_datastores().main
|
|
|
|
def test_room_not_forgotten_after_unban(self) -> None:
|
|
"""
|
|
Test what happens when someone is banned from a room, they forget the room, and
|
|
some time later are unbanned.
|
|
|
|
Currently, when they are unbanned, the room isn't forgotten anymore which may or
|
|
may not be expected.
|
|
"""
|
|
user1_id = self.register_user("user1", "pass")
|
|
user1_tok = self.login(user1_id, "pass")
|
|
user2_id = self.register_user("user2", "pass")
|
|
user2_tok = self.login(user2_id, "pass")
|
|
|
|
room_id = self.helper.create_room_as(user2_id, tok=user2_tok, is_public=True)
|
|
self.helper.join(room_id, user1_id, tok=user1_tok)
|
|
|
|
# User1 is banned and forgets the room
|
|
self.helper.ban(room_id, src=user2_id, targ=user1_id, tok=user2_tok)
|
|
# User1 forgets the room
|
|
self.get_success(self.store.forget(user1_id, room_id))
|
|
|
|
# The room should show up as forgotten
|
|
forgotten_room_ids = self.get_success(
|
|
self.store.get_forgotten_rooms_for_user(user1_id)
|
|
)
|
|
self.assertIncludes(forgotten_room_ids, {room_id}, exact=True)
|
|
|
|
# Unban user1
|
|
self.helper.change_membership(
|
|
room=room_id,
|
|
src=user2_id,
|
|
targ=user1_id,
|
|
membership=Membership.LEAVE,
|
|
tok=user2_tok,
|
|
)
|
|
|
|
# Room is no longer forgotten because it's a new membership
|
|
#
|
|
# XXX: Is this how we actually want it to behave? It seems like ideally, the
|
|
# room forgotten status should only be reset when the user decides to join again
|
|
# (or is invited/knocks). This way the room remains forgotten for any ban/leave
|
|
# transitions.
|
|
forgotten_room_ids = self.get_success(
|
|
self.store.get_forgotten_rooms_for_user(user1_id)
|
|
)
|
|
self.assertIncludes(forgotten_room_ids, set(), exact=True)
|
|
|
|
|
|
class LabelsTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
profile.register_servlets,
|
|
]
|
|
|
|
# Filter that should only catch messages with the label "#fun".
|
|
FILTER_LABELS = {
|
|
"types": [EventTypes.Message],
|
|
"org.matrix.labels": ["#fun"],
|
|
}
|
|
# Filter that should only catch messages without the label "#fun".
|
|
FILTER_NOT_LABELS = {
|
|
"types": [EventTypes.Message],
|
|
"org.matrix.not_labels": ["#fun"],
|
|
}
|
|
# Filter that should only catch messages with the label "#work" but without the label
|
|
# "#notfun".
|
|
FILTER_LABELS_NOT_LABELS = {
|
|
"types": [EventTypes.Message],
|
|
"org.matrix.labels": ["#work"],
|
|
"org.matrix.not_labels": ["#notfun"],
|
|
}
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user_id = self.register_user("test", "test")
|
|
self.tok = self.login("test", "test")
|
|
self.room_id = self.helper.create_room_as(self.user_id, tok=self.tok)
|
|
|
|
def test_context_filter_labels(self) -> None:
|
|
"""Test that we can filter by a label on a /context request."""
|
|
event_id = self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/context/%s?filter=%s"
|
|
% (self.room_id, event_id, json.dumps(self.FILTER_LABELS)),
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
events_before = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(
|
|
len(events_before), 1, [event["content"] for event in events_before]
|
|
)
|
|
self.assertEqual(
|
|
events_before[0]["content"]["body"], "with right label", events_before[0]
|
|
)
|
|
|
|
events_after = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(
|
|
len(events_after), 1, [event["content"] for event in events_after]
|
|
)
|
|
self.assertEqual(
|
|
events_after[0]["content"]["body"], "with right label", events_after[0]
|
|
)
|
|
|
|
def test_context_filter_not_labels(self) -> None:
|
|
"""Test that we can filter by the absence of a label on a /context request."""
|
|
event_id = self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/context/%s?filter=%s"
|
|
% (self.room_id, event_id, json.dumps(self.FILTER_NOT_LABELS)),
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
events_before = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(
|
|
len(events_before), 1, [event["content"] for event in events_before]
|
|
)
|
|
self.assertEqual(
|
|
events_before[0]["content"]["body"], "without label", events_before[0]
|
|
)
|
|
|
|
events_after = channel.json_body["events_after"]
|
|
|
|
self.assertEqual(
|
|
len(events_after), 2, [event["content"] for event in events_after]
|
|
)
|
|
self.assertEqual(
|
|
events_after[0]["content"]["body"], "with wrong label", events_after[0]
|
|
)
|
|
self.assertEqual(
|
|
events_after[1]["content"]["body"], "with two wrong labels", events_after[1]
|
|
)
|
|
|
|
def test_context_filter_labels_not_labels(self) -> None:
|
|
"""Test that we can filter by both a label and the absence of another label on a
|
|
/context request.
|
|
"""
|
|
event_id = self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/context/%s?filter=%s"
|
|
% (self.room_id, event_id, json.dumps(self.FILTER_LABELS_NOT_LABELS)),
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
events_before = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(
|
|
len(events_before), 0, [event["content"] for event in events_before]
|
|
)
|
|
|
|
events_after = channel.json_body["events_after"]
|
|
|
|
self.assertEqual(
|
|
len(events_after), 1, [event["content"] for event in events_after]
|
|
)
|
|
self.assertEqual(
|
|
events_after[0]["content"]["body"], "with wrong label", events_after[0]
|
|
)
|
|
|
|
def test_messages_filter_labels(self) -> None:
|
|
"""Test that we can filter by a label on a /messages request."""
|
|
self._send_labelled_messages_in_room()
|
|
|
|
token = "s0_0_0_0_0_0_0_0_0_0"
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=%s&from=%s&filter=%s"
|
|
% (self.room_id, self.tok, token, json.dumps(self.FILTER_LABELS)),
|
|
)
|
|
|
|
events = channel.json_body["chunk"]
|
|
|
|
self.assertEqual(len(events), 2, [event["content"] for event in events])
|
|
self.assertEqual(events[0]["content"]["body"], "with right label", events[0])
|
|
self.assertEqual(events[1]["content"]["body"], "with right label", events[1])
|
|
|
|
def test_messages_filter_not_labels(self) -> None:
|
|
"""Test that we can filter by the absence of a label on a /messages request."""
|
|
self._send_labelled_messages_in_room()
|
|
|
|
token = "s0_0_0_0_0_0_0_0_0_0"
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=%s&from=%s&filter=%s"
|
|
% (self.room_id, self.tok, token, json.dumps(self.FILTER_NOT_LABELS)),
|
|
)
|
|
|
|
events = channel.json_body["chunk"]
|
|
|
|
self.assertEqual(len(events), 4, [event["content"] for event in events])
|
|
self.assertEqual(events[0]["content"]["body"], "without label", events[0])
|
|
self.assertEqual(events[1]["content"]["body"], "without label", events[1])
|
|
self.assertEqual(events[2]["content"]["body"], "with wrong label", events[2])
|
|
self.assertEqual(
|
|
events[3]["content"]["body"], "with two wrong labels", events[3]
|
|
)
|
|
|
|
def test_messages_filter_labels_not_labels(self) -> None:
|
|
"""Test that we can filter by both a label and the absence of another label on a
|
|
/messages request.
|
|
"""
|
|
self._send_labelled_messages_in_room()
|
|
|
|
token = "s0_0_0_0_0_0_0_0_0_0"
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/rooms/%s/messages?access_token=%s&from=%s&filter=%s"
|
|
% (
|
|
self.room_id,
|
|
self.tok,
|
|
token,
|
|
json.dumps(self.FILTER_LABELS_NOT_LABELS),
|
|
),
|
|
)
|
|
|
|
events = channel.json_body["chunk"]
|
|
|
|
self.assertEqual(len(events), 1, [event["content"] for event in events])
|
|
self.assertEqual(events[0]["content"]["body"], "with wrong label", events[0])
|
|
|
|
def test_search_filter_labels(self) -> None:
|
|
"""Test that we can filter by a label on a /search request."""
|
|
request_data = {
|
|
"search_categories": {
|
|
"room_events": {
|
|
"search_term": "label",
|
|
"filter": self.FILTER_LABELS,
|
|
}
|
|
}
|
|
}
|
|
|
|
self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"POST", "/search?access_token=%s" % self.tok, request_data
|
|
)
|
|
|
|
results = channel.json_body["search_categories"]["room_events"]["results"]
|
|
|
|
self.assertEqual(
|
|
len(results),
|
|
2,
|
|
[result["result"]["content"] for result in results],
|
|
)
|
|
self.assertEqual(
|
|
results[0]["result"]["content"]["body"],
|
|
"with right label",
|
|
results[0]["result"]["content"]["body"],
|
|
)
|
|
self.assertEqual(
|
|
results[1]["result"]["content"]["body"],
|
|
"with right label",
|
|
results[1]["result"]["content"]["body"],
|
|
)
|
|
|
|
def test_search_filter_not_labels(self) -> None:
|
|
"""Test that we can filter by the absence of a label on a /search request."""
|
|
request_data = {
|
|
"search_categories": {
|
|
"room_events": {
|
|
"search_term": "label",
|
|
"filter": self.FILTER_NOT_LABELS,
|
|
}
|
|
}
|
|
}
|
|
|
|
self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"POST", "/search?access_token=%s" % self.tok, request_data
|
|
)
|
|
|
|
results = channel.json_body["search_categories"]["room_events"]["results"]
|
|
|
|
self.assertEqual(
|
|
len(results),
|
|
4,
|
|
[result["result"]["content"] for result in results],
|
|
)
|
|
self.assertEqual(
|
|
results[0]["result"]["content"]["body"],
|
|
"without label",
|
|
results[0]["result"]["content"]["body"],
|
|
)
|
|
self.assertEqual(
|
|
results[1]["result"]["content"]["body"],
|
|
"without label",
|
|
results[1]["result"]["content"]["body"],
|
|
)
|
|
self.assertEqual(
|
|
results[2]["result"]["content"]["body"],
|
|
"with wrong label",
|
|
results[2]["result"]["content"]["body"],
|
|
)
|
|
self.assertEqual(
|
|
results[3]["result"]["content"]["body"],
|
|
"with two wrong labels",
|
|
results[3]["result"]["content"]["body"],
|
|
)
|
|
|
|
def test_search_filter_labels_not_labels(self) -> None:
|
|
"""Test that we can filter by both a label and the absence of another label on a
|
|
/search request.
|
|
"""
|
|
request_data = {
|
|
"search_categories": {
|
|
"room_events": {
|
|
"search_term": "label",
|
|
"filter": self.FILTER_LABELS_NOT_LABELS,
|
|
}
|
|
}
|
|
}
|
|
|
|
self._send_labelled_messages_in_room()
|
|
|
|
channel = self.make_request(
|
|
"POST", "/search?access_token=%s" % self.tok, request_data
|
|
)
|
|
|
|
results = channel.json_body["search_categories"]["room_events"]["results"]
|
|
|
|
self.assertEqual(
|
|
len(results),
|
|
1,
|
|
[result["result"]["content"] for result in results],
|
|
)
|
|
self.assertEqual(
|
|
results[0]["result"]["content"]["body"],
|
|
"with wrong label",
|
|
results[0]["result"]["content"]["body"],
|
|
)
|
|
|
|
def _send_labelled_messages_in_room(self) -> str:
|
|
"""Sends several messages to a room with different labels (or without any) to test
|
|
filtering by label.
|
|
Returns:
|
|
The ID of the event to use if we're testing filtering on /context.
|
|
"""
|
|
self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={
|
|
"msgtype": "m.text",
|
|
"body": "with right label",
|
|
EventContentFields.LABELS: ["#fun"],
|
|
},
|
|
tok=self.tok,
|
|
)
|
|
|
|
self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={"msgtype": "m.text", "body": "without label"},
|
|
tok=self.tok,
|
|
)
|
|
|
|
res = self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={"msgtype": "m.text", "body": "without label"},
|
|
tok=self.tok,
|
|
)
|
|
# Return this event's ID when we test filtering in /context requests.
|
|
event_id = res["event_id"]
|
|
|
|
self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={
|
|
"msgtype": "m.text",
|
|
"body": "with wrong label",
|
|
EventContentFields.LABELS: ["#work"],
|
|
},
|
|
tok=self.tok,
|
|
)
|
|
|
|
self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={
|
|
"msgtype": "m.text",
|
|
"body": "with two wrong labels",
|
|
EventContentFields.LABELS: ["#work", "#notfun"],
|
|
},
|
|
tok=self.tok,
|
|
)
|
|
|
|
self.helper.send_event(
|
|
room_id=self.room_id,
|
|
type=EventTypes.Message,
|
|
content={
|
|
"msgtype": "m.text",
|
|
"body": "with right label",
|
|
EventContentFields.LABELS: ["#fun"],
|
|
},
|
|
tok=self.tok,
|
|
)
|
|
|
|
return event_id
|
|
|
|
|
|
class RelationsTestCase(PaginationTestCase):
|
|
def _filter_messages(self, filter: JsonDict) -> list[str]:
|
|
"""Make a request to /messages with a filter, returns the chunk of events."""
|
|
from_token = self.get_success(
|
|
self.from_token.to_string(self.hs.get_datastores().main)
|
|
)
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{self.room_id}/messages?filter={json.dumps(filter)}&dir=f&from={from_token}",
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
return [ev["event_id"] for ev in channel.json_body["chunk"]]
|
|
|
|
|
|
class ContextTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
account.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user_id = self.register_user("user", "password")
|
|
self.tok = self.login("user", "password")
|
|
self.room_id = self.helper.create_room_as(
|
|
self.user_id, tok=self.tok, is_public=False
|
|
)
|
|
|
|
self.other_user_id = self.register_user("user2", "password")
|
|
self.other_tok = self.login("user2", "password")
|
|
|
|
self.helper.invite(self.room_id, self.user_id, self.other_user_id, tok=self.tok)
|
|
self.helper.join(self.room_id, self.other_user_id, tok=self.other_tok)
|
|
|
|
def test_erased_sender(self) -> None:
|
|
"""Test that an erasure request results in the requester's events being hidden
|
|
from any new member of the room.
|
|
"""
|
|
|
|
# Send a bunch of events in the room.
|
|
|
|
self.helper.send(self.room_id, "message 1", tok=self.tok)
|
|
self.helper.send(self.room_id, "message 2", tok=self.tok)
|
|
event_id = self.helper.send(self.room_id, "message 3", tok=self.tok)["event_id"]
|
|
self.helper.send(self.room_id, "message 4", tok=self.tok)
|
|
self.helper.send(self.room_id, "message 5", tok=self.tok)
|
|
|
|
# Check that we can still see the messages before the erasure request.
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
'/rooms/%s/context/%s?filter={"types":["m.room.message"]}'
|
|
% (self.room_id, event_id),
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
events_before = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(len(events_before), 2, events_before)
|
|
self.assertEqual(
|
|
events_before[0].get("content", {}).get("body"),
|
|
"message 2",
|
|
events_before[0],
|
|
)
|
|
self.assertEqual(
|
|
events_before[1].get("content", {}).get("body"),
|
|
"message 1",
|
|
events_before[1],
|
|
)
|
|
|
|
self.assertEqual(
|
|
channel.json_body["event"].get("content", {}).get("body"),
|
|
"message 3",
|
|
channel.json_body["event"],
|
|
)
|
|
|
|
events_after = channel.json_body["events_after"]
|
|
|
|
self.assertEqual(len(events_after), 2, events_after)
|
|
self.assertEqual(
|
|
events_after[0].get("content", {}).get("body"),
|
|
"message 4",
|
|
events_after[0],
|
|
)
|
|
self.assertEqual(
|
|
events_after[1].get("content", {}).get("body"),
|
|
"message 5",
|
|
events_after[1],
|
|
)
|
|
|
|
# Deactivate the first account and erase the user's data.
|
|
|
|
deactivate_account_handler = self.hs.get_deactivate_account_handler()
|
|
self.get_success(
|
|
deactivate_account_handler.deactivate_account(
|
|
self.user_id, True, create_requester(self.user_id)
|
|
)
|
|
)
|
|
|
|
# Invite another user in the room. This is needed because messages will be
|
|
# pruned only if the user wasn't a member of the room when the messages were
|
|
# sent.
|
|
|
|
invited_user_id = self.register_user("user3", "password")
|
|
invited_tok = self.login("user3", "password")
|
|
|
|
self.helper.invite(
|
|
self.room_id, self.other_user_id, invited_user_id, tok=self.other_tok
|
|
)
|
|
self.helper.join(self.room_id, invited_user_id, tok=invited_tok)
|
|
|
|
# Check that a user that joined the room after the erasure request can't see
|
|
# the messages anymore.
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
'/rooms/%s/context/%s?filter={"types":["m.room.message"]}'
|
|
% (self.room_id, event_id),
|
|
access_token=invited_tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.result)
|
|
|
|
events_before = channel.json_body["events_before"]
|
|
|
|
self.assertEqual(len(events_before), 2, events_before)
|
|
self.assertDictEqual(events_before[0].get("content"), {}, events_before[0])
|
|
self.assertDictEqual(events_before[1].get("content"), {}, events_before[1])
|
|
|
|
self.assertDictEqual(
|
|
channel.json_body["event"].get("content"), {}, channel.json_body["event"]
|
|
)
|
|
|
|
events_after = channel.json_body["events_after"]
|
|
|
|
self.assertEqual(len(events_after), 2, events_after)
|
|
self.assertDictEqual(events_after[0].get("content"), {}, events_after[0])
|
|
self.assertEqual(events_after[1].get("content"), {}, events_after[1])
|
|
|
|
def test_room_event_context_filter_query_validation(self) -> None:
|
|
# Test json validation in (filter) query parameter.
|
|
# Does not test the validity of the filter, only the json validation.
|
|
event_id = self.helper.send(self.room_id, "message 7", tok=self.tok)["event_id"]
|
|
|
|
# Check Get with valid json filter parameter, expect 200.
|
|
valid_filter_str = '{"types": ["m.room.message"]}'
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{self.room_id}/context/{event_id}?filter={valid_filter_str}",
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# Check Get with invalid json filter parameter, expect 400 NOT_JSON.
|
|
invalid_filter_str = "}}}{}"
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/rooms/{self.room_id}/context/{event_id}?filter={invalid_filter_str}",
|
|
access_token=self.tok,
|
|
)
|
|
|
|
self.assertEqual(channel.code, HTTPStatus.BAD_REQUEST, channel.json_body)
|
|
self.assertEqual(
|
|
channel.json_body["errcode"], Codes.NOT_JSON, channel.json_body
|
|
)
|
|
|
|
|
|
class RoomAliasListTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
directory.register_servlets,
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_owner = self.register_user("room_owner", "test")
|
|
self.room_owner_tok = self.login("room_owner", "test")
|
|
|
|
self.room_id = self.helper.create_room_as(
|
|
self.room_owner, tok=self.room_owner_tok
|
|
)
|
|
|
|
def test_no_aliases(self) -> None:
|
|
res = self._get_aliases(self.room_owner_tok)
|
|
self.assertEqual(res["aliases"], [])
|
|
|
|
def test_not_in_room(self) -> None:
|
|
self.register_user("user", "test")
|
|
user_tok = self.login("user", "test")
|
|
res = self._get_aliases(user_tok, expected_code=403)
|
|
self.assertEqual(res["errcode"], "M_FORBIDDEN")
|
|
|
|
def test_admin_user(self) -> None:
|
|
alias1 = self._random_alias()
|
|
self._set_alias_via_directory(alias1)
|
|
|
|
self.register_user("user", "test", admin=True)
|
|
user_tok = self.login("user", "test")
|
|
|
|
res = self._get_aliases(user_tok)
|
|
self.assertEqual(res["aliases"], [alias1])
|
|
|
|
def test_with_aliases(self) -> None:
|
|
alias1 = self._random_alias()
|
|
alias2 = self._random_alias()
|
|
|
|
self._set_alias_via_directory(alias1)
|
|
self._set_alias_via_directory(alias2)
|
|
|
|
res = self._get_aliases(self.room_owner_tok)
|
|
self.assertEqual(set(res["aliases"]), {alias1, alias2})
|
|
|
|
def test_peekable_room(self) -> None:
|
|
alias1 = self._random_alias()
|
|
self._set_alias_via_directory(alias1)
|
|
|
|
self.helper.send_state(
|
|
self.room_id,
|
|
EventTypes.RoomHistoryVisibility,
|
|
body={"history_visibility": "world_readable"},
|
|
tok=self.room_owner_tok,
|
|
)
|
|
|
|
self.register_user("user", "test")
|
|
user_tok = self.login("user", "test")
|
|
|
|
res = self._get_aliases(user_tok)
|
|
self.assertEqual(res["aliases"], [alias1])
|
|
|
|
def _get_aliases(self, access_token: str, expected_code: int = 200) -> JsonDict:
|
|
"""Calls the endpoint under test. returns the json response object."""
|
|
channel = self.make_request(
|
|
"GET",
|
|
"/_matrix/client/r0/rooms/%s/aliases" % (self.room_id,),
|
|
access_token=access_token,
|
|
)
|
|
self.assertEqual(channel.code, expected_code, channel.result)
|
|
res = channel.json_body
|
|
self.assertIsInstance(res, dict)
|
|
if expected_code == 200:
|
|
self.assertIsInstance(res["aliases"], list)
|
|
return res
|
|
|
|
def _random_alias(self) -> str:
|
|
return RoomAlias(random_string(5), self.hs.hostname).to_string()
|
|
|
|
def _set_alias_via_directory(self, alias: str, expected_code: int = 200) -> None:
|
|
url = "/_matrix/client/r0/directory/room/" + alias
|
|
request_data = {"room_id": self.room_id}
|
|
|
|
channel = self.make_request(
|
|
"PUT", url, request_data, access_token=self.room_owner_tok
|
|
)
|
|
self.assertEqual(channel.code, expected_code, channel.result)
|
|
|
|
|
|
class RoomCanonicalAliasTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
directory.register_servlets,
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.room_owner = self.register_user("room_owner", "test")
|
|
self.room_owner_tok = self.login("room_owner", "test")
|
|
|
|
self.room_id = self.helper.create_room_as(
|
|
self.room_owner, tok=self.room_owner_tok
|
|
)
|
|
|
|
self.alias = "#alias:test"
|
|
self._set_alias_via_directory(self.alias)
|
|
|
|
def _set_alias_via_directory(self, alias: str, expected_code: int = 200) -> None:
|
|
url = "/_matrix/client/r0/directory/room/" + alias
|
|
request_data = {"room_id": self.room_id}
|
|
|
|
channel = self.make_request(
|
|
"PUT", url, request_data, access_token=self.room_owner_tok
|
|
)
|
|
self.assertEqual(channel.code, expected_code, channel.result)
|
|
|
|
def _get_canonical_alias(self, expected_code: int = 200) -> JsonDict:
|
|
"""Calls the endpoint under test. returns the json response object."""
|
|
channel = self.make_request(
|
|
"GET",
|
|
"rooms/%s/state/m.room.canonical_alias" % (self.room_id,),
|
|
access_token=self.room_owner_tok,
|
|
)
|
|
self.assertEqual(channel.code, expected_code, channel.result)
|
|
res = channel.json_body
|
|
self.assertIsInstance(res, dict)
|
|
return res
|
|
|
|
def _set_canonical_alias(
|
|
self, content: JsonDict, expected_code: int = 200
|
|
) -> JsonDict:
|
|
"""Calls the endpoint under test. returns the json response object."""
|
|
channel = self.make_request(
|
|
"PUT",
|
|
"rooms/%s/state/m.room.canonical_alias" % (self.room_id,),
|
|
content,
|
|
access_token=self.room_owner_tok,
|
|
)
|
|
self.assertEqual(channel.code, expected_code, channel.result)
|
|
res = channel.json_body
|
|
self.assertIsInstance(res, dict)
|
|
return res
|
|
|
|
def test_canonical_alias(self) -> None:
|
|
"""Test a basic alias message."""
|
|
# There is no canonical alias to start with.
|
|
self._get_canonical_alias(expected_code=404)
|
|
|
|
# Create an alias.
|
|
self._set_canonical_alias({"alias": self.alias})
|
|
|
|
# Canonical alias now exists!
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {"alias": self.alias})
|
|
|
|
# Now remove the alias.
|
|
self._set_canonical_alias({})
|
|
|
|
# There is an alias event, but it is empty.
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {})
|
|
|
|
def test_alt_aliases(self) -> None:
|
|
"""Test a canonical alias message with alt_aliases."""
|
|
# Create an alias.
|
|
self._set_canonical_alias({"alt_aliases": [self.alias]})
|
|
|
|
# Canonical alias now exists!
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {"alt_aliases": [self.alias]})
|
|
|
|
# Now remove the alt_aliases.
|
|
self._set_canonical_alias({})
|
|
|
|
# There is an alias event, but it is empty.
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {})
|
|
|
|
def test_alias_alt_aliases(self) -> None:
|
|
"""Test a canonical alias message with an alias and alt_aliases."""
|
|
# Create an alias.
|
|
self._set_canonical_alias({"alias": self.alias, "alt_aliases": [self.alias]})
|
|
|
|
# Canonical alias now exists!
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {"alias": self.alias, "alt_aliases": [self.alias]})
|
|
|
|
# Now remove the alias and alt_aliases.
|
|
self._set_canonical_alias({})
|
|
|
|
# There is an alias event, but it is empty.
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {})
|
|
|
|
def test_partial_modify(self) -> None:
|
|
"""Test removing only the alt_aliases."""
|
|
# Create an alias.
|
|
self._set_canonical_alias({"alias": self.alias, "alt_aliases": [self.alias]})
|
|
|
|
# Canonical alias now exists!
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {"alias": self.alias, "alt_aliases": [self.alias]})
|
|
|
|
# Now remove the alt_aliases.
|
|
self._set_canonical_alias({"alias": self.alias})
|
|
|
|
# There is an alias event, but it is empty.
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(res, {"alias": self.alias})
|
|
|
|
def test_add_alias(self) -> None:
|
|
"""Test removing only the alt_aliases."""
|
|
# Create an additional alias.
|
|
second_alias = "#second:test"
|
|
self._set_alias_via_directory(second_alias)
|
|
|
|
# Add the canonical alias.
|
|
self._set_canonical_alias({"alias": self.alias, "alt_aliases": [self.alias]})
|
|
|
|
# Then add the second alias.
|
|
self._set_canonical_alias(
|
|
{"alias": self.alias, "alt_aliases": [self.alias, second_alias]}
|
|
)
|
|
|
|
# Canonical alias now exists!
|
|
res = self._get_canonical_alias()
|
|
self.assertEqual(
|
|
res, {"alias": self.alias, "alt_aliases": [self.alias, second_alias]}
|
|
)
|
|
|
|
def test_bad_data(self) -> None:
|
|
"""Invalid data for alt_aliases should cause errors."""
|
|
self._set_canonical_alias({"alt_aliases": "@bad:test"}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": None}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": 0}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": 1}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": False}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": True}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": {}}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": [0]}, expected_code=400)
|
|
|
|
def test_bad_alias(self) -> None:
|
|
"""An alias which does not point to the room raises a SynapseError."""
|
|
self._set_canonical_alias({"alias": {"@unknown:test": "a"}}, expected_code=400)
|
|
self._set_canonical_alias({"alias": "@unknown:test"}, expected_code=400)
|
|
self._set_canonical_alias({"alt_aliases": ["@unknown:test"]}, expected_code=400)
|
|
|
|
|
|
class ThreepidInviteTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
admin.register_servlets,
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user_id = self.register_user("thomas", "hackme")
|
|
self.tok = self.login("thomas", "hackme")
|
|
|
|
self.room_id = self.helper.create_room_as(self.user_id, tok=self.tok)
|
|
|
|
def test_threepid_invite_spamcheck_deprecated(self) -> None:
|
|
"""
|
|
Test allowing/blocking threepid invites with a spam-check module.
|
|
|
|
In this test, we use the deprecated API in which callbacks return a bool.
|
|
"""
|
|
# Mock a few functions to prevent the test from failing due to failing to talk to
|
|
# a remote IS. We keep the mock for make_and_store_3pid_invite around so we
|
|
# can check its call_count later on during the test.
|
|
make_invite_mock = AsyncMock(return_value=(Mock(event_id="abc"), 0))
|
|
self.hs.get_room_member_handler()._make_and_store_3pid_invite = make_invite_mock # type: ignore[method-assign]
|
|
self.hs.get_identity_handler().lookup_3pid = AsyncMock( # type: ignore[method-assign]
|
|
return_value=None,
|
|
)
|
|
|
|
# Add a mock to the spamchecker callbacks for user_may_send_3pid_invite. Make it
|
|
# allow everything for now.
|
|
# `spec` argument is needed for this function mock to have `__qualname__`, which
|
|
# is needed for `Measure` metrics buried in SpamChecker.
|
|
mock = AsyncMock(return_value=True, spec=lambda *x: None)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_send_3pid_invite_callbacks.append(
|
|
mock
|
|
)
|
|
|
|
# Send a 3PID invite into the room and check that it succeeded.
|
|
email_to_invite = "teresa@example.com"
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"id_access_token": "sometoken",
|
|
"medium": "email",
|
|
"address": email_to_invite,
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
# Check that the callback was called with the right params.
|
|
mock.assert_called_with(self.user_id, "email", email_to_invite, self.room_id)
|
|
|
|
# Check that the call to send the invite was made.
|
|
make_invite_mock.assert_called_once()
|
|
|
|
# Now change the return value of the callback to deny any invite and test that
|
|
# we can't send the invite.
|
|
mock.return_value = False
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"id_access_token": "sometoken",
|
|
"medium": "email",
|
|
"address": email_to_invite,
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
|
|
# Also check that it stopped before calling _make_and_store_3pid_invite.
|
|
make_invite_mock.assert_called_once()
|
|
|
|
def test_threepid_invite_spamcheck(self) -> None:
|
|
"""
|
|
Test allowing/blocking threepid invites with a spam-check module.
|
|
|
|
In this test, we use the more recent API in which callbacks return a `Codes | Literal["NOT_SPAM"]`.
|
|
"""
|
|
# Mock a few functions to prevent the test from failing due to failing to talk to
|
|
# a remote IS. We keep the mock for make_and_store_3pid_invite around so we
|
|
# can check its call_count later on during the test.
|
|
make_invite_mock = AsyncMock(return_value=(Mock(event_id="abc"), 0))
|
|
self.hs.get_room_member_handler()._make_and_store_3pid_invite = make_invite_mock # type: ignore[method-assign]
|
|
self.hs.get_identity_handler().lookup_3pid = AsyncMock( # type: ignore[method-assign]
|
|
return_value=None,
|
|
)
|
|
|
|
# Add a mock to the spamchecker callbacks for user_may_send_3pid_invite. Make it
|
|
# allow everything for now.
|
|
# `spec` argument is needed for this function mock to have `__qualname__`, which
|
|
# is needed for `Measure` metrics buried in SpamChecker.
|
|
mock = AsyncMock(
|
|
return_value=synapse.module_api.NOT_SPAM,
|
|
spec=lambda *x: None,
|
|
)
|
|
self.hs.get_module_api_callbacks().spam_checker._user_may_send_3pid_invite_callbacks.append(
|
|
mock
|
|
)
|
|
|
|
# Send a 3PID invite into the room and check that it succeeded.
|
|
email_to_invite = "teresa@example.com"
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"id_access_token": "sometoken",
|
|
"medium": "email",
|
|
"address": email_to_invite,
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
# Check that the callback was called with the right params.
|
|
mock.assert_called_with(self.user_id, "email", email_to_invite, self.room_id)
|
|
|
|
# Check that the call to send the invite was made.
|
|
make_invite_mock.assert_called_once()
|
|
|
|
# Now change the return value of the callback to deny any invite and test that
|
|
# we can't send the invite. We pick an arbitrary error code to be able to check
|
|
# that the same code has been returned
|
|
mock.return_value = Codes.CONSENT_NOT_GIVEN
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"id_access_token": "sometoken",
|
|
"medium": "email",
|
|
"address": email_to_invite,
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
self.assertEqual(channel.json_body["errcode"], Codes.CONSENT_NOT_GIVEN)
|
|
|
|
# Also check that it stopped before calling _make_and_store_3pid_invite.
|
|
make_invite_mock.assert_called_once()
|
|
|
|
# Run variant with `Tuple[Codes, dict]`.
|
|
mock.return_value = (Codes.EXPIRED_ACCOUNT, {"field": "value"})
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"id_access_token": "sometoken",
|
|
"medium": "email",
|
|
"address": email_to_invite,
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 403)
|
|
self.assertEqual(channel.json_body["errcode"], Codes.EXPIRED_ACCOUNT)
|
|
self.assertEqual(channel.json_body["field"], "value")
|
|
|
|
# Also check that it stopped before calling _make_and_store_3pid_invite.
|
|
make_invite_mock.assert_called_once()
|
|
|
|
def test_400_missing_param_without_id_access_token(self) -> None:
|
|
"""
|
|
Test that a 3pid invite request returns 400 M_MISSING_PARAM
|
|
if we do not include id_access_token.
|
|
"""
|
|
channel = self.make_request(
|
|
method="POST",
|
|
path="/rooms/" + self.room_id + "/invite",
|
|
content={
|
|
"id_server": "example.com",
|
|
"medium": "email",
|
|
"address": "teresa@example.com",
|
|
},
|
|
access_token=self.tok,
|
|
)
|
|
self.assertEqual(channel.code, 400)
|
|
self.assertEqual(channel.json_body["errcode"], "M_MISSING_PARAM")
|
|
|
|
|
|
class TimestampLookupTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
admin.register_servlets,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self._storage_controllers = self.hs.get_storage_controllers()
|
|
|
|
self.room_owner = self.register_user("room_owner", "test")
|
|
self.room_owner_tok = self.login("room_owner", "test")
|
|
|
|
def _inject_outlier(self, room_id: str) -> EventBase:
|
|
event, _context = self.get_success(
|
|
create_event(
|
|
self.hs,
|
|
room_id=room_id,
|
|
type="m.test",
|
|
sender="@test_remote_user:remote",
|
|
)
|
|
)
|
|
|
|
event.internal_metadata.outlier = True
|
|
persistence = self._storage_controllers.persistence
|
|
assert persistence is not None
|
|
self.get_success(
|
|
persistence.persist_event(
|
|
event, EventContext.for_outlier(self._storage_controllers)
|
|
)
|
|
)
|
|
return event
|
|
|
|
def test_no_outliers(self) -> None:
|
|
"""
|
|
Test to make sure `/timestamp_to_event` does not return `outlier` events.
|
|
We're unable to determine whether an `outlier` is next to a gap so we
|
|
don't know whether it's actually the closest event. Instead, let's just
|
|
ignore `outliers` with this endpoint.
|
|
|
|
This test is really seeing that we choose the non-`outlier` event behind the
|
|
`outlier`. Since the gap checking logic considers the latest message in the room
|
|
as *not* next to a gap, asking over federation does not come into play here.
|
|
"""
|
|
room_id = self.helper.create_room_as(self.room_owner, tok=self.room_owner_tok)
|
|
|
|
outlier_event = self._inject_outlier(room_id)
|
|
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"/_matrix/client/v1/rooms/{room_id}/timestamp_to_event?dir=b&ts={outlier_event.origin_server_ts}",
|
|
access_token=self.room_owner_tok,
|
|
)
|
|
self.assertEqual(HTTPStatus.OK, channel.code, msg=channel.json_body)
|
|
|
|
# Make sure the outlier event is not returned
|
|
self.assertNotEqual(channel.json_body["event_id"], outlier_event.event_id)
|
|
|
|
|
|
class UserSuspensionTests(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
admin.register_servlets,
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
profile.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user1 = self.register_user("thomas", "hackme")
|
|
self.tok1 = self.login("thomas", "hackme")
|
|
|
|
self.user2 = self.register_user("teresa", "hackme")
|
|
self.tok2 = self.login("teresa", "hackme")
|
|
|
|
self.admin = self.register_user("admin", "pass", True)
|
|
self.admin_tok = self.login("admin", "pass")
|
|
|
|
self.room1 = self.helper.create_room_as(
|
|
room_creator=self.user1, tok=self.tok1, room_version="11"
|
|
)
|
|
self.store = hs.get_datastores().main
|
|
|
|
self.room2 = self.helper.create_room_as(
|
|
room_creator=self.user1, is_public=False, tok=self.tok1
|
|
)
|
|
self.helper.send_state(
|
|
self.room2,
|
|
EventTypes.RoomEncryption,
|
|
{EventContentFields.ENCRYPTION_ALGORITHM: "m.megolm.v1.aes-sha2"},
|
|
tok=self.tok1,
|
|
)
|
|
|
|
def test_suspended_user_cannot_send_message_to_public_room(self) -> None:
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/rooms/{self.room1}/send/m.room.message/1",
|
|
access_token=self.tok1,
|
|
content={"body": "hello", "msgtype": "m.text"},
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_cannot_send_message_to_encrypted_room(self) -> None:
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_synapse/admin/v1/suspend/{self.user1}",
|
|
{"suspend": True},
|
|
access_token=self.admin_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self.assertEqual(channel.json_body, {f"user_{self.user1}_suspended": True})
|
|
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/rooms/{self.room2}/send/m.room.encrypted/1",
|
|
access_token=self.tok1,
|
|
content={},
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_cannot_change_profile_data(self) -> None:
|
|
# set the user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/profile/{self.user1}/avatar_url",
|
|
access_token=self.tok1,
|
|
content={"avatar_url": "mxc://matrix.org/wefh34uihSDRGhw34"},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
channel2 = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/profile/{self.user1}/displayname",
|
|
access_token=self.tok1,
|
|
content={"displayname": "something offensive"},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel2.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
def test_suspended_user_cannot_redact_messages_other_than_their_own(self) -> None:
|
|
# first user sends message
|
|
self.make_request("POST", f"/rooms/{self.room1}/join", access_token=self.tok2)
|
|
res = self.helper.send_event(
|
|
self.room1,
|
|
"m.room.message",
|
|
{"body": "hello", "msgtype": "m.text"},
|
|
tok=self.tok2,
|
|
)
|
|
event_id = res["event_id"]
|
|
|
|
# second user sends message
|
|
self.make_request("POST", f"/rooms/{self.room1}/join", access_token=self.tok1)
|
|
res2 = self.helper.send_event(
|
|
self.room1,
|
|
"m.room.message",
|
|
{"body": "bad_message", "msgtype": "m.text"},
|
|
tok=self.tok1,
|
|
)
|
|
event_id2 = res2["event_id"]
|
|
|
|
# set the second user as suspended
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
# second user can't redact first user's message
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/redact/{event_id}/1",
|
|
access_token=self.tok1,
|
|
content={"reason": "bogus"},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
# but can redact their own
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/redact/{event_id2}/1",
|
|
access_token=self.tok1,
|
|
content={"reason": "bogus"},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/send/m.room.redaction/3456346",
|
|
access_token=self.tok1,
|
|
content={"reason": "bogus", "redacts": event_id},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
channel = self.make_request(
|
|
"PUT",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/send/m.room.redaction/3456346",
|
|
access_token=self.tok1,
|
|
content={"reason": "bogus", "redacts": event_id2},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
def test_suspended_user_cannot_ban_others(self) -> None:
|
|
# user to ban joins room user1 created
|
|
self.make_request("POST", f"/rooms/{self.room1}/join", access_token=self.tok2)
|
|
|
|
# suspend user1
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, True))
|
|
|
|
# user1 tries to ban other user while suspended
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/ban",
|
|
access_token=self.tok1,
|
|
content={"reason": "spite", "user_id": self.user2},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.json_body["errcode"], "M_USER_SUSPENDED")
|
|
|
|
# un-suspend user1
|
|
self.get_success(self.store.set_user_suspended_status(self.user1, False))
|
|
|
|
# ban now goes through
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/v3/rooms/{self.room1}/ban",
|
|
access_token=self.tok1,
|
|
content={"reason": "spite", "user_id": self.user2},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
|
|
class RoomParticipantTestCase(unittest.HomeserverTestCase):
|
|
servlets = [
|
|
login.register_servlets,
|
|
room.register_servlets,
|
|
profile.register_servlets,
|
|
admin.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user1 = self.register_user("thomas", "hackme")
|
|
self.tok1 = self.login("thomas", "hackme")
|
|
|
|
self.user2 = self.register_user("teresa", "hackme")
|
|
self.tok2 = self.login("teresa", "hackme")
|
|
|
|
self.room1 = self.helper.create_room_as(
|
|
room_creator=self.user1,
|
|
tok=self.tok1,
|
|
# Allow user2 to send state events into the room.
|
|
extra_content={
|
|
"power_level_content_override": {
|
|
"state_default": 0,
|
|
},
|
|
},
|
|
)
|
|
self.store = hs.get_datastores().main
|
|
|
|
@parameterized.expand(
|
|
[
|
|
# Should record participation.
|
|
param(
|
|
is_state=False,
|
|
event_type="m.room.message",
|
|
event_content={
|
|
"msgtype": "m.text",
|
|
"body": "I am engaging in this room",
|
|
},
|
|
record_participation=True,
|
|
),
|
|
param(
|
|
is_state=False,
|
|
event_type="m.room.encrypted",
|
|
event_content={
|
|
"algorithm": "m.megolm.v1.aes-sha2",
|
|
"ciphertext": "AwgAEnACgAkLmt6qF84IK++J7UDH2Za1YVchHyprqTqsg...",
|
|
"device_id": "RJYKSTBOIE",
|
|
"sender_key": "IlRMeOPX2e0MurIyfWEucYBRVOEEUMrOHqn/8mLqMjA",
|
|
"session_id": "X3lUlvLELLYxeTx4yOVu6UDpasGEVO0Jbu+QFnm0cKQ",
|
|
},
|
|
record_participation=True,
|
|
),
|
|
# Should not record participation.
|
|
param(
|
|
is_state=False,
|
|
event_type="m.sticker",
|
|
event_content={
|
|
"body": "My great sticker",
|
|
"info": {},
|
|
"url": "mxc://unused/mxcurl",
|
|
},
|
|
record_participation=False,
|
|
),
|
|
# An invalid **state event** with type `m.room.message`
|
|
param(
|
|
is_state=True,
|
|
event_type="m.room.message",
|
|
event_content={
|
|
"msgtype": "m.text",
|
|
"body": "I am engaging in this room",
|
|
},
|
|
record_participation=False,
|
|
),
|
|
# An invalid **state event** with type `m.room.encrypted`
|
|
# Note: this may become valid in the future with encrypted state, though we
|
|
# still may not want to consider it grounds for marking a user as participating.
|
|
param(
|
|
is_state=True,
|
|
event_type="m.room.encrypted",
|
|
event_content={
|
|
"algorithm": "m.megolm.v1.aes-sha2",
|
|
"ciphertext": "AwgAEnACgAkLmt6qF84IK++J7UDH2Za1YVchHyprqTqsg...",
|
|
"device_id": "RJYKSTBOIE",
|
|
"sender_key": "IlRMeOPX2e0MurIyfWEucYBRVOEEUMrOHqn/8mLqMjA",
|
|
"session_id": "X3lUlvLELLYxeTx4yOVu6UDpasGEVO0Jbu+QFnm0cKQ",
|
|
},
|
|
record_participation=False,
|
|
),
|
|
]
|
|
)
|
|
def test_sending_message_records_participation(
|
|
self,
|
|
is_state: bool,
|
|
event_type: str,
|
|
event_content: JsonDict,
|
|
record_participation: bool,
|
|
) -> None:
|
|
"""
|
|
Test that sending an various events into a room causes the user to
|
|
appropriately marked or not marked as a participant in that room.
|
|
"""
|
|
self.helper.join(self.room1, self.user2, tok=self.tok2)
|
|
|
|
# user has not sent any messages, so should not be a participant
|
|
participant = self.get_success(
|
|
self.store.get_room_participation(self.user2, self.room1)
|
|
)
|
|
self.assertFalse(participant)
|
|
|
|
# send an event into the room
|
|
if is_state:
|
|
# send a state event
|
|
self.helper.send_state(
|
|
self.room1,
|
|
event_type,
|
|
body=event_content,
|
|
tok=self.tok2,
|
|
)
|
|
else:
|
|
# send a non-state event
|
|
self.helper.send_event(
|
|
self.room1,
|
|
event_type,
|
|
content=event_content,
|
|
tok=self.tok2,
|
|
)
|
|
|
|
# check whether the user has been marked as a participant
|
|
participant = self.get_success(
|
|
self.store.get_room_participation(self.user2, self.room1)
|
|
)
|
|
self.assertEqual(participant, record_participation)
|
|
|
|
@parameterized.expand(
|
|
[
|
|
param(
|
|
event_type="m.room.message",
|
|
event_content={
|
|
"msgtype": "m.text",
|
|
"body": "I am engaging in this room",
|
|
},
|
|
),
|
|
param(
|
|
event_type="m.room.encrypted",
|
|
event_content={
|
|
"algorithm": "m.megolm.v1.aes-sha2",
|
|
"ciphertext": "AwgAEnACgAkLmt6qF84IK++J7UDH2Za1YVchHyprqTqsg...",
|
|
"device_id": "RJYKSTBOIE",
|
|
"sender_key": "IlRMeOPX2e0MurIyfWEucYBRVOEEUMrOHqn/8mLqMjA",
|
|
"session_id": "X3lUlvLELLYxeTx4yOVu6UDpasGEVO0Jbu+QFnm0cKQ",
|
|
},
|
|
),
|
|
]
|
|
)
|
|
def test_sending_event_and_leaving_does_not_record_participation(
|
|
self,
|
|
event_type: str,
|
|
event_content: JsonDict,
|
|
) -> None:
|
|
"""
|
|
Test that sending an event into a room that should mark a user as a
|
|
participant, but then leaving the room, results in the user no longer
|
|
be marked as a participant in that room.
|
|
"""
|
|
self.helper.join(self.room1, self.user2, tok=self.tok2)
|
|
|
|
# user has not sent any messages, so should not be a participant
|
|
participant = self.get_success(
|
|
self.store.get_room_participation(self.user2, self.room1)
|
|
)
|
|
self.assertFalse(participant)
|
|
|
|
# sending a message should now mark user as participant
|
|
self.helper.send_event(
|
|
self.room1,
|
|
event_type,
|
|
content=event_content,
|
|
tok=self.tok2,
|
|
)
|
|
participant = self.get_success(
|
|
self.store.get_room_participation(self.user2, self.room1)
|
|
)
|
|
self.assertTrue(participant)
|
|
|
|
# leave the room
|
|
self.helper.leave(self.room1, self.user2, tok=self.tok2)
|
|
|
|
# user should no longer be considered a participant
|
|
participant = self.get_success(
|
|
self.store.get_room_participation(self.user2, self.room1)
|
|
)
|
|
self.assertFalse(participant)
|
|
|
|
|
|
class MSC4293RedactOnBanKickTestCase(unittest.FederatingHomeserverTestCase):
|
|
servlets = [
|
|
synapse.rest.admin.register_servlets_for_client_rest_resource,
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
admin.register_servlets,
|
|
]
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
super().prepare(reactor, clock, hs)
|
|
self.creator = self.register_user("creator", "test")
|
|
self.creator_tok = self.login("creator", "test")
|
|
|
|
self.good_admin = self.register_user("good_admin", "test")
|
|
self.good_admin_tok = self.login("good_admin", "test")
|
|
|
|
self.bad_user_id = self.register_user("bad", "test")
|
|
self.bad_tok = self.login("bad", "test")
|
|
|
|
self.room_id = self.helper.create_room_as(self.creator, tok=self.creator_tok)
|
|
|
|
self.store = hs.get_datastores().main
|
|
self._storage_controllers = hs.get_storage_controllers()
|
|
|
|
self.federation_event_handler = self.hs.get_federation_event_handler()
|
|
|
|
self.hs.config.experimental.msc4293_enabled = True
|
|
|
|
self.room_version = self.hs.config.server.default_room_version
|
|
|
|
def _check_redactions(
|
|
self,
|
|
original_events: list[EventBase],
|
|
pulled_events: list[JsonDict],
|
|
expect_redaction: bool,
|
|
reason: str | None = None,
|
|
) -> None:
|
|
"""
|
|
Checks a set of original events against a second set of the same events, pulled
|
|
from the /messages api. If expect_redaction is true, we expect that the second
|
|
set of events will be redacted, and the test will fail if that is not the case.
|
|
Otherwise, verifies that the events have not been redacted and fails if not.
|
|
|
|
Args:
|
|
original_events: A list of the original events sent
|
|
pulled_events: A list of the same events as the orignal events, fetched
|
|
over the /messages api
|
|
expect_redaction: Whether or not the pulled_events should be redacted
|
|
reason: If the events are expected to be redacted, the expected reason
|
|
for the redaction
|
|
|
|
"""
|
|
if expect_redaction:
|
|
redacted_count = 0
|
|
for pulled_event in pulled_events:
|
|
for old_event in original_events:
|
|
if pulled_event["event_id"] != old_event.event_id:
|
|
continue
|
|
# we have a matching event, check that it is redacted
|
|
event_content = pulled_event["content"]
|
|
if event_content:
|
|
self.fail(f"Expected event {pulled_event} to be redacted")
|
|
redacting_event = pulled_event.get("redacted_because")
|
|
if not redacting_event:
|
|
self.fail(
|
|
f"Expected event {pulled_event} to have a redacting event."
|
|
)
|
|
# check that the redacting event records the expected reason, and the
|
|
# redact_events flag
|
|
content = redacting_event["content"]
|
|
self.assertEqual(content["reason"], reason)
|
|
self.assertEqual(content["org.matrix.msc4293.redact_events"], True)
|
|
redacted_count += 1
|
|
# all provided events should be redacted
|
|
self.assertEqual(len(original_events), redacted_count)
|
|
else:
|
|
unredacted_events = 0
|
|
for pulled_event in pulled_events:
|
|
for old_event in original_events:
|
|
if pulled_event["event_id"] != old_event.event_id:
|
|
continue
|
|
# we have a matching event, make sure it is not redacted
|
|
redacted_because = pulled_event.get("redacted_because")
|
|
if redacted_because:
|
|
self.fail("Event should not have been redacted")
|
|
self.assertEqual(old_event.content, pulled_event["content"])
|
|
unredacted_events += 1
|
|
# all provided events should not have been redacted
|
|
self.assertEqual(unredacted_events, len(original_events))
|
|
|
|
def test_banning_local_member_with_flag_redacts_their_events(self) -> None:
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
originals = []
|
|
for i in range(5):
|
|
event = {"body": f"bothersome noise {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
originals.append(res["event_id"])
|
|
|
|
# grab original events for comparison
|
|
original_events = [self.get_success(self.store.get_event(x)) for x in originals]
|
|
|
|
# creator bans user with redaction flag set
|
|
content = {
|
|
"reason": "flooding",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.creator,
|
|
self.bad_user_id,
|
|
"ban",
|
|
content,
|
|
self.creator_tok,
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_events,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="flooding",
|
|
)
|
|
|
|
def test_banning_remote_member_with_flag_redacts_their_events(self) -> None:
|
|
bad_user = "@remote_bad_user:" + self.OTHER_SERVER_NAME
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member
|
|
r = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(r[("m.room.member", bad_user)].membership, "join")
|
|
|
|
auth_ids = [
|
|
r[("m.room.power_levels", "")].event_id,
|
|
r[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"remote bummer{i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
original_messages.append(remote_message)
|
|
|
|
# creator bans bad user with redaction flag set
|
|
content = {
|
|
"reason": "bummer messages",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
res = self.helper.change_membership(
|
|
self.room_id, self.creator, bad_user, "ban", content, self.creator_tok
|
|
)
|
|
ban_event_id = res["event_id"]
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_messages,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="bummer messages",
|
|
)
|
|
|
|
# any future messages that are soft-failed are also redacted - send messages referencing
|
|
# dag before ban, they should be soft-failed but also redacted
|
|
new_original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"soft-fail remote bummer{i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
new_original_messages.append(remote_message)
|
|
|
|
# pull them from the db to check because they should be soft-failed and thus not available over
|
|
# cs-api
|
|
for message in new_original_messages:
|
|
original = self.get_success(self.store.get_event(message.event_id))
|
|
if not original:
|
|
self.fail("Expected to find remote message in DB")
|
|
redacted_by = original.internal_metadata.redacted_by
|
|
if not redacted_by:
|
|
self.fail("Did not find redacted_by field")
|
|
self.assertEqual(redacted_by, ban_event_id)
|
|
|
|
def test_unbanning_remote_user_stops_redaction_action(self) -> None:
|
|
bad_user = "@remote_bad_user:" + self.OTHER_SERVER_NAME
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member
|
|
r = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(r[("m.room.member", bad_user)].membership, "join")
|
|
|
|
auth_ids = [
|
|
r[("m.room.power_levels", "")].event_id,
|
|
r[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"annoying messages {i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
original_messages.append(remote_message)
|
|
|
|
# creator bans bad user with redaction flag set
|
|
content = {
|
|
"reason": "this dude sucks",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id, self.creator, bad_user, "ban", content, self.creator_tok
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_messages,
|
|
channel.json_body["chunk"],
|
|
True,
|
|
reason="this dude sucks",
|
|
)
|
|
|
|
# unban user
|
|
self.helper.change_membership(
|
|
self.room_id, self.creator, bad_user, "unban", {}, self.creator_tok
|
|
)
|
|
|
|
# user should be able to join again
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member again
|
|
new_state = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(new_state[("m.room.member", bad_user)].membership, "join")
|
|
|
|
new_state = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
auth_ids = [
|
|
new_state[("m.room.power_levels", "")].event_id,
|
|
new_state[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
|
|
# messages after unban and join proceed unredacted
|
|
new_original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"no longer a bummer {i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
new_original_messages.append(remote_message)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(new_original_messages, channel.json_body["chunk"], False)
|
|
|
|
def test_redaction_flag_ignored_for_user_if_banner_lacks_redaction_power(
|
|
self,
|
|
) -> None:
|
|
# change power levels so room admin can ban but not redact. Do not use the
|
|
# room's creator, as the power level they hold is not consistently definable
|
|
# after msc4289(creator has infinite power level).
|
|
self.helper.send_state(
|
|
self.room_id,
|
|
"m.room.power_levels",
|
|
{"events_default": 0, "redact": 100, "users": {self.good_admin: 75}},
|
|
tok=self.creator_tok,
|
|
)
|
|
self.helper.join(self.room_id, self.good_admin, tok=self.good_admin_tok)
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
original_ids = []
|
|
for i in range(15):
|
|
event = {"body": f"being a menace {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
original_ids.append(res["event_id"])
|
|
|
|
# grab original events before ban
|
|
originals = [self.get_success(self.store.get_event(x)) for x in original_ids]
|
|
|
|
# creator bans bad user with redaction flag
|
|
content = {
|
|
"reason": "flooding",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.good_admin,
|
|
self.bad_user_id,
|
|
"ban",
|
|
content,
|
|
self.good_admin_tok,
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
# messages are not redacted
|
|
self._check_redactions(originals, channel.json_body["chunk"], False)
|
|
|
|
def test_kicking_local_member_with_flag_redacts_their_events(self) -> None:
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
originals = []
|
|
for i in range(5):
|
|
event = {"body": f"bothersome noise {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
originals.append(res["event_id"])
|
|
|
|
# grab original events for comparison
|
|
original_events = [self.get_success(self.store.get_event(x)) for x in originals]
|
|
|
|
# creator kicks user with redaction flag set
|
|
content = {
|
|
"reason": "flooding",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.creator,
|
|
self.bad_user_id,
|
|
"kick",
|
|
content,
|
|
self.creator_tok,
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_events,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="flooding",
|
|
)
|
|
|
|
def test_kicking_remote_member_with_flag_redacts_their_events(self) -> None:
|
|
bad_user = "@remote_bad_user:" + self.OTHER_SERVER_NAME
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member
|
|
r = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(r[("m.room.member", bad_user)].membership, "join")
|
|
|
|
auth_ids = [
|
|
r[("m.room.power_levels", "")].event_id,
|
|
r[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"remote bummer{i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
original_messages.append(remote_message)
|
|
|
|
# creator kicks bad user with redaction flag set
|
|
content = {
|
|
"reason": "bummer messages",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
res = self.helper.change_membership(
|
|
self.room_id, self.creator, bad_user, "kick", content, self.creator_tok
|
|
)
|
|
ban_event_id = res["event_id"]
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_messages,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="bummer messages",
|
|
)
|
|
|
|
# any future messages that are soft-failed are also redacted - send messages referencing
|
|
# dag before ban, they should be soft-failed but also redacted
|
|
new_original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"soft-fail remote bummer{i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
new_original_messages.append(remote_message)
|
|
|
|
# pull them from the db to check because they should be soft-failed and thus not available over
|
|
# cs-api
|
|
for message in new_original_messages:
|
|
original = self.get_success(self.store.get_event(message.event_id))
|
|
if not original:
|
|
self.fail("Expected to find remote message in DB")
|
|
self.assertEqual(original.internal_metadata.redacted_by, ban_event_id)
|
|
|
|
def test_rejoining_kicked_remote_user_stops_redaction_action(self) -> None:
|
|
bad_user = "@remote_bad_user:" + self.OTHER_SERVER_NAME
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member
|
|
r = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(r[("m.room.member", bad_user)].membership, "join")
|
|
|
|
auth_ids = [
|
|
r[("m.room.power_levels", "")].event_id,
|
|
r[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"annoying messages {i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
original_messages.append(remote_message)
|
|
|
|
# creator kicks bad user with redaction flag set
|
|
content = {
|
|
"reason": "this dude sucks",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id, self.creator, bad_user, "kick", content, self.creator_tok
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
original_messages,
|
|
channel.json_body["chunk"],
|
|
True,
|
|
reason="this dude sucks",
|
|
)
|
|
|
|
# user re-joins after kick
|
|
channel = self.make_signed_federation_request(
|
|
"GET",
|
|
f"/_matrix/federation/v1/make_join/{self.room_id}/{bad_user}?ver={self.room_version.identifier}",
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
join_result = channel.json_body
|
|
|
|
join_event_dict = join_result["event"]
|
|
self.add_hashes_and_signatures_from_other_server(join_event_dict)
|
|
channel = self.make_signed_federation_request(
|
|
"PUT",
|
|
f"/_matrix/federation/v2/send_join/{self.room_id}/x",
|
|
content=join_event_dict,
|
|
)
|
|
self.assertEqual(channel.code, HTTPStatus.OK, channel.json_body)
|
|
|
|
# the room should show that the bad user is a member again
|
|
new_state = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
self.assertEqual(new_state[("m.room.member", bad_user)].membership, "join")
|
|
|
|
new_state = self.get_success(
|
|
self._storage_controllers.state.get_current_state(self.room_id)
|
|
)
|
|
auth_ids = [
|
|
new_state[("m.room.power_levels", "")].event_id,
|
|
new_state[("m.room.member", "@remote_bad_user:other.example.com")].event_id,
|
|
]
|
|
|
|
# messages after kick and re-join proceed unredacted
|
|
new_original_messages = []
|
|
for i in range(5):
|
|
remote_message = make_event_from_dict(
|
|
self.add_hashes_and_signatures_from_other_server(
|
|
{
|
|
"room_id": self.room_id,
|
|
"sender": bad_user,
|
|
"depth": 1000,
|
|
"origin_server_ts": 1,
|
|
"type": "m.room.message",
|
|
"content": {"body": f"no longer a bummer {i}"},
|
|
"auth_events": auth_ids,
|
|
"prev_events": auth_ids,
|
|
}
|
|
),
|
|
room_version=self.room_version,
|
|
)
|
|
|
|
self.get_success(
|
|
self.federation_event_handler.on_receive_pdu(
|
|
self.OTHER_SERVER_NAME, remote_message
|
|
)
|
|
)
|
|
new_original_messages.append(remote_message)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(new_original_messages, channel.json_body["chunk"], False)
|
|
|
|
def test_redaction_flag_ignored_for_user_if_kicker_lacks_redaction_power(
|
|
self,
|
|
) -> None:
|
|
# change power levels so room admin can kick but not redact. Do not use the
|
|
# room's creator, as the power level they hold is not consistently definable
|
|
# after msc4289(creator has infinite power level).
|
|
self.helper.send_state(
|
|
self.room_id,
|
|
"m.room.power_levels",
|
|
{"events_default": 0, "redact": 100, "users": {self.good_admin: 75}},
|
|
tok=self.creator_tok,
|
|
)
|
|
self.helper.join(self.room_id, self.good_admin, tok=self.good_admin_tok)
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
original_ids = []
|
|
for i in range(15):
|
|
event = {"body": f"being a menace {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
original_ids.append(res["event_id"])
|
|
|
|
# grab original events before ban
|
|
originals = [self.get_success(self.store.get_event(x)) for x in original_ids]
|
|
|
|
# creator kicks bad user with redaction flag
|
|
content = {
|
|
"reason": "flooding",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.good_admin,
|
|
self.bad_user_id,
|
|
"kick",
|
|
content,
|
|
self.good_admin_tok,
|
|
)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
# messages are not redacted
|
|
self._check_redactions(originals, channel.json_body["chunk"], False)
|
|
|
|
def test_MSC4293_flag_ignored_in_other_membership_events(self) -> None:
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
original_ids = []
|
|
for i in range(15):
|
|
event = {"body": f"being a menace {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
original_ids.append(res["event_id"])
|
|
|
|
# grab original events before ban
|
|
originals = [self.get_success(self.store.get_event(x)) for x in original_ids]
|
|
|
|
# bad user leaves on their own with flag
|
|
content = {
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.bad_user_id,
|
|
self.bad_user_id,
|
|
"leave",
|
|
content,
|
|
self.bad_tok,
|
|
)
|
|
|
|
# their messages are not redacted
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(originals, channel.json_body["chunk"], False)
|
|
|
|
# bad user is invited with flag in invite event
|
|
content = {
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.creator,
|
|
self.bad_user_id,
|
|
"invite",
|
|
content,
|
|
self.creator_tok,
|
|
)
|
|
|
|
# their messages are still not redacted
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(originals, channel.json_body["chunk"], False)
|
|
|
|
# bad user joins with flag in invite event
|
|
content = {
|
|
"org.matrix.msc4293.redact_events": True,
|
|
}
|
|
self.helper.change_membership(
|
|
self.room_id,
|
|
self.bad_user_id,
|
|
self.bad_user_id,
|
|
"join",
|
|
content,
|
|
self.bad_tok,
|
|
)
|
|
|
|
# and still their messages are not redacted
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(originals, channel.json_body["chunk"], False)
|
|
|
|
def test_MSC4293_redaction_applied_via_kick_api(self) -> None:
|
|
"""
|
|
Test that MSC4239 field passed through and applied when using /kick
|
|
"""
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
original_ids = []
|
|
for i in range(15):
|
|
event = {"body": f"being a menace {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
original_ids.append(res["event_id"])
|
|
|
|
# grab original events before kick
|
|
originals = [self.get_success(self.store.get_event(x)) for x in original_ids]
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/v3/rooms/{self.room_id}/kick",
|
|
access_token=self.creator_tok,
|
|
content={
|
|
"reason": "being annoying",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
"user_id": self.bad_user_id,
|
|
},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
originals,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="being annoying",
|
|
)
|
|
|
|
def test_MSC4293_redaction_applied_via_ban_api(self) -> None:
|
|
"""
|
|
Test that MSC4239 field passed through and applied when using /ban
|
|
"""
|
|
self.helper.join(self.room_id, self.bad_user_id, tok=self.bad_tok)
|
|
|
|
# bad user sends some messages
|
|
original_ids = []
|
|
for i in range(15):
|
|
event = {"body": f"being a menace {i}", "msgtype": "m.text"}
|
|
res = self.helper.send_event(
|
|
self.room_id, "m.room.message", event, tok=self.bad_tok, expect_code=200
|
|
)
|
|
original_ids.append(res["event_id"])
|
|
|
|
# grab original events before ban
|
|
originals = [self.get_success(self.store.get_event(x)) for x in original_ids]
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
f"/_matrix/client/v3/rooms/{self.room_id}/ban",
|
|
access_token=self.creator_tok,
|
|
content={
|
|
"reason": "being disruptive",
|
|
"org.matrix.msc4293.redact_events": True,
|
|
"user_id": self.bad_user_id,
|
|
},
|
|
shorthand=False,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
|
|
filter = json.dumps({"types": [EventTypes.Message]})
|
|
channel = self.make_request(
|
|
"GET",
|
|
f"rooms/{self.room_id}/messages?filter={filter}&limit=50",
|
|
access_token=self.creator_tok,
|
|
)
|
|
self.assertEqual(channel.code, 200)
|
|
self._check_redactions(
|
|
originals,
|
|
channel.json_body["chunk"],
|
|
expect_redaction=True,
|
|
reason="being disruptive",
|
|
)
|
|
|
|
|
|
class CreateRoomRemoteInviteTestCase(unittest.FederatingHomeserverTestCase):
|
|
"""
|
|
Tests error propagation from remote invites during /createRoom.
|
|
|
|
Regression test for https://github.com/element-hq/synapse/security/advisories/GHSA-95fh-hv8c-chvq.
|
|
"""
|
|
|
|
servlets = [
|
|
room.register_servlets,
|
|
login.register_servlets,
|
|
register.register_servlets,
|
|
admin.register_servlets,
|
|
]
|
|
|
|
hijack_auth = False
|
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
|
self.user_id = self.register_user("creator", "test")
|
|
self.token = self.login("creator", "test")
|
|
|
|
def _mock_remote_invite_http_error(
|
|
self,
|
|
status: int,
|
|
error_body: JsonDict,
|
|
) -> None:
|
|
"""
|
|
Make the remote homeserver reply to its `/invite` endpoint with an error.
|
|
|
|
Args:
|
|
status: the HTTP status to return
|
|
error_body: the JSON error body to return
|
|
"""
|
|
federation_http_client = self.hs.get_federation_http_client()
|
|
|
|
fake_agent = create_autospec(Agent, spec_set=True)
|
|
|
|
def request(
|
|
method: bytes,
|
|
uri: bytes,
|
|
headers: object = None,
|
|
bodyProducer: object = None,
|
|
) -> "defer.Deferred":
|
|
# For our test, we don't expect any other outbound request
|
|
assert b"/invite/" in uri, f"unexpected outbound request to {uri!r}"
|
|
return defer.succeed(
|
|
FakeResponse.json(
|
|
code=status,
|
|
payload=error_body,
|
|
)
|
|
)
|
|
|
|
fake_agent.request.side_effect = request
|
|
federation_http_client.agent = fake_agent
|
|
|
|
@parameterized.expand(
|
|
(
|
|
(
|
|
HTTPStatus.IM_A_TEAPOT,
|
|
{
|
|
"errcode": "M_FORBIDDEN",
|
|
"error": "You can't invite this user",
|
|
},
|
|
HTTPStatus.IM_A_TEAPOT,
|
|
{
|
|
"errcode": "M_FORBIDDEN",
|
|
"error": "You can't invite this user",
|
|
},
|
|
),
|
|
# This case is https://github.com/element-hq/synapse/security/advisories/GHSA-95fh-hv8c-chvq
|
|
# The error is rewritten for safety.
|
|
(
|
|
HTTPStatus.UNAUTHORIZED,
|
|
{"errcode": "M_UNKNOWN_TOKEN", "error": "unknown token"},
|
|
HTTPStatus.BAD_REQUEST,
|
|
{
|
|
"errcode": "M_UNKNOWN",
|
|
"error": "unknown token",
|
|
},
|
|
),
|
|
)
|
|
)
|
|
def test_remote_invite_bubbles_errors(
|
|
self,
|
|
policy_server_error_status: HTTPStatus,
|
|
policy_server_error_body: JsonDict,
|
|
expected_client_facing_error_status: HTTPStatus,
|
|
expected_client_facing_error_body: JsonDict,
|
|
) -> None:
|
|
"""
|
|
Test that, when creating a room involving a remote invite,
|
|
when the remote homeserver returns an error, we bubble it
|
|
to the client carefully.
|
|
|
|
Regression test for https://github.com/element-hq/synapse/security/advisories/GHSA-95fh-hv8c-chvq
|
|
"""
|
|
# Mock the remote homeserver (at the HTTP level) to return the configured error
|
|
self._mock_remote_invite_http_error(
|
|
policy_server_error_status,
|
|
policy_server_error_body,
|
|
)
|
|
|
|
channel = self.make_request(
|
|
"POST",
|
|
"/createRoom",
|
|
{"invite": ["@alice:" + self.OTHER_SERVER_NAME]},
|
|
access_token=self.token,
|
|
)
|
|
|
|
self.assertEqual(
|
|
channel.code, expected_client_facing_error_status, channel.result
|
|
)
|
|
self.assertEqual(channel.json_body, expected_client_facing_error_body)
|