Skip to content

Commit f4f66d1

Browse files
samikshya-dbclaude
andcommitted
Fix telemetry and feature flag implementation
- Fix event listener names: use 'connection.open' not 'telemetry.connection.open' - Fix feature flag endpoint: use NODEJS client type instead of OSS_NODEJS - Fix telemetry endpoints: use /telemetry-ext and /telemetry-unauth (not /api/2.0/sql/...) - Update telemetry payload to match proto: use system_configuration with snake_case fields - Add URL utility to handle hosts with or without protocol - Add telemetryBatchSize and telemetryAuthenticatedExport config options - Remove debug statements and temporary feature flag override Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 981936e commit f4f66d1

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

lib/DBSQLClient.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,7 @@ export default class DBSQLClient extends EventEmitter implements IDBSQLClient, I
290290

291291
// Check if telemetry enabled via feature flag
292292
const enabled = await this.featureFlagCache.isTelemetryEnabled(this.host);
293+
293294
if (!enabled) {
294295
this.logger.log(LogLevel.debug, 'Telemetry disabled via feature flag');
295296
return;

0 commit comments

Comments
 (0)