Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Option to only share presence with members of rooms with m.direct #11977

Open
svenpaulsen opened this issue Feb 11, 2022 · 3 comments
Open

Option to only share presence with members of rooms with m.direct #11977

svenpaulsen opened this issue Feb 11, 2022 · 3 comments
Labels
A-Presence P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches roadmap T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@svenpaulsen
Copy link

svenpaulsen commented Feb 11, 2022

As you know, presence updates have a major impact on the overall server performance... but just turning it off is not a real solution in most use-cases. Therefore, I'm suggesting a new option in the presence configuration called limit_to_direct_rooms (or something like that).

The idea is, that when collecting the list of interested parties for presence updates, only room ids with a m.direct membership should be used. As a result, users would retrieve presence updates for their "contacts", but not for everyone who's sharing a group chat room. This could help decreasing the server load and increase privacy at the same time.

What do you think? Could this be a reasonable solution?

@babolivier babolivier added the T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. label Feb 14, 2022
@3nprob
Copy link
Contributor

3nprob commented Mar 27, 2022

This is a great proposal. Would even argue that it'd be a sane future default for privacy reasons.

Related: #8055 #10423

@richvdh richvdh added P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches A-Presence labels Aug 1, 2022
@svenpaulsen
Copy link
Author

svenpaulsen commented Dec 6, 2022

Any feedback on this? Presence is still a major performance/privacy issue, but disabling it completely is not an option for us. Haven't looked at Dendrite yet. Would that be worth a try?

@pmaier1 pmaier1 added the roadmap label Jun 2, 2023
@svenpaulsen
Copy link
Author

svenpaulsen commented Jul 28, 2023

We've initiated the implementation of this feature in our system and preliminary testing suggests a substantial reduction in both database usage and CPU load. You can track our ongoing efforts at the following link:

https://github.com/teamspeak/synapse

Our upcoming work will focus on introducing a user-configurable option for this feature. Upon completion, we will share these improvements by submitting a pull request.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Presence P4 (OBSOLETE: use S- labels.) Okay backlog: will not schedule, will accept patches roadmap T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
Development

No branches or pull requests

5 participants