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

FInd more efficient way to check room memberships for appservices #468

Open
anoadragon453 opened this issue May 25, 2018 · 2 comments
Open

FInd more efficient way to check room memberships for appservices #468

anoadragon453 opened this issue May 25, 2018 · 2 comments
Labels
C-App-Services T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. X-Performance Issue/PR around something that is slow or taking lots of memory
Milestone
Application Services

Comments

@anoadragon453
Copy link
Member

Currently app services will get events related to their namespace from any room on the homeserver, even if they are not part of that room yet. This isn't really intended behavior, and thus should be scoped to only the room the app service is a part of.
@anoadragon453 anoadragon453 added this to the Application Services milestone May 25, 2018
@turt2live turt2live mentioned this issue Jun 2, 2018
Closed
@Half-Shot
Copy link
Contributor

I don't believe this is still true, do you think @neilalexander ?

@neilalexander neilalexander changed the title AppServices: Check whether app service is part of a room before sending room events to it FInd more efficient way to check room memberships for appservices Mar 3, 2021
@neilalexander
Copy link
Contributor

We now have this as of #1680 but the process isn't amazing so we'd like to find a better/faster way to do it.

@kegsay kegsay added X-Performance Issue/PR around something that is slow or taking lots of memory T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. labels Dec 5, 2022
This was referenced Oct 30, 2024
Closed
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
C-App-Services T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. X-Performance Issue/PR around something that is slow or taking lots of memory
Projects
None yet
Milestone
Application Services
Development

No branches or pull requests
4 participants
@neilalexander @anoadragon453 @Half-Shot @kegsay