Merge pull request #72 from cloudflare/update-deps

update deps, log muteUser warning correctly
cloudflare · Aug 8, 2024 · 722b554 · 722b554
2 parents d06530b + 11a9138
commit 722b554
Show file tree

Hide file tree

Showing 4 changed files with 526 additions and 405 deletions.
diff --git a/app/durableObjects/ChatRoom.server.ts b/app/durableObjects/ChatRoom.server.ts
@@ -125,26 +125,32 @@ export class ChatRoom extends Server<Env> {
 					break
 
 				case 'muteUser':
-					for (const otherConnection of this.getConnections<User>()) {
-						if (otherConnection.id === data.id) {
-							otherConnection.setState({
-								...otherConnection.state!,
-								tracks: {
-									...otherConnection.state!.tracks,
-									audioEnabled: false,
-								},
-							})
-							this.sendMessage(otherConnection, {
-								type: 'muteMic',
-							})
-
-							this.broadcastState()
-							break
+					{
+						let mutedUser = false
+						for (const otherConnection of this.getConnections<User>()) {
+							if (otherConnection.id === data.id) {
+								otherConnection.setState({
+									...otherConnection.state!,
+									tracks: {
+										...otherConnection.state!.tracks,
+										audioEnabled: false,
+									},
+								})
+								this.sendMessage(otherConnection, {
+									type: 'muteMic',
+								})
+
+								this.broadcastState()
+								mutedUser = true
+								break
+							}
+						}
+						if (!mutedUser) {
+							console.warn(
+								`User with id "${data.id}" not found, cannot mute user from "${connection.state!.name}"`
+							)
 						}
 					}
-					console.warn(
-						`User with id "${data.id}" not found, cannot mute user from "${connection.state!.name}"`
-					)
 
 					break
 				case 'partyserver-ping':