Skip to content

Commit 17025b9

Browse files
author
Artem
committed
#RI-3907 - close opened connection after analysis performed
1 parent dd00d7a commit 17025b9

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

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

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,10 @@ export class DatabaseAnalysisService {
2929
clientMetadata: ClientMetadata,
3030
dto: CreateDatabaseAnalysisDto,
3131
): Promise<DatabaseAnalysis> {
32+
let client;
33+
3234
try {
33-
const client = await this.databaseConnectionService.createClient(clientMetadata);
35+
client = await this.databaseConnectionService.createClient(clientMetadata);
3436

3537
const scanResults = await this.scanner.scan(client, {
3638
filter: dto.filter,
@@ -54,8 +56,10 @@ export class DatabaseAnalysisService {
5456
progress,
5557
}, [].concat(...scanResults.map((nodeResult) => nodeResult.keys))));
5658

59+
client.disconnect();
5760
return this.databaseAnalysisProvider.create(analysis);
5861
} catch (e) {
62+
client?.disconnect();
5963
this.logger.error('Unable to analyze database', e);
6064

6165
if (e instanceof HttpException) {

0 commit comments

Comments
 (0)