Skip to content

default push notifications to be noisy#6277

Closed
Reaster0 wants to merge 1 commit intomatrix-org:mainfrom
Reaster0:default-noisy-push
Closed

default push notifications to be noisy#6277
Reaster0 wants to merge 1 commit intomatrix-org:mainfrom
Reaster0:default-noisy-push

Conversation

@Reaster0
Copy link

some servers (continuwuity) don't send any actions/tweaks in the push notifications after push registration using the flag event_id_only that discard them (just as synapse btw),

in that case they send push notifications without filling the "tweaks" field https://spec.matrix.org/v1.17/push-gateway-api/#post_matrixpushv1notify_request_device

other clients like fluffychat, element classic, etc default this situation to noisy notifications but element x default to silent (because of the behaviour of this lib) in that case there's no way to get noisy notifications on element x and continuwuity,

more informations about this issue can be found there:
element-hq/element-x-ios#5132
https://forgejo.ellis.link/continuwuation/continuwuity/issues/1533
https://forgejo.ellis.link/continuwuation/continuwuity/issues/1424

@Reaster0 Reaster0 requested a review from a team as a code owner March 11, 2026 21:52
@Reaster0 Reaster0 requested review from poljar and removed request for a team March 11, 2026 21:52
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 11, 2026

Merging this PR will not alter performance

✅ 50 untouched benchmarks


Comparing Reaster0:default-noisy-push (cd37aa1) with main (791f4bc)

Open in CodSpeed

@BillCarsonFr
Copy link
Member

@Reaster0
Can you provide the default push rules set created by continuwuity? just to check if there is a difference with synapse

@BillCarsonFr
Copy link
Member

There are some ongoing investigation regarding that here element-hq/element-x-ios#5132 (comment)
Apparently continuwuity and synapse use slightly different default pushrules. This can explain alone the difference of behavior. This needs more investigations and maybe spec clarifications

@Reaster0
Copy link
Author

better/cleaner pr that include the tests here: #6281

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.

2 participants