Skip to content

Commit 700c957

Browse files
author
Artem
committed
#RI-4718 - fix unhandled error issue in recommendations
1 parent c95dde4 commit 700c957

File tree

1 file changed

+25
-20
lines changed

1 file changed

+25
-20
lines changed

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

Lines changed: 25 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -61,28 +61,33 @@ export class DatabaseRecommendationService {
6161
recommendationName: string,
6262
data: any,
6363
): Promise<DatabaseRecommendation> {
64-
const newClientMetadata = {
65-
...clientMetadata,
66-
db: clientMetadata.db ?? (await this.databaseService.get(clientMetadata.databaseId))?.db ?? 0,
67-
};
68-
const isRecommendationExist = await this.databaseRecommendationRepository.isExist(
69-
newClientMetadata,
70-
recommendationName,
71-
);
72-
if (!isRecommendationExist) {
73-
const recommendation = await this.scanner.determineRecommendation(recommendationName, data);
74-
75-
if (recommendation) {
76-
const entity = plainToClass(
77-
DatabaseRecommendation,
78-
{ databaseId: newClientMetadata?.databaseId, ...recommendation },
79-
);
80-
81-
return await this.create(newClientMetadata, entity);
64+
try {
65+
const newClientMetadata = {
66+
...clientMetadata,
67+
db: clientMetadata.db ?? (await this.databaseService.get(clientMetadata.databaseId))?.db ?? 0,
68+
};
69+
const isRecommendationExist = await this.databaseRecommendationRepository.isExist(
70+
newClientMetadata,
71+
recommendationName,
72+
);
73+
if (!isRecommendationExist) {
74+
const recommendation = await this.scanner.determineRecommendation(recommendationName, data);
75+
76+
if (recommendation) {
77+
const entity = plainToClass(
78+
DatabaseRecommendation,
79+
{ databaseId: newClientMetadata?.databaseId, ...recommendation },
80+
);
81+
82+
return await this.create(newClientMetadata, entity);
83+
}
8284
}
83-
}
8485

85-
return null;
86+
return null;
87+
} catch (e) {
88+
this.logger.warn('Unable to check recommendation', e);
89+
return null;
90+
}
8691
}
8792

8893
/**

0 commit comments

Comments
 (0)