fix: Log failed sync message that didn't merge (#576)

This commit is contained in:
adityapk00
2023-02-16 10:32:54 -08:00
committed by GitHub
parent 9a3807d135
commit daeda43ed9

View File

@@ -289,10 +289,22 @@ class SyncEngine extends TypedEmitter<SyncEvents> {
}
log.info(
{ messages: mergeResults.length, success: mergeResults.filter((r) => r.isOk()).length },
{
total: mergeResults.length,
success: mergeResults.filter((r) => r.isOk()).length,
},
'Merged messages'
);
// If there was a failed merge, log the error and move on. We'll only log one error, since they're likely all the same.
const failedMerge = mergeResults.find((r) => r.isErr());
if (failedMerge) {
log.warn(
{ error: failedMerge._unsafeUnwrapErr(), errorMessage: failedMerge._unsafeUnwrapErr().message },
'Failed to merge message'
);
}
return mergeResults;
}