matching: don't let one FeedEntity's matching failure interrupt all others 🐛, add logging

Author: derhuerst

lib/match.js

@@ -116,15 +116,28 @@ const createParseAndProcessFeed = (cfg) => {
 					feedEntity,
 				}, 'matching FeedEntity')
 
-				if (feedEntity.trip_update) {
-					await matchTripUpdate(feedEntity.trip_update, {now})
-				}
-				if (feedEntity.vehicle) {
-					await matchVehiclePosition(feedEntity.vehicle)
-				}
-				if (feedEntity.alert) {
-					await matchAlert(feedEntity.alert)
+				try {
+					if (feedEntity.trip_update) {
+						await matchTripUpdate(feedEntity.trip_update, {now})
+					}
+					if (feedEntity.vehicle) {
+						await matchVehiclePosition(feedEntity.vehicle)
+					}
+					if (feedEntity.alert) {
+						await matchAlert(feedEntity.alert)
+					}
+				} catch (err) {
+					_logger.info({
+						..._logCtx,
+						error: err,
+						feedEntitiesIdx,
+					}, 'failed to match FeedEntity')
+					return; // suppress errors, to let other parallel matchFeedEntity() calls keep running
 				}
+				_logger.trace({
+					..._logCtx,
+					feedEntitiesIdx,
+				}, 'successfully matched FeedEntity')
 			}
 
 			return runConcurrenctly(matchFeedEntity)