Skip to content

Conversation

@reivilibre
Copy link
Contributor

@reivilibre reivilibre commented Aug 19, 2025

Follows: #18762

Implements: MSC4306

Closes: #18431
Closes: #18437

  1. Move the MSC4306 push rules to a new kind postcontent

  2. Prevent users from creating user-defined postcontent rules

@reivilibre reivilibre force-pushed the rei/msc4306_postcontent branch from 49ad465 to 8821d72 Compare August 19, 2025 10:48
@reivilibre reivilibre changed the title Update push rules for experimental [MSC4306: Thread Subscriptions](https://github.com/matrix-org/matrix-doc/issues/4306) to follow newer draft. Update push rules for experimental MSC4306: Thread Subscriptions to follow newer draft. Aug 19, 2025
@reivilibre reivilibre force-pushed the rei/msc4306_postcontent branch from 8821d72 to 0aaab37 Compare August 19, 2025 12:42
@reivilibre reivilibre marked this pull request as ready for review August 19, 2025 15:48
@reivilibre reivilibre requested a review from a team as a code owner August 19, 2025 15:48
@reivilibre reivilibre force-pushed the rei/msc4306_postcontent branch from 0aaab37 to 3de8c21 Compare August 20, 2025 15:29
Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should any new tests be added to check that the new priority order works as intended?

@bnjbvr
Copy link
Member

bnjbvr commented Aug 27, 2025

Should any new tests be added to check that the new priority order works as intended?

For what it's worth: I've added a few tests like that in the Rust SDK 1, which implements this MSC. The tests are checking that the subscribed-to threads will or will not generate notifications, for all combinations of "all messages" / "mentions only" / "mute", set globally or on a per-room basis.

For "mute", as expected, these produce no notifications in all cases.

What the new push rules fix, is that when a room or global is set to "mentions only", subscribed threads will trigger notifications (while they didn't, before this change).

Footnotes

  1. https://github.com/matrix-org/matrix-rust-sdk/pull/5565 shows the behavior before these changes to push rules.

@reivilibre reivilibre merged commit c68c5dd into develop Sep 9, 2025
22 of 23 checks passed
@reivilibre reivilibre deleted the rei/msc4306_postcontent branch September 9, 2025 17:37
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Oct 8, 2025
# Synapse 1.139.0 (2025-09-30)

## Features

- Add experimental support for [MSC4308: Thread Subscriptions extension to Sliding Sync](matrix-org/matrix-spec-proposals#4308) when [MSC4306: Thread Subscriptions](matrix-org/matrix-spec-proposals#4306) and [MSC4186: Simplified Sliding Sync](matrix-org/matrix-spec-proposals#4186) are enabled. ([\#18695](element-hq/synapse#18695))
- Update push rules for experimental [MSC4306: Thread Subscriptions](matrix-org/matrix-spec-proposals#4306) to follow a newer draft. ([\#18846](element-hq/synapse#18846))
- Add `get_media_upload_limits_for_user` and `on_media_upload_limit_exceeded` module API callbacks to the media repository. ([\#18848](element-hq/synapse#18848))
- Support [MSC4169](matrix-org/matrix-spec-proposals#4169) for backwards-compatible redaction sending using the `/send` endpoint. Contributed by @SpiritCroc @ Beeper. ([\#18898](element-hq/synapse#18898))
- Add an in-memory cache to `_get_e2e_cross_signing_signatures_for_devices` to reduce DB load. ([\#18899](element-hq/synapse#18899))
- Update [MSC4190](matrix-org/matrix-spec-proposals#4190) support to return correct errors and allow appservices to reset cross-signing keys without user-interactive authentication. Contributed by @tulir @ Beeper. ([\#18946](element-hq/synapse#18946))

## Deprecations and Removals

- Remove obsolete and experimental `/sync/e2ee` endpoint. ([\#18583](element-hq/synapse#18583))

# Synapse 1.138.0 (2025-09-09)

## Features

- Support for the stable endpoint and scopes of [MSC3861](matrix-org/matrix-spec-proposals#3861) & co. ([\#18549](element-hq/synapse#18549))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants