Merge pull request #2386 from RedisInsight/be/bugfix/RI-4699_dangerous_rule_not_connected

#RI-4699 - 500 error on connecting to user with -@dangerous rule

Author: vlad-dargel

redisinsight/api/src/modules/database/providers/database-info.provider.spec.ts

@@ -370,6 +370,18 @@ describe('DatabaseInfoProvider', () => {
         nodes: [mockRedisGeneralInfo, mockRedisGeneralInfo],
       });
     });
+    it('should throw an error if no permission to run \'info\' command', async () => {
+      mockIORedisClient.info.mockRejectedValue({
+        message: 'NOPERM this user has no permissions to run the \'info\' command'
+      });
+
+      try {
+        await service.getRedisGeneralInfo(mockIORedisClient);
+        fail('Should throw an error');
+      } catch (err) {
+        expect(err).toBeInstanceOf(ForbiddenException);
+      }
+    });
   });
 
   describe('getMasterEndpoints', () => {

redisinsight/api/src/modules/database/providers/database-info.provider.ts

@@ -216,30 +216,34 @@ export class DatabaseInfoProvider {
   private async getRedisNodeGeneralInfo(
     client: IORedis.Redis,
   ): Promise<RedisDatabaseInfoResponse> {
-    const info = convertRedisInfoReplyToObject(
-      await client.info(),
-    );
-    const serverInfo = info['server'];
-    const memoryInfo = info['memory'];
-    const keyspaceInfo = info['keyspace'];
-    const clientsInfo = info['clients'];
-    const statsInfo = info['stats'];
-    const replicationInfo = info['replication'];
-    const databases = await this.getDatabasesCount(client, keyspaceInfo);
-    return {
-      version: serverInfo?.redis_version,
-      databases,
-      role: get(replicationInfo, 'role') || undefined,
-      totalKeys: this.getRedisNodeTotalKeysCount(keyspaceInfo),
-      usedMemory: parseInt(get(memoryInfo, 'used_memory'), 10) || undefined,
-      connectedClients:
-        parseInt(get(clientsInfo, 'connected_clients'), 10) || undefined,
-      uptimeInSeconds:
-        parseInt(get(serverInfo, 'uptime_in_seconds'), 10) || undefined,
-      hitRatio: this.getRedisHitRatio(statsInfo),
-      cashedScripts: parseInt(get(memoryInfo, 'number_of_cached_scripts'), 10) || undefined,
-      server: serverInfo,
-    };
+    try {
+      const info = convertRedisInfoReplyToObject(
+        await client.info(),
+      );
+      const serverInfo = info['server'];
+      const memoryInfo = info['memory'];
+      const keyspaceInfo = info['keyspace'];
+      const clientsInfo = info['clients'];
+      const statsInfo = info['stats'];
+      const replicationInfo = info['replication'];
+      const databases = await this.getDatabasesCount(client, keyspaceInfo);
+      return {
+        version: serverInfo?.redis_version,
+        databases,
+        role: get(replicationInfo, 'role') || undefined,
+        totalKeys: this.getRedisNodeTotalKeysCount(keyspaceInfo),
+        usedMemory: parseInt(get(memoryInfo, 'used_memory'), 10) || undefined,
+        connectedClients:
+          parseInt(get(clientsInfo, 'connected_clients'), 10) || undefined,
+        uptimeInSeconds:
+          parseInt(get(serverInfo, 'uptime_in_seconds'), 10) || undefined,
+        hitRatio: this.getRedisHitRatio(statsInfo),
+        cashedScripts: parseInt(get(memoryInfo, 'number_of_cached_scripts'), 10) || undefined,
+        server: serverInfo,
+      };
+    } catch (error) {
+      throw catchAclError(error);
+    }
   }
 
   private async getRedisMasterNodesGeneralInfo(