Skip to content

Commit 3754c58

Browse files
Merge pull request #2558 from RedisInsight/be/feature/RI-4927-telemetry_event_sent_if_redis<7
#RI-4927 - [BE] resp, lib-name, lib-version are sent for db with redi…
2 parents 6761bd0 + 5964cbe commit 3754c58

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

redisinsight/api/src/modules/database/database-connection.service.ts

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -165,20 +165,16 @@ export class DatabaseConnectionService {
165165
private async collectClientInfo(clientMetadata: ClientMetadata, client: any, version?: string) {
166166
try {
167167
const intVersion = parseInt(version, 10) || 0;
168-
if (intVersion < 7) {
169-
return
170-
}
171-
172168
const clients = await this.databaseInfoProvider.getClientListInfo(client) || [];
173169

174170
this.analytics.sendDatabaseConnectedClientListEvent(
175171
clientMetadata.databaseId,
176172
{
177173
clients: clients.map((c) => ({
178174
version: version || 'n/a',
179-
resp: c?.['resp'] || 'n/a',
180-
libName: c?.['lib-name'] || 'n/a',
181-
libVer: c?.['lib-ver'] || 'n/a',
175+
resp: intVersion < 7 ? undefined : c?.['resp'] || 'n/a',
176+
libName: intVersion < 7 ? undefined : c?.['lib-name'] || 'n/a',
177+
libVer: intVersion < 7 ? undefined : c?.['lib-ver'] || 'n/a',
182178
})),
183179
},
184180
);

0 commit comments

Comments
 (0)