123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268 |
- # Copyright 2014-2021 The Matrix.org Foundation C.I.C.
- #
- # 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.
- from twisted.test.proto_helpers import MemoryReactor
- from synapse.api.constants import UserTypes
- from synapse.api.errors import ThreepidValidationError
- from synapse.server import HomeServer
- from synapse.types import JsonDict, UserID
- from synapse.util import Clock
- from tests.unittest import HomeserverTestCase, override_config
- class RegistrationStoreTestCase(HomeserverTestCase):
- def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
- self.store = hs.get_datastores().main
- self.user_id = "@my-user:test"
- self.tokens = ["AbCdEfGhIjKlMnOpQrStUvWxYz", "BcDeFgHiJkLmNoPqRsTuVwXyZa"]
- self.pwhash = "{xx1}123456789"
- self.device_id = "akgjhdjklgshg"
- def test_register(self) -> None:
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- self.assertEqual(
- {
- # TODO(paul): Surely this field should be 'user_id', not 'name'
- "name": self.user_id,
- "password_hash": self.pwhash,
- "admin": 0,
- "is_guest": 0,
- "consent_version": None,
- "consent_ts": None,
- "consent_server_notice_sent": None,
- "appservice_id": None,
- "creation_ts": 0,
- "user_type": None,
- "deactivated": 0,
- "shadow_banned": 0,
- "approved": 1,
- },
- (self.get_success(self.store.get_user_by_id(self.user_id))),
- )
- def test_consent(self) -> None:
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- before_consent = self.clock.time_msec()
- self.reactor.advance(5)
- self.get_success(self.store.user_set_consent_version(self.user_id, "1"))
- self.reactor.advance(5)
- user = self.get_success(self.store.get_user_by_id(self.user_id))
- assert user
- self.assertEqual(user["consent_version"], "1")
- self.assertGreater(user["consent_ts"], before_consent)
- self.assertLess(user["consent_ts"], self.clock.time_msec())
- def test_add_tokens(self) -> None:
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- self.get_success(
- self.store.add_access_token_to_user(
- self.user_id, self.tokens[1], self.device_id, valid_until_ms=None
- )
- )
- result = self.get_success(self.store.get_user_by_access_token(self.tokens[1]))
- assert result
- self.assertEqual(result.user_id, self.user_id)
- self.assertEqual(result.device_id, self.device_id)
- self.assertIsNotNone(result.token_id)
- def test_user_delete_access_tokens(self) -> None:
- # add some tokens
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- self.get_success(
- self.store.add_access_token_to_user(
- self.user_id, self.tokens[0], device_id=None, valid_until_ms=None
- )
- )
- self.get_success(
- self.store.add_access_token_to_user(
- self.user_id, self.tokens[1], self.device_id, valid_until_ms=None
- )
- )
- # now delete some
- self.get_success(
- self.store.user_delete_access_tokens(self.user_id, device_id=self.device_id)
- )
- # check they were deleted
- user = self.get_success(self.store.get_user_by_access_token(self.tokens[1]))
- self.assertIsNone(user, "access token was not deleted by device_id")
- # check the one not associated with the device was not deleted
- user = self.get_success(self.store.get_user_by_access_token(self.tokens[0]))
- assert user
- self.assertEqual(self.user_id, user.user_id)
- # now delete the rest
- self.get_success(self.store.user_delete_access_tokens(self.user_id))
- user = self.get_success(self.store.get_user_by_access_token(self.tokens[0]))
- self.assertIsNone(user, "access token was not deleted without device_id")
- def test_is_support_user(self) -> None:
- TEST_USER = "@test:test"
- SUPPORT_USER = "@support:test"
- res = self.get_success(self.store.is_support_user(None)) # type: ignore[arg-type]
- self.assertFalse(res)
- self.get_success(
- self.store.register_user(user_id=TEST_USER, password_hash=None)
- )
- res = self.get_success(self.store.is_support_user(TEST_USER))
- self.assertFalse(res)
- self.get_success(
- self.store.register_user(
- user_id=SUPPORT_USER, password_hash=None, user_type=UserTypes.SUPPORT
- )
- )
- res = self.get_success(self.store.is_support_user(SUPPORT_USER))
- self.assertTrue(res)
- def test_3pid_inhibit_invalid_validation_session_error(self) -> None:
- """Tests that enabling the configuration option to inhibit 3PID errors on
- /requestToken also inhibits validation errors caused by an unknown session ID.
- """
- # Check that, with the config setting set to false (the default value), a
- # validation error is caused by the unknown session ID.
- e = self.get_failure(
- self.store.validate_threepid_session(
- "fake_sid",
- "fake_client_secret",
- "fake_token",
- 0,
- ),
- ThreepidValidationError,
- )
- self.assertEqual(e.value.msg, "Unknown session_id", e)
- # Set the config setting to true.
- self.store._ignore_unknown_session_error = True
- # Check that now the validation error is caused by the token not matching.
- e = self.get_failure(
- self.store.validate_threepid_session(
- "fake_sid",
- "fake_client_secret",
- "fake_token",
- 0,
- ),
- ThreepidValidationError,
- )
- self.assertEqual(e.value.msg, "Validation token not found or has expired", e)
- class ApprovalRequiredRegistrationTestCase(HomeserverTestCase):
- def default_config(self) -> JsonDict:
- config = super().default_config()
- # If there's already some config for this feature in the default config, it
- # means we're overriding it with @override_config. In this case we don't want
- # to do anything more with it.
- msc3866_config = config.get("experimental_features", {}).get("msc3866")
- if msc3866_config is not None:
- return config
- # Require approval for all new accounts.
- config["experimental_features"] = {
- "msc3866": {
- "enabled": True,
- "require_approval_for_new_accounts": True,
- }
- }
- return config
- def prepare(self, reactor: MemoryReactor, clock: Clock, hs: HomeServer) -> None:
- self.store = hs.get_datastores().main
- self.user_id = "@my-user:test"
- self.pwhash = "{xx1}123456789"
- @override_config(
- {
- "experimental_features": {
- "msc3866": {
- "enabled": True,
- "require_approval_for_new_accounts": False,
- }
- }
- }
- )
- def test_approval_not_required(self) -> None:
- """Tests that if we don't require approval for new accounts, newly created
- accounts are automatically marked as approved.
- """
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- user = self.get_success(self.store.get_user_by_id(self.user_id))
- assert user is not None
- self.assertTrue(user["approved"])
- approved = self.get_success(self.store.is_user_approved(self.user_id))
- self.assertTrue(approved)
- def test_approval_required(self) -> None:
- """Tests that if we require approval for new accounts, newly created accounts
- are not automatically marked as approved.
- """
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- user = self.get_success(self.store.get_user_by_id(self.user_id))
- assert user is not None
- self.assertFalse(user["approved"])
- approved = self.get_success(self.store.is_user_approved(self.user_id))
- self.assertFalse(approved)
- def test_override(self) -> None:
- """Tests that if we require approval for new accounts, but we explicitly say the
- new user should be considered approved, they're marked as approved.
- """
- self.get_success(
- self.store.register_user(
- self.user_id,
- self.pwhash,
- approved=True,
- )
- )
- user = self.get_success(self.store.get_user_by_id(self.user_id))
- self.assertIsNotNone(user)
- assert user is not None
- self.assertEqual(user["approved"], 1)
- approved = self.get_success(self.store.is_user_approved(self.user_id))
- self.assertTrue(approved)
- def test_approve_user(self) -> None:
- """Tests that approving the user updates their approval status."""
- self.get_success(self.store.register_user(self.user_id, self.pwhash))
- approved = self.get_success(self.store.is_user_approved(self.user_id))
- self.assertFalse(approved)
- self.get_success(
- self.store.update_user_approval_status(
- UserID.from_string(self.user_id), True
- )
- )
- approved = self.get_success(self.store.is_user_approved(self.user_id))
- self.assertTrue(approved)
|