Skip to content

Commit d9cc2c9

Browse files
samikshya-dbclaude
andcommitted
Add connection.close event listener to telemetry wire-up
Fix missing event listener for CONNECTION_CLOSE events in DBSQLClient telemetry initialization. Without this listener, connection close events were being emitted but not routed to the aggregator for processing. Now all 3 telemetry events are properly exported: - CONNECTION_OPEN (connection latency) - STATEMENT_COMPLETE (execution latency) - CONNECTION_CLOSE (session duration) Verified with e2e test showing 3 successful telemetry exports. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent e7f2144 commit d9cc2c9

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

lib/DBSQLClient.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -346,6 +346,14 @@ export default class DBSQLClient extends EventEmitter implements IDBSQLClient, I
346346
}
347347
});
348348

349+
this.telemetryEmitter.on('connection.close', (event) => {
350+
try {
351+
this.telemetryAggregator?.processEvent(event);
352+
} catch (error: any) {
353+
this.logger.log(LogLevel.debug, `Error processing connection.close event: ${error.message}`);
354+
}
355+
});
356+
349357
this.telemetryEmitter.on('statement.start', (event) => {
350358
try {
351359
this.telemetryAggregator?.processEvent(event);

0 commit comments

Comments
 (0)