chore: remove unneeded try catch block from getServerSetInRoom

Author: ricardogarim

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

@@ -784,38 +784,28 @@ export class StateService {
 	// }
 	async getServerSetInRoom(roomId: string) {
 		const servers = new Set<string>();
+		const state = await this.getLatestRoomState(roomId);
 
-		try {
-			const state = await this.getLatestRoomState(roomId);
-
-			for (const event of state.values()) {
-				if (!event.isMembershipEvent() || event.getMembership() !== 'join') {
-					continue;
-				}
+		for (const event of state.values()) {
+			if (!event.isMembershipEvent() || event.getMembership() !== 'join') {
+				continue;
+			}
 
-				try {
-					const server = extractDomainFromId(event.stateKey as string);
-					if (server) {
-						servers.add(server);
-					}
-				} catch (error) {
-					this.logger.error({
-						err: error,
-						eventId: event.eventId,
-						msg: 'error extracting server',
-					});
+			try {
+				const server = extractDomainFromId(event.stateKey as string);
+				if (server) {
+					servers.add(server);
 				}
+			} catch (error) {
+				this.logger.error({
+					err: error,
+					eventId: event.eventId,
+					msg: 'error extracting server',
+				});
 			}
-
-			return servers;
-		} catch (error) {
-			this.logger.error({
-				err: error,
-				roomId: roomId,
-				msg: 'error getting server set in room',
-			});
-			return servers;
 		}
+
+		return servers;
 	}
 
 	// @deprecated use getServerSetInRoom