add TODO

Author: sampaiodiego

packages/federation-sdk/src/services/edu.service.ts

@@ -24,6 +24,7 @@ export class EduService {
 
 			this.logger.debug(`Sending typing notification for room ${roomId}: ${userId} (typing: ${typing}) to all servers in room`);
 
+			// TODO we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.
 			const servers = await this.stateService.getServerSetInRoom(roomId);
 			const uniqueServers = Array.from(servers).filter((server) => server !== origin);
 
@@ -49,6 +50,7 @@ export class EduService {
 
 			await Promise.all(
 				roomIds.map(async (roomId) => {
+					// TODO we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.
 					const servers = await this.stateService.getServerSetInRoom(roomId);
 					for (const server of servers) {
 						if (server !== origin) {
@@ -104,6 +106,7 @@ export class EduService {
 				`Sending read receipt for user ${userId} in room ${roomId} for events ${eventIds.join(', ')} to all servers in room`,
 			);
 
+			// TODO we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.
 			const servers = await this.stateService.getServerSetInRoom(roomId);
 			const uniqueServers = Array.from(servers).filter((server) => server !== origin);
 

packages/federation-sdk/src/services/event-authorization.service.ts

@@ -249,6 +249,7 @@ export class EventAuthorizationService {
 			return false;
 		}
 
+		// TODO we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.
 		const serversInRoom = await this.stateService.getServerSetInRoom(roomId, state);
 		if (serversInRoom.has(serverName)) {
 			this.logger.debug(`Server ${serverName} is in room, allowing access`);

packages/federation-sdk/src/services/federation.service.ts

@@ -197,6 +197,7 @@ export class FederationService {
 	}
 
 	async sendEventToAllServersInRoom(event: PersistentEventBase) {
+		// TODO we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.
 		const servers = await this.stateService.getServerSetInRoom(event.roomId);
 
 		if (event.stateKey) {