Skip to content

Commit

Permalink
Merge pull request #5894 from Countly/ar2rsawseen/master
Browse files Browse the repository at this point in the history
Fixed logging errors from populator
  • Loading branch information
ar2rsawseen authored Jan 8, 2025
2 parents c23b57b + 60563a4 commit c3ad30a
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions api/utils/requestProcessor.js
100755 → 100644
Original file line number Diff line number Diff line change
Expand Up @@ -3193,7 +3193,7 @@ const processBulkRequest = (i, requests, params) => {
const appKey = params.qstring.app_key;
if (i === requests.length) {
common.unblockResponses(params);
if (params.qstring.safe_api_response || plugins.getConfig("api", params.app && params.app.plugins, true).safe && !params.res.finished) {
if ((params.qstring.safe_api_response || plugins.getConfig("api", params.app && params.app.plugins, true).safe) && !params.res.finished) {
common.returnMessage(params, 200, 'Success');
}
return;
Expand All @@ -3220,7 +3220,8 @@ const processBulkRequest = (i, requests, params) => {
'req': params.req,
'promises': [],
'bulk': true,
'populator': params.qstring.populator
'populator': params.qstring.populator,
'blockResponses': true
};

tmpParams.qstring.app_key = (requests[i].app_key || appKey) + "";
Expand Down

0 comments on commit c3ad30a

Please sign in to comment.