Merge pull request #127 from cloudflare/log-onclose-events

Add logging for onClose
cloudflare · Oct 4, 2024 · 78c8447 · 78c8447
2 parents 036b9d6 + ea5c893
commit 78c8447
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/app/durableObjects/ChatRoom.server.ts b/app/durableObjects/ChatRoom.server.ts
@@ -194,6 +194,23 @@ export class ChatRoom extends Server<Env> {
 		}
 	}
 
+	async onClose(
+		connection: Connection,
+		code: number,
+		reason: string,
+		wasClean: boolean
+	) {
+		const meetingId = await this.getMeetingId()
+		log({
+			eventName: 'onClose',
+			meetingId,
+			connectionId: connection.id,
+			code,
+			reason,
+			wasClean,
+		})
+	}
+
 	async onMessage(
 		connection: Connection<User>,
 		message: WSMessage

diff --git a/app/utils/logging.ts b/app/utils/logging.ts
@@ -13,6 +13,14 @@ export type LogEvent =
 			foundInStorage: boolean
 			connectionId: string
 	  }
+	| {
+			eventName: 'onClose'
+			meetingId?: string
+			connectionId: string
+			code: number
+			reason: string
+			wasClean: boolean
+	  }
 	| {
 			eventName: 'userLeft'
 			meetingId?: string