2016-04-20 10:21:40 -04:00
|
|
|
# Copyright 2015, 2016 OpenMarket Ltd
|
|
|
|
#
|
|
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
|
|
# you may not use this file except in compliance with the License.
|
|
|
|
# You may obtain a copy of the License at
|
|
|
|
#
|
|
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
#
|
|
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
|
|
# See the License for the specific language governing permissions and
|
|
|
|
# limitations under the License.
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
from typing import Any, Collection, List, Optional, Tuple
|
2021-04-09 13:44:38 -04:00
|
|
|
from unittest.mock import Mock
|
2018-07-09 02:09:20 -04:00
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
from twisted.test.proto_helpers import MemoryReactor
|
|
|
|
|
2022-06-17 08:48:55 -04:00
|
|
|
from synapse.api.auth.internal import InternalAuth
|
2018-12-14 13:20:59 -05:00
|
|
|
from synapse.api.constants import UserTypes
|
2021-09-30 14:06:02 -04:00
|
|
|
from synapse.api.errors import (
|
|
|
|
CodeMessageException,
|
|
|
|
Codes,
|
|
|
|
ResourceLimitError,
|
|
|
|
SynapseError,
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
from synapse.module_api import ModuleApi
|
|
|
|
from synapse.server import HomeServer
|
2020-08-20 15:42:58 -04:00
|
|
|
from synapse.spam_checker_api import RegistrationBehaviour
|
2022-12-16 06:53:01 -05:00
|
|
|
from synapse.types import (
|
|
|
|
JsonDict,
|
|
|
|
Requester,
|
|
|
|
RoomAlias,
|
|
|
|
RoomID,
|
|
|
|
UserID,
|
|
|
|
create_requester,
|
|
|
|
)
|
|
|
|
from synapse.util import Clock
|
2016-04-20 10:21:40 -04:00
|
|
|
|
2020-08-11 17:21:13 -04:00
|
|
|
from tests.test_utils import make_awaitable
|
2020-06-30 15:41:36 -04:00
|
|
|
from tests.unittest import override_config
|
2020-08-20 15:42:58 -04:00
|
|
|
from tests.utils import mock_getRawHeaders
|
2020-06-30 15:41:36 -04:00
|
|
|
|
2018-07-09 02:09:20 -04:00
|
|
|
from .. import unittest
|
2016-04-20 10:21:40 -04:00
|
|
|
|
|
|
|
|
2021-06-18 07:15:52 -04:00
|
|
|
class TestSpamChecker:
|
2022-12-16 06:53:01 -05:00
|
|
|
def __init__(self, config: None, api: ModuleApi):
|
2021-06-18 07:15:52 -04:00
|
|
|
api.register_spam_checker_callbacks(
|
|
|
|
check_registration_for_spam=self.check_registration_for_spam,
|
|
|
|
)
|
|
|
|
|
|
|
|
@staticmethod
|
2022-12-16 06:53:01 -05:00
|
|
|
def parse_config(config: JsonDict) -> None:
|
|
|
|
return None
|
2021-06-18 07:15:52 -04:00
|
|
|
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
auth_provider_id: Optional[str],
|
|
|
|
) -> RegistrationBehaviour:
|
Use mypy 1.0 (#15052)
* Update mypy and mypy-zope
* Remove unused ignores
These used to suppress
```
synapse/storage/engines/__init__.py:28: error: "__new__" must return a
class instance (got "NoReturn") [misc]
```
and
```
synapse/http/matrixfederationclient.py:1270: error: "BaseException" has no attribute "reasons" [attr-defined]
```
(note that we check `hasattr(e, "reasons")` above)
* Avoid empty body warnings, sometimes by marking methods as abstract
E.g.
```
tests/handlers/test_register.py:58: error: Missing return statement [empty-body]
tests/handlers/test_register.py:108: error: Missing return statement [empty-body]
```
* Suppress false positive about `JaegerConfig`
Complaint was
```
synapse/logging/opentracing.py:450: error: Function "Type[Config]" could always be true in boolean context [truthy-function]
```
* Fix not calling `is_state()`
Oops!
```
tests/rest/client/test_third_party_rules.py:428: error: Function "Callable[[], bool]" could always be true in boolean context [truthy-function]
```
* Suppress false positives from ParamSpecs
````
synapse/logging/opentracing.py:971: error: Argument 2 to "_custom_sync_async_decorator" has incompatible type "Callable[[Arg(Callable[P, R], 'func'), **P], _GeneratorContextManager[None]]"; expected "Callable[[Callable[P, R], **P], _GeneratorContextManager[None]]" [arg-type]
synapse/logging/opentracing.py:1017: error: Argument 2 to "_custom_sync_async_decorator" has incompatible type "Callable[[Arg(Callable[P, R], 'func'), **P], _GeneratorContextManager[None]]"; expected "Callable[[Callable[P, R], **P], _GeneratorContextManager[None]]" [arg-type]
````
* Drive-by improvement to `wrapping_logic` annotation
* Workaround false "unreachable" positives
See https://github.com/Shoobx/mypy-zope/issues/91
```
tests/http/test_proxyagent.py:626: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:762: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:826: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:838: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:845: error: Statement is unreachable [unreachable]
tests/http/federation/test_matrix_federation_agent.py:151: error: Statement is unreachable [unreachable]
tests/http/federation/test_matrix_federation_agent.py:452: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:60: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:93: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:127: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:152: error: Statement is unreachable [unreachable]
```
* Changelog
* Tweak DBAPI2 Protocol to be accepted by mypy 1.0
Some extra context in:
- https://github.com/matrix-org/python-canonicaljson/pull/57
- https://github.com/python/mypy/issues/6002
- https://mypy.readthedocs.io/en/latest/common_issues.html#covariant-subtyping-of-mutable-protocol-members-is-rejected
* Pull in updated canonicaljson lib
so the protocol check just works
* Improve comments in opentracing
I tried to workaround the ignores but found it too much trouble.
I think the corresponding issue is
https://github.com/python/mypy/issues/12909. The mypy repo has a PR
claiming to fix this (https://github.com/python/mypy/pull/14677) which
might mean this gets resolved soon?
* Better annotation for INTERACTIVE_AUTH_CHECKERS
* Drive-by AUTH_TYPE annotation, to remove an ignore
2023-02-16 11:09:11 -05:00
|
|
|
return RegistrationBehaviour.ALLOW
|
2021-06-18 07:15:52 -04:00
|
|
|
|
|
|
|
|
|
|
|
class DenyAll(TestSpamChecker):
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
auth_provider_id: Optional[str],
|
|
|
|
) -> RegistrationBehaviour:
|
2021-06-18 07:15:52 -04:00
|
|
|
return RegistrationBehaviour.DENY
|
|
|
|
|
|
|
|
|
|
|
|
class BanAll(TestSpamChecker):
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
auth_provider_id: Optional[str],
|
|
|
|
) -> RegistrationBehaviour:
|
2021-06-18 07:15:52 -04:00
|
|
|
return RegistrationBehaviour.SHADOW_BAN
|
|
|
|
|
|
|
|
|
|
|
|
class BanBadIdPUser(TestSpamChecker):
|
|
|
|
async def check_registration_for_spam(
|
2022-12-16 06:53:01 -05:00
|
|
|
self,
|
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
auth_provider_id: Optional[str] = None,
|
|
|
|
) -> RegistrationBehaviour:
|
2021-06-18 07:15:52 -04:00
|
|
|
# Reject any user coming from CAS and whose username contains profanity
|
2022-12-16 06:53:01 -05:00
|
|
|
if auth_provider_id == "cas" and username and "flimflob" in username:
|
2021-06-18 07:15:52 -04:00
|
|
|
return RegistrationBehaviour.DENY
|
|
|
|
return RegistrationBehaviour.ALLOW
|
|
|
|
|
|
|
|
|
2021-06-23 11:22:08 -04:00
|
|
|
class TestLegacyRegistrationSpamChecker:
|
2022-12-16 06:53:01 -05:00
|
|
|
def __init__(self, config: None, api: ModuleApi):
|
2021-06-23 11:22:08 -04:00
|
|
|
pass
|
|
|
|
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
) -> RegistrationBehaviour:
|
Use mypy 1.0 (#15052)
* Update mypy and mypy-zope
* Remove unused ignores
These used to suppress
```
synapse/storage/engines/__init__.py:28: error: "__new__" must return a
class instance (got "NoReturn") [misc]
```
and
```
synapse/http/matrixfederationclient.py:1270: error: "BaseException" has no attribute "reasons" [attr-defined]
```
(note that we check `hasattr(e, "reasons")` above)
* Avoid empty body warnings, sometimes by marking methods as abstract
E.g.
```
tests/handlers/test_register.py:58: error: Missing return statement [empty-body]
tests/handlers/test_register.py:108: error: Missing return statement [empty-body]
```
* Suppress false positive about `JaegerConfig`
Complaint was
```
synapse/logging/opentracing.py:450: error: Function "Type[Config]" could always be true in boolean context [truthy-function]
```
* Fix not calling `is_state()`
Oops!
```
tests/rest/client/test_third_party_rules.py:428: error: Function "Callable[[], bool]" could always be true in boolean context [truthy-function]
```
* Suppress false positives from ParamSpecs
````
synapse/logging/opentracing.py:971: error: Argument 2 to "_custom_sync_async_decorator" has incompatible type "Callable[[Arg(Callable[P, R], 'func'), **P], _GeneratorContextManager[None]]"; expected "Callable[[Callable[P, R], **P], _GeneratorContextManager[None]]" [arg-type]
synapse/logging/opentracing.py:1017: error: Argument 2 to "_custom_sync_async_decorator" has incompatible type "Callable[[Arg(Callable[P, R], 'func'), **P], _GeneratorContextManager[None]]"; expected "Callable[[Callable[P, R], **P], _GeneratorContextManager[None]]" [arg-type]
````
* Drive-by improvement to `wrapping_logic` annotation
* Workaround false "unreachable" positives
See https://github.com/Shoobx/mypy-zope/issues/91
```
tests/http/test_proxyagent.py:626: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:762: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:826: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:838: error: Statement is unreachable [unreachable]
tests/http/test_proxyagent.py:845: error: Statement is unreachable [unreachable]
tests/http/federation/test_matrix_federation_agent.py:151: error: Statement is unreachable [unreachable]
tests/http/federation/test_matrix_federation_agent.py:452: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:60: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:93: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:127: error: Statement is unreachable [unreachable]
tests/logging/test_remote_handler.py:152: error: Statement is unreachable [unreachable]
```
* Changelog
* Tweak DBAPI2 Protocol to be accepted by mypy 1.0
Some extra context in:
- https://github.com/matrix-org/python-canonicaljson/pull/57
- https://github.com/python/mypy/issues/6002
- https://mypy.readthedocs.io/en/latest/common_issues.html#covariant-subtyping-of-mutable-protocol-members-is-rejected
* Pull in updated canonicaljson lib
so the protocol check just works
* Improve comments in opentracing
I tried to workaround the ignores but found it too much trouble.
I think the corresponding issue is
https://github.com/python/mypy/issues/12909. The mypy repo has a PR
claiming to fix this (https://github.com/python/mypy/pull/14677) which
might mean this gets resolved soon?
* Better annotation for INTERACTIVE_AUTH_CHECKERS
* Drive-by AUTH_TYPE annotation, to remove an ignore
2023-02-16 11:09:11 -05:00
|
|
|
return RegistrationBehaviour.ALLOW
|
2021-06-23 11:22:08 -04:00
|
|
|
|
|
|
|
|
|
|
|
class LegacyAllowAll(TestLegacyRegistrationSpamChecker):
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
) -> RegistrationBehaviour:
|
2021-06-23 11:22:08 -04:00
|
|
|
return RegistrationBehaviour.ALLOW
|
|
|
|
|
|
|
|
|
|
|
|
class LegacyDenyAll(TestLegacyRegistrationSpamChecker):
|
|
|
|
async def check_registration_for_spam(
|
|
|
|
self,
|
2022-12-16 06:53:01 -05:00
|
|
|
email_threepid: Optional[dict],
|
|
|
|
username: Optional[str],
|
|
|
|
request_info: Collection[Tuple[str, str]],
|
|
|
|
) -> RegistrationBehaviour:
|
2021-06-23 11:22:08 -04:00
|
|
|
return RegistrationBehaviour.DENY
|
|
|
|
|
|
|
|
|
2019-03-21 11:10:21 -04:00
|
|
|
class RegistrationTestCase(unittest.HomeserverTestCase):
|
2021-06-17 10:20:06 -04:00
|
|
|
"""Tests the RegistrationHandler."""
|
2016-04-20 10:21:40 -04:00
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
2020-03-24 14:33:49 -04:00
|
|
|
hs_config = self.default_config()
|
2019-03-19 08:05:05 -04:00
|
|
|
|
|
|
|
# some of the tests rely on us having a user consent version
|
2021-09-30 14:06:02 -04:00
|
|
|
hs_config.setdefault("user_consent", {}).update(
|
|
|
|
{
|
|
|
|
"version": "test_consent_version",
|
|
|
|
"template_dir": ".",
|
|
|
|
}
|
|
|
|
)
|
2019-05-13 16:01:14 -04:00
|
|
|
hs_config["max_mau_value"] = 50
|
|
|
|
hs_config["limit_usage_by_mau"] = True
|
2019-03-19 08:05:05 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
# Don't attempt to reach out over federation.
|
|
|
|
self.mock_federation_client = Mock()
|
|
|
|
self.mock_federation_client.make_query.side_effect = CodeMessageException(
|
|
|
|
500, ""
|
|
|
|
)
|
|
|
|
|
|
|
|
hs = self.setup_test_homeserver(
|
|
|
|
config=hs_config, federation_client=self.mock_federation_client
|
|
|
|
)
|
2021-06-18 07:15:52 -04:00
|
|
|
|
2019-03-21 11:10:21 -04:00
|
|
|
return hs
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
2019-02-20 02:47:31 -05:00
|
|
|
self.handler = self.hs.get_registration_handler()
|
2022-02-23 06:04:02 -05:00
|
|
|
self.store = self.hs.get_datastores().main
|
2018-08-02 11:57:35 -04:00
|
|
|
self.lots_of_users = 100
|
|
|
|
self.small_number_of_users = 1
|
2016-04-20 10:21:40 -04:00
|
|
|
|
2018-10-04 12:00:27 -04:00
|
|
|
self.requester = create_requester("@requester:test")
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_user_is_created_and_logged_in_if_doesnt_exist(self) -> None:
|
2018-10-04 12:00:27 -04:00
|
|
|
frank = UserID.from_string("@frank:test")
|
|
|
|
user_id = frank.to_string()
|
|
|
|
requester = create_requester(user_id)
|
2019-03-21 11:10:21 -04:00
|
|
|
result_user_id, result_token = self.get_success(
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_or_create_user(requester, frank.localpart, "Frankie")
|
2016-11-28 04:55:21 -05:00
|
|
|
)
|
2022-02-28 07:12:29 -05:00
|
|
|
self.assertEqual(result_user_id, user_id)
|
2021-05-12 10:04:51 -04:00
|
|
|
self.assertIsInstance(result_token, str)
|
|
|
|
self.assertGreater(len(result_token), 20)
|
2016-05-23 05:14:11 -04:00
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_if_user_exists(self) -> None:
|
2022-02-23 06:04:02 -05:00
|
|
|
store = self.hs.get_datastores().main
|
2016-05-23 05:14:11 -04:00
|
|
|
frank = UserID.from_string("@frank:test")
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_success(
|
2019-07-10 11:26:49 -04:00
|
|
|
store.register_user(user_id=frank.to_string(), password_hash=None)
|
2016-05-23 05:14:11 -04:00
|
|
|
)
|
2018-10-04 12:00:27 -04:00
|
|
|
local_part = frank.localpart
|
|
|
|
user_id = frank.to_string()
|
|
|
|
requester = create_requester(user_id)
|
2019-03-21 11:10:21 -04:00
|
|
|
result_user_id, result_token = self.get_success(
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_or_create_user(requester, local_part, None)
|
2016-11-28 04:55:21 -05:00
|
|
|
)
|
2022-02-28 07:12:29 -05:00
|
|
|
self.assertEqual(result_user_id, user_id)
|
2018-12-14 13:20:59 -05:00
|
|
|
self.assertTrue(result_token is not None)
|
2018-07-30 10:55:57 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"limit_usage_by_mau": False})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_mau_limits_when_disabled(self) -> None:
|
2018-07-30 10:55:57 -04:00
|
|
|
# Ensure does not throw exception
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_success(self.get_or_create_user(self.requester, "a", "display_name"))
|
2018-07-30 10:55:57 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"limit_usage_by_mau": True})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_get_or_create_user_mau_not_blocked(self) -> None:
|
2023-02-14 14:03:35 -05:00
|
|
|
self.store.count_monthly_users = Mock( # type: ignore[assignment]
|
2021-09-29 06:44:15 -04:00
|
|
|
return_value=make_awaitable(self.hs.config.server.max_mau_value - 1)
|
2018-08-02 11:57:35 -04:00
|
|
|
)
|
2018-07-30 10:55:57 -04:00
|
|
|
# Ensure does not throw exception
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_success(self.get_or_create_user(self.requester, "c", "User"))
|
2018-07-30 10:55:57 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"limit_usage_by_mau": True})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_get_or_create_user_mau_blocked(self) -> None:
|
2022-04-27 09:03:44 -04:00
|
|
|
self.store.get_monthly_active_count = Mock(
|
2020-09-08 07:26:55 -04:00
|
|
|
return_value=make_awaitable(self.lots_of_users)
|
2018-08-02 11:57:35 -04:00
|
|
|
)
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_failure(
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_or_create_user(self.requester, "b", "display_name"),
|
2019-03-21 11:10:21 -04:00
|
|
|
ResourceLimitError,
|
|
|
|
)
|
2018-07-30 10:55:57 -04:00
|
|
|
|
2022-04-27 09:03:44 -04:00
|
|
|
self.store.get_monthly_active_count = Mock(
|
2021-09-29 06:44:15 -04:00
|
|
|
return_value=make_awaitable(self.hs.config.server.max_mau_value)
|
2018-08-14 10:28:15 -04:00
|
|
|
)
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_failure(
|
2019-07-08 09:52:26 -04:00
|
|
|
self.get_or_create_user(self.requester, "b", "display_name"),
|
2019-03-21 11:10:21 -04:00
|
|
|
ResourceLimitError,
|
|
|
|
)
|
2018-08-14 10:28:15 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"limit_usage_by_mau": True})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_register_mau_blocked(self) -> None:
|
2018-08-02 11:57:35 -04:00
|
|
|
self.store.get_monthly_active_count = Mock(
|
2020-09-08 07:26:55 -04:00
|
|
|
return_value=make_awaitable(self.lots_of_users)
|
2018-08-02 11:57:35 -04:00
|
|
|
)
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_failure(
|
2019-07-08 14:01:08 -04:00
|
|
|
self.handler.register_user(localpart="local_part"), ResourceLimitError
|
2019-03-21 11:10:21 -04:00
|
|
|
)
|
2018-08-14 10:04:48 -04:00
|
|
|
|
|
|
|
self.store.get_monthly_active_count = Mock(
|
2021-09-29 06:44:15 -04:00
|
|
|
return_value=make_awaitable(self.hs.config.server.max_mau_value)
|
2018-08-14 10:04:48 -04:00
|
|
|
)
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_failure(
|
2019-07-08 14:01:08 -04:00
|
|
|
self.handler.register_user(localpart="local_part"), ResourceLimitError
|
2019-03-21 11:10:21 -04:00
|
|
|
)
|
2018-08-01 05:21:56 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config(
|
|
|
|
{"auto_join_rooms": ["#room:test"], "auto_join_rooms_for_guests": False}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_join_rooms_for_guests(self) -> None:
|
2020-06-05 13:18:15 -04:00
|
|
|
user_id = self.get_success(
|
|
|
|
self.handler.register_user(localpart="jeff", make_guest=True),
|
|
|
|
)
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertEqual(len(rooms), 0)
|
|
|
|
|
2020-06-30 15:41:36 -04:00
|
|
|
@override_config({"auto_join_rooms": ["#room:test"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms(self) -> None:
|
2018-10-04 12:00:27 -04:00
|
|
|
room_alias_str = "#room:test"
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2018-10-04 12:00:27 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
2019-03-21 11:10:21 -04:00
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
2018-10-04 12:00:27 -04:00
|
|
|
|
|
|
|
self.assertTrue(room_id["room_id"] in rooms)
|
|
|
|
self.assertEqual(len(rooms), 1)
|
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"auto_join_rooms": []})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms_with_no_rooms(self) -> None:
|
2018-10-04 12:00:27 -04:00
|
|
|
frank = UserID.from_string("@frank:test")
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(frank.localpart))
|
|
|
|
self.assertEqual(user_id, frank.to_string())
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2018-10-12 13:17:36 -04:00
|
|
|
self.assertEqual(len(rooms), 0)
|
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"auto_join_rooms": ["#room:another"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_where_room_is_another_domain(self) -> None:
|
2018-10-12 13:17:36 -04:00
|
|
|
frank = UserID.from_string("@frank:test")
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(frank.localpart))
|
|
|
|
self.assertEqual(user_id, frank.to_string())
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2018-10-12 13:17:36 -04:00
|
|
|
self.assertEqual(len(rooms), 0)
|
2018-10-04 12:00:27 -04:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config(
|
|
|
|
{"auto_join_rooms": ["#room:test"], "autocreate_auto_join_rooms": False}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_where_auto_create_is_false(self) -> None:
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2018-10-04 12:00:27 -04:00
|
|
|
self.assertEqual(len(rooms), 0)
|
2018-11-28 06:24:57 -05:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"auto_join_rooms": ["#room:test"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms_when_user_is_not_a_real_user(self) -> None:
|
2018-12-14 13:20:59 -05:00
|
|
|
room_alias_str = "#room:test"
|
2020-08-11 17:21:13 -04:00
|
|
|
self.store.is_real_user = Mock(return_value=make_awaitable(False))
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="support"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2018-12-14 13:20:59 -05:00
|
|
|
self.assertEqual(len(rooms), 0)
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2018-12-14 13:20:59 -05:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
2019-03-21 11:10:21 -04:00
|
|
|
self.get_failure(directory_handler.get_association(room_alias), SynapseError)
|
2018-12-14 13:20:59 -05:00
|
|
|
|
2020-06-30 15:41:36 -04:00
|
|
|
@override_config({"auto_join_rooms": ["#room:test"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms_when_user_is_the_first_real_user(self) -> None:
|
2019-09-09 07:43:51 -04:00
|
|
|
room_alias_str = "#room:test"
|
|
|
|
|
2023-02-14 14:03:35 -05:00
|
|
|
self.store.count_real_users = Mock(return_value=make_awaitable(1)) # type: ignore[assignment]
|
2020-08-11 17:21:13 -04:00
|
|
|
self.store.is_real_user = Mock(return_value=make_awaitable(True))
|
2019-09-09 07:43:51 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="real"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2019-09-09 07:43:51 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
self.assertTrue(room_id["room_id"] in rooms)
|
|
|
|
self.assertEqual(len(rooms), 1)
|
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config({"auto_join_rooms": ["#room:test"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms_when_user_is_not_the_first_real_user(
|
|
|
|
self,
|
|
|
|
) -> None:
|
2023-02-14 14:03:35 -05:00
|
|
|
self.store.count_real_users = Mock(return_value=make_awaitable(2)) # type: ignore[assignment]
|
2020-08-11 17:21:13 -04:00
|
|
|
self.store.is_real_user = Mock(return_value=make_awaitable(True))
|
2019-09-09 07:43:51 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="real"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertEqual(len(rooms), 0)
|
|
|
|
|
2020-06-30 15:41:36 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"auto_join_rooms": ["#room:test"],
|
|
|
|
"autocreate_auto_join_rooms_federated": False,
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_rooms_federated(self) -> None:
|
2020-06-30 15:41:36 -04:00
|
|
|
"""
|
|
|
|
Auto-created rooms that are private require an invite to go to the user
|
|
|
|
(instead of directly joining it).
|
|
|
|
"""
|
|
|
|
room_alias_str = "#room:test"
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
|
|
|
|
# Ensure the room was created.
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2020-06-30 15:41:36 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
# Ensure the room is properly not federated.
|
|
|
|
room = self.get_success(self.store.get_room_with_stats(room_id["room_id"]))
|
2023-02-14 14:03:35 -05:00
|
|
|
assert room is not None
|
2020-06-30 15:41:36 -04:00
|
|
|
self.assertFalse(room["federatable"])
|
|
|
|
self.assertFalse(room["public"])
|
|
|
|
self.assertEqual(room["join_rules"], "public")
|
|
|
|
self.assertIsNone(room["guest_access"])
|
|
|
|
|
|
|
|
# The user should be in the room.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
@override_config(
|
|
|
|
{"auto_join_rooms": ["#room:test"], "auto_join_mxid_localpart": "support"}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_join_mxid_localpart(self) -> None:
|
2020-06-30 15:41:36 -04:00
|
|
|
"""
|
|
|
|
Ensure the user still needs up in the room created by a different user.
|
|
|
|
"""
|
|
|
|
# Ensure the support user exists.
|
|
|
|
inviter = "@support:test"
|
|
|
|
|
|
|
|
room_alias_str = "#room:test"
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
|
|
|
|
# Ensure the room was created.
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2020-06-30 15:41:36 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
# Ensure the room is properly a public room.
|
|
|
|
room = self.get_success(self.store.get_room_with_stats(room_id["room_id"]))
|
2023-02-14 14:03:35 -05:00
|
|
|
assert room is not None
|
2020-06-30 15:41:36 -04:00
|
|
|
self.assertEqual(room["join_rules"], "public")
|
|
|
|
|
|
|
|
# Both users should be in the room.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(inviter))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
# Register a second user, which should also end up in the room.
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="bob"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"auto_join_rooms": ["#room:test"],
|
|
|
|
"autocreate_auto_join_room_preset": "private_chat",
|
|
|
|
"auto_join_mxid_localpart": "support",
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_room_preset(self) -> None:
|
2020-06-30 15:41:36 -04:00
|
|
|
"""
|
|
|
|
Auto-created rooms that are private require an invite to go to the user
|
|
|
|
(instead of directly joining it).
|
|
|
|
"""
|
|
|
|
# Ensure the support user exists.
|
|
|
|
inviter = "@support:test"
|
|
|
|
|
|
|
|
room_alias_str = "#room:test"
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
|
|
|
|
# Ensure the room was created.
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2020-06-30 15:41:36 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
# Ensure the room is properly a private room.
|
|
|
|
room = self.get_success(self.store.get_room_with_stats(room_id["room_id"]))
|
2023-02-14 14:03:35 -05:00
|
|
|
assert room is not None
|
2020-06-30 15:41:36 -04:00
|
|
|
self.assertFalse(room["public"])
|
|
|
|
self.assertEqual(room["join_rules"], "invite")
|
|
|
|
self.assertEqual(room["guest_access"], "can_join")
|
|
|
|
|
|
|
|
# Both users should be in the room.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(inviter))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
# Register a second user, which should also end up in the room.
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="bob"))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"auto_join_rooms": ["#room:test"],
|
|
|
|
"autocreate_auto_join_room_preset": "private_chat",
|
|
|
|
"auto_join_mxid_localpart": "support",
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_room_preset_guest(self) -> None:
|
2020-06-30 15:41:36 -04:00
|
|
|
"""
|
|
|
|
Auto-created rooms that are private require an invite to go to the user
|
|
|
|
(instead of directly joining it).
|
|
|
|
|
|
|
|
This should also work for guests.
|
|
|
|
"""
|
|
|
|
inviter = "@support:test"
|
|
|
|
|
|
|
|
room_alias_str = "#room:test"
|
|
|
|
user_id = self.get_success(
|
|
|
|
self.handler.register_user(localpart="jeff", make_guest=True)
|
|
|
|
)
|
|
|
|
|
|
|
|
# Ensure the room was created.
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2020-06-30 15:41:36 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
# Ensure the room is properly a private room.
|
|
|
|
room = self.get_success(self.store.get_room_with_stats(room_id["room_id"]))
|
2023-02-14 14:03:35 -05:00
|
|
|
assert room is not None
|
2020-06-30 15:41:36 -04:00
|
|
|
self.assertFalse(room["public"])
|
|
|
|
self.assertEqual(room["join_rules"], "invite")
|
|
|
|
self.assertEqual(room["guest_access"], "can_join")
|
|
|
|
|
|
|
|
# Both users should be in the room.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(inviter))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"auto_join_rooms": ["#room:test"],
|
|
|
|
"autocreate_auto_join_room_preset": "private_chat",
|
|
|
|
"auto_join_mxid_localpart": "support",
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_room_preset_invalid_permissions(self) -> None:
|
2020-06-30 15:41:36 -04:00
|
|
|
"""
|
|
|
|
Auto-created rooms that are private require an invite, check that
|
|
|
|
registration doesn't completely break if the inviter doesn't have proper
|
|
|
|
permissions.
|
|
|
|
"""
|
|
|
|
inviter = "@support:test"
|
|
|
|
|
|
|
|
# Register an initial user to create the room and such (essentially this
|
|
|
|
# is a subset of test_auto_create_auto_join_room_preset).
|
|
|
|
room_alias_str = "#room:test"
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
|
|
|
|
# Ensure the room was created.
|
2020-10-09 07:24:34 -04:00
|
|
|
directory_handler = self.hs.get_directory_handler()
|
2020-06-30 15:41:36 -04:00
|
|
|
room_alias = RoomAlias.from_string(room_alias_str)
|
|
|
|
room_id = self.get_success(directory_handler.get_association(room_alias))
|
|
|
|
|
|
|
|
# Ensure the room exists.
|
|
|
|
self.get_success(self.store.get_room_with_stats(room_id["room_id"]))
|
|
|
|
|
|
|
|
# Both users should be in the room.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(inviter))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertIn(room_id["room_id"], rooms)
|
|
|
|
|
|
|
|
# Lower the permissions of the inviter.
|
|
|
|
event_creation_handler = self.hs.get_event_creation_handler()
|
|
|
|
requester = create_requester(inviter)
|
2023-02-24 16:15:29 -05:00
|
|
|
event, unpersisted_context = self.get_success(
|
2020-06-30 15:41:36 -04:00
|
|
|
event_creation_handler.create_event(
|
|
|
|
requester,
|
|
|
|
{
|
|
|
|
"type": "m.room.power_levels",
|
|
|
|
"state_key": "",
|
|
|
|
"room_id": room_id["room_id"],
|
|
|
|
"content": {"invite": 100, "users": {inviter: 0}},
|
|
|
|
"sender": inviter,
|
|
|
|
},
|
|
|
|
)
|
|
|
|
)
|
2023-02-24 16:15:29 -05:00
|
|
|
context = self.get_success(unpersisted_context.persist(event))
|
2020-06-30 15:41:36 -04:00
|
|
|
self.get_success(
|
2022-09-28 06:11:48 -04:00
|
|
|
event_creation_handler.handle_new_client_event(
|
|
|
|
requester, events_and_context=[(event, context)]
|
|
|
|
)
|
2020-06-30 15:41:36 -04:00
|
|
|
)
|
|
|
|
|
|
|
|
# Register a second user, which won't be be in the room (or even have an invite)
|
|
|
|
# since the inviter no longer has the proper permissions.
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="bob"))
|
|
|
|
|
|
|
|
# This user should not be in any rooms.
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
invited_rooms = self.get_success(
|
|
|
|
self.store.get_invited_rooms_for_local_user(user_id)
|
|
|
|
)
|
|
|
|
self.assertEqual(rooms, set())
|
|
|
|
self.assertEqual(invited_rooms, [])
|
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"user_consent": {
|
|
|
|
"block_events_error": "Error",
|
|
|
|
"require_at_registration": True,
|
|
|
|
},
|
|
|
|
"form_secret": "53cr3t",
|
|
|
|
"public_baseurl": "http://test",
|
|
|
|
"auto_join_rooms": ["#room:test"],
|
|
|
|
},
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_where_no_consent(self) -> None:
|
2019-03-19 07:38:59 -04:00
|
|
|
"""Test to ensure that the first user is not auto-joined to a room if
|
|
|
|
they have not given general consent.
|
|
|
|
"""
|
|
|
|
|
|
|
|
# Given:-
|
|
|
|
# * a user must give consent,
|
|
|
|
# * they have not given that consent
|
|
|
|
# * The server is configured to auto-join to a room
|
|
|
|
# (and autocreate if necessary)
|
|
|
|
|
|
|
|
# When:-
|
2021-09-30 14:06:02 -04:00
|
|
|
# * the user is registered
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="jeff"))
|
2019-03-19 07:38:59 -04:00
|
|
|
|
|
|
|
# Then:-
|
|
|
|
# * Ensure that they have not been joined to the room
|
2019-07-08 14:01:08 -04:00
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
2018-11-28 06:24:57 -05:00
|
|
|
self.assertEqual(len(rooms), 0)
|
2018-12-14 13:20:59 -05:00
|
|
|
|
2021-09-30 14:06:02 -04:00
|
|
|
# The user provides consent; ensure they are now in the rooms.
|
|
|
|
self.get_success(self.handler.post_consent_actions(user_id))
|
|
|
|
rooms = self.get_success(self.store.get_rooms_for_user(user_id))
|
|
|
|
self.assertEqual(len(rooms), 1)
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_register_support_user(self) -> None:
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(
|
|
|
|
self.handler.register_user(localpart="user", user_type=UserTypes.SUPPORT)
|
2019-03-21 11:10:21 -04:00
|
|
|
)
|
2019-07-08 14:01:08 -04:00
|
|
|
d = self.store.is_support_user(user_id)
|
|
|
|
self.assertTrue(self.get_success(d))
|
2018-12-14 13:20:59 -05:00
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_register_not_support_user(self) -> None:
|
2019-07-08 14:01:08 -04:00
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="user"))
|
|
|
|
d = self.store.is_support_user(user_id)
|
|
|
|
self.assertFalse(self.get_success(d))
|
2019-05-20 06:20:08 -04:00
|
|
|
|
2023-05-05 12:13:50 -04:00
|
|
|
def test_invalid_user_id(self) -> None:
|
|
|
|
invalid_user_id = "+abcd"
|
|
|
|
self.get_failure(
|
|
|
|
self.handler.register_user(localpart=invalid_user_id), SynapseError
|
|
|
|
)
|
|
|
|
|
|
|
|
@override_config({"experimental_features": {"msc4009_e164_mxids": True}})
|
|
|
|
def text_extended_user_ids(self) -> None:
|
|
|
|
"""+ should be allowed according to MSC4009."""
|
|
|
|
valid_user_id = "+1234"
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart=valid_user_id))
|
|
|
|
self.assertEqual(user_id, valid_user_id)
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_invalid_user_id_length(self) -> None:
|
2019-05-20 06:20:08 -04:00
|
|
|
invalid_user_id = "x" * 256
|
2019-07-08 14:01:08 -04:00
|
|
|
self.get_failure(
|
|
|
|
self.handler.register_user(localpart=invalid_user_id), SynapseError
|
|
|
|
)
|
2019-07-08 09:52:26 -04:00
|
|
|
|
2021-06-18 07:15:52 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"modules": [
|
|
|
|
{
|
|
|
|
"module": TestSpamChecker.__module__ + ".DenyAll",
|
|
|
|
}
|
|
|
|
]
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_spam_checker_deny(self) -> None:
|
2020-08-20 15:42:58 -04:00
|
|
|
"""A spam checker can deny registration, which results in an error."""
|
|
|
|
self.get_failure(self.handler.register_user(localpart="user"), SynapseError)
|
|
|
|
|
2021-06-23 11:22:08 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"spam_checker": [
|
|
|
|
{
|
|
|
|
"module": TestSpamChecker.__module__ + ".LegacyAllowAll",
|
|
|
|
}
|
|
|
|
]
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_spam_checker_legacy_allow(self) -> None:
|
2021-06-23 11:22:08 -04:00
|
|
|
"""Tests that a legacy spam checker implementing the legacy 3-arg version of the
|
|
|
|
check_registration_for_spam callback is correctly called.
|
|
|
|
|
|
|
|
In this test and the following one we test both success and failure to make sure
|
|
|
|
any failure comes from the spam checker (and not something else failing in the
|
|
|
|
call stack) and any success comes from the spam checker (and not because a
|
|
|
|
misconfiguration prevented it from being loaded).
|
|
|
|
"""
|
|
|
|
self.get_success(self.handler.register_user(localpart="user"))
|
|
|
|
|
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"spam_checker": [
|
|
|
|
{
|
|
|
|
"module": TestSpamChecker.__module__ + ".LegacyDenyAll",
|
|
|
|
}
|
|
|
|
]
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_spam_checker_legacy_deny(self) -> None:
|
2021-06-23 11:22:08 -04:00
|
|
|
"""Tests that a legacy spam checker implementing the legacy 3-arg version of the
|
|
|
|
check_registration_for_spam callback is correctly called.
|
|
|
|
|
|
|
|
In this test and the previous one we test both success and failure to make sure
|
|
|
|
any failure comes from the spam checker (and not something else failing in the
|
|
|
|
call stack) and any success comes from the spam checker (and not because a
|
|
|
|
misconfiguration prevented it from being loaded).
|
|
|
|
"""
|
|
|
|
self.get_failure(self.handler.register_user(localpart="user"), SynapseError)
|
|
|
|
|
2021-06-18 07:15:52 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"modules": [
|
|
|
|
{
|
|
|
|
"module": TestSpamChecker.__module__ + ".BanAll",
|
|
|
|
}
|
|
|
|
]
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_spam_checker_shadow_ban(self) -> None:
|
2020-08-20 15:42:58 -04:00
|
|
|
"""A spam checker can choose to shadow-ban a user, which allows registration to succeed."""
|
|
|
|
user_id = self.get_success(self.handler.register_user(localpart="user"))
|
|
|
|
|
|
|
|
# Get an access token.
|
2021-05-12 10:04:51 -04:00
|
|
|
token = "testtok"
|
2020-08-20 15:42:58 -04:00
|
|
|
self.get_success(
|
|
|
|
self.store.add_access_token_to_user(
|
|
|
|
user_id=user_id, token=token, device_id=None, valid_until_ms=None
|
|
|
|
)
|
|
|
|
)
|
|
|
|
|
|
|
|
# Ensure the user was marked as shadow-banned.
|
|
|
|
request = Mock(args={})
|
|
|
|
request.args[b"access_token"] = [token.encode("ascii")]
|
|
|
|
request.requestHeaders.getRawHeaders = mock_getRawHeaders()
|
2022-06-17 08:48:55 -04:00
|
|
|
auth = InternalAuth(self.hs)
|
2020-08-20 15:42:58 -04:00
|
|
|
requester = self.get_success(auth.get_user_by_req(request))
|
|
|
|
|
|
|
|
self.assertTrue(requester.shadow_banned)
|
|
|
|
|
2021-06-18 07:15:52 -04:00
|
|
|
@override_config(
|
|
|
|
{
|
|
|
|
"modules": [
|
|
|
|
{
|
|
|
|
"module": TestSpamChecker.__module__ + ".BanBadIdPUser",
|
|
|
|
}
|
|
|
|
]
|
|
|
|
}
|
|
|
|
)
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_spam_checker_receives_sso_type(self) -> None:
|
2021-03-16 08:41:41 -04:00
|
|
|
"""Test rejecting registration based on SSO type"""
|
|
|
|
f = self.get_failure(
|
|
|
|
self.handler.register_user(localpart="bobflimflob", auth_provider_id="cas"),
|
|
|
|
SynapseError,
|
|
|
|
)
|
|
|
|
exception = f.value
|
|
|
|
|
|
|
|
# We return 429 from the spam checker for denied registrations
|
|
|
|
self.assertIsInstance(exception, SynapseError)
|
|
|
|
self.assertEqual(exception.code, 429)
|
|
|
|
|
|
|
|
# Check the same username can register using SAML
|
|
|
|
self.get_success(
|
|
|
|
self.handler.register_user(localpart="bobflimflob", auth_provider_id="saml")
|
|
|
|
)
|
|
|
|
|
2020-05-01 10:15:36 -04:00
|
|
|
async def get_or_create_user(
|
2022-12-16 06:53:01 -05:00
|
|
|
self,
|
|
|
|
requester: Requester,
|
|
|
|
localpart: str,
|
|
|
|
displayname: Optional[str],
|
|
|
|
password_hash: Optional[str] = None,
|
|
|
|
) -> Tuple[str, str]:
|
2019-07-08 09:52:26 -04:00
|
|
|
"""Creates a new user if the user does not exist,
|
|
|
|
else revokes all previous access tokens and generates a new one.
|
|
|
|
|
|
|
|
XXX: this used to be in the main codebase, but was only used by this file,
|
|
|
|
so got moved here. TODO: get rid of it, probably
|
|
|
|
|
|
|
|
Args:
|
|
|
|
localpart : The local part of the user ID to register. If None,
|
|
|
|
one will be randomly generated.
|
|
|
|
Returns:
|
|
|
|
A tuple of (user_id, access_token).
|
|
|
|
"""
|
|
|
|
if localpart is None:
|
|
|
|
raise SynapseError(400, "Request must include user id")
|
2022-06-14 04:51:15 -04:00
|
|
|
await self.hs.get_auth_blocking().check_auth_blocking()
|
2019-07-08 09:52:26 -04:00
|
|
|
need_register = True
|
|
|
|
|
|
|
|
try:
|
2020-05-01 10:15:36 -04:00
|
|
|
await self.handler.check_username(localpart)
|
2019-07-08 09:52:26 -04:00
|
|
|
except SynapseError as e:
|
|
|
|
if e.errcode == Codes.USER_IN_USE:
|
|
|
|
need_register = False
|
|
|
|
else:
|
|
|
|
raise
|
|
|
|
|
|
|
|
user = UserID(localpart, self.hs.hostname)
|
|
|
|
user_id = user.to_string()
|
2021-05-12 10:04:51 -04:00
|
|
|
token = self.hs.get_auth_handler().generate_access_token(user)
|
2019-07-08 09:52:26 -04:00
|
|
|
|
|
|
|
if need_register:
|
2020-05-01 10:15:36 -04:00
|
|
|
await self.handler.register_with_store(
|
2019-07-08 09:52:26 -04:00
|
|
|
user_id=user_id,
|
|
|
|
password_hash=password_hash,
|
|
|
|
create_profile_with_displayname=user.localpart,
|
|
|
|
)
|
|
|
|
else:
|
2020-05-01 10:15:36 -04:00
|
|
|
await self.hs.get_auth_handler().delete_access_tokens_for_user(user_id)
|
2019-07-12 12:26:02 -04:00
|
|
|
|
2020-05-01 10:15:36 -04:00
|
|
|
await self.store.add_access_token_to_user(
|
2019-07-12 12:26:02 -04:00
|
|
|
user_id=user_id, token=token, device_id=None, valid_until_ms=None
|
|
|
|
)
|
2019-07-08 09:52:26 -04:00
|
|
|
|
|
|
|
if displayname is not None:
|
|
|
|
# logger.info("setting user display name: %s -> %s", user_id, displayname)
|
2020-05-01 10:15:36 -04:00
|
|
|
await self.hs.get_profile_handler().set_displayname(
|
2019-07-08 09:52:26 -04:00
|
|
|
user, requester, displayname, by_admin=True
|
|
|
|
)
|
|
|
|
|
2019-08-30 11:28:26 -04:00
|
|
|
return user_id, token
|
2021-06-23 10:14:52 -04:00
|
|
|
|
|
|
|
|
|
|
|
class RemoteAutoJoinTestCase(unittest.HomeserverTestCase):
|
|
|
|
"""Tests auto-join on remote rooms."""
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def make_homeserver(self, reactor: MemoryReactor, clock: Clock) -> HomeServer:
|
2021-06-23 10:14:52 -04:00
|
|
|
self.room_id = "!roomid:remotetest"
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
async def update_membership(*args: Any, **kwargs: Any) -> None:
|
2021-06-23 10:14:52 -04:00
|
|
|
pass
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
async def lookup_room_alias(
|
|
|
|
*args: Any, **kwargs: Any
|
|
|
|
) -> Tuple[RoomID, List[str]]:
|
2021-06-23 10:14:52 -04:00
|
|
|
return RoomID.from_string(self.room_id), ["remotetest"]
|
|
|
|
|
|
|
|
self.room_member_handler = Mock(spec=["update_membership", "lookup_room_alias"])
|
|
|
|
self.room_member_handler.update_membership.side_effect = update_membership
|
|
|
|
self.room_member_handler.lookup_room_alias.side_effect = lookup_room_alias
|
|
|
|
|
|
|
|
hs = self.setup_test_homeserver(room_member_handler=self.room_member_handler)
|
|
|
|
return hs
|
|
|
|
|
2022-12-16 06:53:01 -05:00
|
|
|
def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
|
2021-06-23 10:14:52 -04:00
|
|
|
self.handler = self.hs.get_registration_handler()
|
2022-02-23 06:04:02 -05:00
|
|
|
self.store = self.hs.get_datastores().main
|
2021-06-23 10:14:52 -04:00
|
|
|
|
|
|
|
@override_config({"auto_join_rooms": ["#room:remotetest"]})
|
2022-12-16 06:53:01 -05:00
|
|
|
def test_auto_create_auto_join_remote_room(self) -> None:
|
2021-06-23 10:14:52 -04:00
|
|
|
"""Tests that we don't attempt to create remote rooms, and that we don't attempt
|
|
|
|
to invite ourselves to rooms we're not in."""
|
|
|
|
|
|
|
|
# Register a first user; this should call _create_and_join_rooms
|
|
|
|
self.get_success(self.handler.register_user(localpart="jeff"))
|
|
|
|
|
|
|
|
_, kwargs = self.room_member_handler.update_membership.call_args
|
|
|
|
|
|
|
|
self.assertEqual(kwargs["room_id"], self.room_id)
|
|
|
|
self.assertEqual(kwargs["action"], "join")
|
|
|
|
self.assertEqual(kwargs["remote_room_hosts"], ["remotetest"])
|
|
|
|
|
|
|
|
# Register a second user; this should call _join_rooms
|
|
|
|
self.get_success(self.handler.register_user(localpart="jeff2"))
|
|
|
|
|
|
|
|
_, kwargs = self.room_member_handler.update_membership.call_args
|
|
|
|
|
|
|
|
self.assertEqual(kwargs["room_id"], self.room_id)
|
|
|
|
self.assertEqual(kwargs["action"], "join")
|
|
|
|
self.assertEqual(kwargs["remote_room_hosts"], ["remotetest"])
|