14110 Commits (3914576b2b6272a4be790c89d87649d257018dcf)

Author	SHA1	Message	Date
Patrick Cloke	d0e78af35e	Add missing type hints to synapse.replication. (#11938)	3 years ago
Patrick Cloke	8c94b3abe9	Experimental support to include bundled aggregations in search results (MSC3666) (#11837)	3 years ago
Patrick Cloke	6c0984e3f0	Remove unnecessary ignores due to Twisted upgrade. (#11939) ... Twisted 22.1.0 fixed some internal type hints, allowing Synapse to remove ignore calls for parameters to connectTCP.	3 years ago
Patrick Cloke	63d90f10ec	Add missing type hints to synapse.replication.http. (#11856)	3 years ago
Patrick Cloke	8b309adb43	Fetch edits for multiple events in a single query. (#11660) ... This should reduce database usage when fetching bundled aggregations as the number of individual queries (and round trips to the database) are reduced.	3 years ago
Patrick Cloke	380c3d40f4	Return JSON errors for unknown resources under /matrix/client. (#11930) ... Re-applies the changes from 3e0cfd447e (#11602), reverting d93ec0a0ba (#11764) now that the conflict with the webclient listener was fixed in 119edf51eb (#11895).	3 years ago
Brendan Abolivier	7d56b6c083	1.52.0	3 years ago
Andrew Morgan	3655585e85	Add a docstring to `add_device_change_to_streams` and fix some nearby types (#11912)	3 years ago
Brendan Abolivier	0640f8ebaa	Add a callback to allow modules to deny 3PID (#11854) ... Part of the Tchap Synapse mainlining. This allows modules to implement extra logic to figure out whether a given 3PID can be added to the local homeserver. In the Tchap use case, this will allow a Synapse module to interface with the custom endpoint /internal_info.	3 years ago
Eric Eastwood	fef2e792be	Fix historical messages backfilling in random order on remote homeservers (MSC2716) (#11114) ... Fix https://github.com/matrix-org/synapse/issues/11091 Fix https://github.com/matrix-org/synapse/issues/10764 (side-stepping the issue because we no longer have to deal with `fake_prev_event_id`) 1. Made the `/backfill` response return messages in `(depth, stream_ordering)` order (previously only sorted by `depth`) - Technically, it shouldn't really matter how `/backfill` returns things but I'm just trying to make the `stream_ordering` a little more consistent from the origin to the remote homeservers in order to get the order of messages from `/messages` consistent ([sorted by `(topological_ordering, stream_ordering)`](https://github.com/matrix-org/synapse/blob/develop/docs/development/room-dag-concepts.md#depth-and-stream-ordering)). - Even now that we return backfilled messages in order, it still doesn't guarantee the same `stream_ordering` (and more importantly the [`/messages` order](https://github.com/matrix-org/synapse/blob/develop/docs/development/room-dag-concepts.md#depth-and-stream-ordering)) on the other server. For example, if a room has a bunch of history imported and someone visits a permalink to a historical message back in time, their homeserver will skip over the historical messages in between and insert the permalink as the next message in the `stream_order` and totally throw off the sort. - This will be even more the case when we add the [MSC3030 jump to date API endpoint](https://github.com/matrix-org/matrix-doc/pull/3030) so the static archives can navigate and jump to a certain date. - We're solving this in the future by switching to [online topological ordering](https://github.com/matrix-org/gomatrixserverlib/issues/187) and [chunking](https://github.com/matrix-org/synapse/issues/3785) which by its nature will apply retroactively to fix any inconsistencies introduced by people permalinking 2. As we're navigating `prev_events` to return in `/backfill`, we order by `depth` first (newest -> oldest) and now also tie-break based on the `stream_ordering` (newest -> oldest). This is technically important because MSC2716 inserts a bunch of historical messages at the same `depth` so it's best to be prescriptive about which ones we should process first. In reality, I think the code already looped over the historical messages as expected because the database is already in order. 3. Making the historical state chain and historical event chain float on their own by having no `prev_events` instead of a fake `prev_event` which caused backfill to get clogged with an unresolvable event. Fixes https://github.com/matrix-org/synapse/issues/11091 and https://github.com/matrix-org/synapse/issues/10764 4. We no longer find connected insertion events by finding a potential `prev_event` connection to the current event we're iterating over. We now solely rely on marker events which when processed, add the insertion event as an extremity and the federating homeserver can ask about it when time calls. - Related discussion, https://github.com/matrix-org/synapse/pull/11114#discussion_r741514793 Before | After --- | ---  |  #### Why aren't we sorting topologically when receiving backfill events? > The main reason we're going to opt to not sort topologically when receiving backfill events is because it's probably best to do whatever is easiest to make it just work. People will probably have opinions once they look at [MSC2716](https://github.com/matrix-org/matrix-doc/pull/2716) which could change whatever implementation anyway. > > As mentioned, ideally we would do this but code necessary to make the fake edges but it gets confusing and gives an impression of “just whyyyy” (feels icky). This problem also dissolves with online topological ordering. > > -- https://github.com/matrix-org/synapse/pull/11114#discussion_r741517138 See https://github.com/matrix-org/synapse/pull/11114#discussion_r739610091 for the technical difficulties	3 years ago
Andrew Morgan	cf06783d54	Remove optional state of `ApplicationService.is_interested`'s `store` parameter (#11911)	3 years ago
Patrick Cloke	314ca4c86d	Pass the proper type when uploading files. (#11927) ... The Content-Length header should be treated as an int, not a string. This shouldn't have any user-facing change.	3 years ago
Andrew Morgan	e03dde259b	Clean up an indirect reference to the homeserver datastore (#11914)	3 years ago
Jonathan de Jong	02632b3504	Stabilise MSC3231 (Token Based Registration) (#11867)	3 years ago
Patrick Cloke	119edf51eb	Remove support for the webclient listener. (#11895) ... Also remove support for non-HTTP(S) web_client_location.	3 years ago
Brendan Abolivier	d80d39b035	Add a ratelimiter for 3pid invite (#11892)	3 years ago
Richard van der Hoff	964f5b9324	Improve opentracing for federation requests (#11870) ... The idea here is to set the parent span for incoming federation requests to the *outgoing* span on the other end. That means that you can see (most of) the full end-to-end flow when you have a process that includes federation requests. However, in order not to lose information, we still want a link to the `incoming-federation-request` span from the servlet, so we have to create another span to do exactly that.	3 years ago
Richard van der Hoff	31b554c297	Fixes for opentracing scopes (#11869) ... `start_active_span` was inconsistent as to whether it would activate the span immediately, or wait for `scope.__enter__` to happen (it depended on whether the current logcontext already had an associated scope). The inconsistency was rather confusing if you were hoping to set up a couple of separate spans before activating either. Looking at the other implementations of opentracing `ScopeManager`s, the intention is that it *should* be activated immediately, as the name implies. Indeed, the idea is that you don't have to use the scope as a contextmanager at all - you can just call `.close` on the result. Hence, our cleanup has to happen in `.close` rather than `.__exit__`. So, the main change here is to ensure that `start_active_span` does activate the span, and that `scope.close()` does close the scope. We also add some tests, which requires a `tracer` param so that we don't have to rely on the global variable in unit tests.	3 years ago
Patrick Cloke	a8da046907	Invalidate the get_users_in_room{_with_profile} caches only when necessary. (#11878) ... The get_users_in_room and get_users_in_room_with_profiles are now only invalidated when the membership of a room changes, instead of during any state change in the room.	3 years ago
reivilibre	41818cda1f	Fix type errors introduced by new annotations in the Prometheus Client library. (#11832) ... Co-authored-by: David Robertson <davidr@element.io>	3 years ago
David Robertson	dd7f825118	Fix losing incoming EDUs if debug logging enabled (#11890) ... * Fix losing incoming EDUs if debug logging enabled Fixes #11889. Homeservers should only be affected if the `synapse.8631_debug` logger was enabled for DEBUG mode. I am not sure if this merits a bugfix release: I think the logging can be disabled in config if anyone is affected? But it is still pretty bad.	3 years ago
David Robertson	f510fba4ba	Describe `prune_unread_entries` in docstrings (#11876) ... Should have been caught in #10826.	3 years ago
Patrick Cloke	acda9f07c8	Revert experimental push rules from #7997. (#11884) ... Manually reverts the merge from cdbb8e6d6e.	3 years ago
reivilibre	af795173be	Add a background database update to purge account data for deactivated users. (#11655)	3 years ago
Brendan Abolivier	5c16c33021	Allow modules to retrieve server and worker names (#11868) ... Fixes #10701	3 years ago
Andrew Morgan	64ec45fc1b	Send to-device messages to application services (#11215) ... Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>	3 years ago
David Robertson	a35e9db9be	1.52.0rc1	3 years ago
Patrick Cloke	02755c3188	Remove the obsolete MSC1849 configuration flag. (#11843) ... MSC1849 was replaced by MSC2675, which was merged. The configuration flag, which defaulted to true, is no longer useful.	3 years ago
Brendan Abolivier	bf60da1a60	Configurable limits on avatars (#11846) ... Only allow files which file size and content types match configured limits to be set as avatar. Most of the inspiration from the non-test code comes from matrix-org/synapse-dinsic#19	3 years ago
Brendan Abolivier	6d482ba259	Pass `isolation_level` to `runWithConnection` (#11847) ... This was missed in https://github.com/matrix-org/synapse/pull/11799	3 years ago
Richard van der Hoff	57e4786e90	Create singletons for `StateFilter.{all,none}()` (#11836) ... No point recreating these for each call, since they are frozen	3 years ago
Dirk Klimpel	fd65139714	Fix some indentation inconsistencies in the sample config (modules) (#11838)	3 years ago
Vaishnav Nair	cef0d5d90a	Include `prev_content` field in AS events (#11798) ... * Include 'prev_content' field in AS events Signed-off-by: Vaishnav Nair <nairvaishnav007@icloud.com> Co-authored-by: Brendan Abolivier <babolivier@matrix.org>	3 years ago
Brendan Abolivier	2d3bd9aa67	Add a module callback to set username at registration (#11790) ... This is in the context of mainlining the Tchap fork of Synapse. Currently in Tchap usernames are derived from the user's email address (extracted from the UIA results, more specifically the m.login.email.identity step). This change also exports the check_username method from the registration handler as part of the module API, so that a module can check if the username it's trying to generate is correct and doesn't conflict with an existing one, and fallback gracefully if not. Co-authored-by: David Robertson <davidr@element.io>	3 years ago
Patrick Cloke	2897fb6b4f	Improvements to bundling aggregations. (#11815) ... This is some odds and ends found during the review of #11791 and while continuing to work in this code: * Return attrs classes instead of dictionaries from some methods to improve type safety. * Call `get_bundled_aggregations` fewer times. * Adds a missing assertion in the tests. * Do not return empty bundled aggregations for an event (preferring to not include the bundle at all, as the docstring states).	3 years ago
David Robertson	d8df8e6c14	Don't print HTTPStatus.* in "Processed..." logs (#11827) ... * Don't print HTTPStatus.* in "Processed..." logs Fixes #11812. See also #7118 and https://github.com/matrix-org/synapse/pull/7188#r401719326 in particular. Co-authored-by: Brendan Abolivier <babolivier@matrix.org>	3 years ago
David Robertson	c5815567a4	Avoid type annotation problems in prom-client (#11834)	3 years ago
Brendan Abolivier	95b3f952fa	Add a config flag to inhibit `M_USER_IN_USE` during registration (#11743) ... This is mostly motivated by the tchap use case, where usernames are automatically generated from the user's email address (in a way that allows figuring out the email address from the username). Therefore, it's an issue if we respond to requests on /register and /register/available with M_USER_IN_USE, because it can potentially leak email addresses (which include the user's real name and place of work). This commit adds a flag to inhibit the M_USER_IN_USE errors that are raised both by /register/available, and when providing a username early into the registration process. This error will still be raised if the user completes the registration process but the username conflicts. This is particularly useful when using modules (https://github.com/matrix-org/synapse/pull/11790 adds a module callback to set the username of users at registration) or SSO, since they can ensure the username is unique. More context is available in the PR that introduced this behaviour to synapse-dinsic: matrix-org/synapse-dinsic#48 - as well as the issue in the matrix-dinsic repo: matrix-org/matrix-dinsic#476	3 years ago
David Robertson	74e4419eb4	Fix another jsonschema typecheck error (#11830) ... Similar to #11817. In `_create_power_level_validator` we - retrieve `validator`. This is a class implementing the `jsonschema.protocols.Validator` interface. In other words, `validator: Type[jsonschema.protocols.Validator]`. - we then create an second validator class by modifying the original `validator`. We return that class, which is also of type `Type[jsonschema.protocols.Validator]`. So the original annotation was incorrect: it claimed we were returning an instance of jsonSchema.Draft7Validator, not the class (or a subclass) itself. (Strictly speaking this is incorrect, because `POWER_LEVELS_SCHEMA` isn't pinned to a particular version of JSON Schema. But there are other complications with the type stubs if you try to fix this; I felt like the change herein was a decent compromise that better expresses intent). (I suspect/hope the typeshed project would welcome an effort to improve the jsonschema stubs. Let's see if I get some spare time.)	3 years ago
Shay	b8bf600700	Check that `gc` method is available before using in `synapse/app/_base` (#11816) ... * add check that gc.freeze is available before calling * newsfragment * lint * Update comment Co-authored-by: Dan Callahan <danc@element.io> Co-authored-by: Dan Callahan <danc@element.io>	3 years ago
Dirk Klimpel	6a72c910f1	Add admin API to get a list of federated rooms (#11658)	3 years ago
Brendan Abolivier	1d5f7b2cc6	Log modules at startup (#11813)	3 years ago
Nick Barrett	b59d285f7c	Db txn set isolation level (#11799) ... Co-authored-by: Brendan Abolivier <babolivier@matrix.org>	3 years ago
Richard van der Hoff	fc8598bc87	Minor updates, and docs, for schema delta files (#11823) ... * Make functions in python deltas optional It's annoying to always have to write stubs for these. * Documentation for delta files * changelog	3 years ago
Dirk Klimpel	0d6cfea9b8	Add admin API to reset connection timeouts for remote server (#11639) ... * Fix get federation status of destination if no error occured	3 years ago
David Robertson	874365fc05	1.51.0	3 years ago
Patrick Cloke	15c2a6a106	Ignore the jsonschema type. (#11817)	3 years ago
Patrick Cloke	02d99f044e	Apply a timeout to reading the body when fetching a file. (#11784) ... This prevents the URL preview code from reading a stream forever.	3 years ago
Patrick Cloke	807efd26ae	Support rendering previews with data: URLs in them (#11767) ... Images which are data URLs will no longer break URL previews and will properly be "downloaded" and thumbnailed.	3 years ago
Andrew Morgan	36f37acf53	1.50.2	3 years ago