#RI-4927 - [BE] resp, lib-name, lib-version are sent for db with redis version <7.0

egor-zalenski · egor-zalenski · commit c8d762b88def · 2023-09-07T14:13:33.000+02:00
diff --git a/redisinsight/api/src/modules/database/database-connection.service.ts b/redisinsight/api/src/modules/database/database-connection.service.ts
@@ -162,8 +162,13 @@ export class DatabaseConnectionService {
     }
   }
 
-  private async collectClientInfo(clientMetadata: ClientMetadata, client: any, version?: string) {
+  private async collectClientInfo(clientMetadata: ClientMetadata, client: any, version: string) {
     try {
+      const intVersion = parseInt(version, 10) || 0;
+      if (intVersion < 7) {
+        return
+      }
+
       const clients = await this.databaseInfoProvider.getClientListInfo(client) || [];
 
       this.analytics.sendDatabaseConnectedClientListEvent(

Original file line number	Diff line number	Diff line change
`@@ -162,8 +162,13 @@ export class DatabaseConnectionService {`
`162`	`162`	`}`
`163`	`163`	`}`
`164`	`164`
`165`		`- private async collectClientInfo(clientMetadata: ClientMetadata, client: any, version?: string) {`
	`165`	`+ private async collectClientInfo(clientMetadata: ClientMetadata, client: any, version: string) {`
`166`	`166`	`try {`
	`167`	`+ const intVersion = parseInt(version, 10) \|\| 0;`
	`168`	`+ if (intVersion < 7) {`
	`169`	`+ return`
	`170`	`+ }`
	`171`	`+`
`167`	`172`	`const clients = await this.databaseInfoProvider.getClientListInfo(client) \|\| [];`
`168`	`173`
`169`	`174`	`this.analytics.sendDatabaseConnectedClientListEvent(`