Skip to content

fix: preserve encrypted room mention notifications#3160

Open
PolynomialDivision wants to merge 1 commit into
krille-chan:mainfrom
PolynomialDivision:fixpushnotiications
Open

fix: preserve encrypted room mention notifications#3160
PolynomialDivision wants to merge 1 commit into
krille-chan:mainfrom
PolynomialDivision:fixpushnotiications

Conversation

@PolynomialDivision

Copy link
Copy Markdown
Contributor

MSC4028 (push all encrypted events) is still experimental and disabled on most homeservers including matrix.org. Without it, a room-level mentions-only rule blocks all server pushes for encrypted events, since the server cannot inspect ciphertext to evaluate mention rules. The device never wakes up, so no mention notification reaches the user.

To work around this, install a companion override push rule for each encrypted room in mentions-only mode. The override matches m.room.encrypted events server-side, waking the device. After decryption the event type changes, so the override no longer matches client-side and the room rule correctly filters out non-mentions.

Pull Request has been tested on:

  • Android
  • iOS
  • Browser (Chromium based)
  • Browser (Firefox based)
  • Browser (WebKit based)
  • Desktop Linux
  • Desktop Windows
  • Desktop macOS

MSC4028 (push all encrypted events) is still experimental and disabled
on most homeservers including matrix.org. Without it, a room-level
mentions-only rule blocks all server pushes for encrypted events, since
the server cannot inspect ciphertext to evaluate mention rules. The
device never wakes up, so no mention notification reaches the user.

To work around this, install a companion override push rule for each
encrypted room in mentions-only mode. The override matches
m.room.encrypted events server-side, waking the device. After decryption
the event type changes, so the override no longer matches client-side and
the room rule correctly filters out non-mentions.
@PolynomialDivision PolynomialDivision marked this pull request as ready for review June 26, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant