Fixes unhandled promise rejection error while sending push (#4172)

This commit is contained in:
Florent Vilmart
2017-09-17 09:57:07 -04:00
committed by GitHub
parent 8d8a8b250e
commit ba0a51ddc9
2 changed files with 6 additions and 3 deletions

View File

@@ -19,7 +19,9 @@ export class PushRouter extends PromiseRouter {
const promise = new Promise((_resolve) => {
resolve = _resolve;
});
pushController.sendPush(req.body, where, req.config, req.auth, (pushStatusId) => {
let pushStatusId;
pushController.sendPush(req.body, where, req.config, req.auth, (objectId) => {
pushStatusId = objectId;
resolve({
headers: {
'X-Parse-Push-Status-Id': pushStatusId
@@ -28,7 +30,9 @@ export class PushRouter extends PromiseRouter {
result: true
}
});
}).catch(req.config.loggerController.error);
}).catch((err) => {
req.config.loggerController.error(`_PushStatus ${pushStatusId}: error while sending push`, err);
});
return promise;
}

View File

@@ -247,7 +247,6 @@ export function pushStatusHandler(config, objectId = newObjectId(config.objectId
status: 'failed',
updatedAt: new Date()
}
logger.warn(`_PushStatus ${objectId}: error while sending push`, err);
return handler.update({ objectId }, update);
}