Преглед изворни кода

also block "delete component" messages from sentry

Brian Gough пре 8 година
родитељ
комит
37f431f148
1 измењених фајлова са 7 додато и 2 уклоњено
  1. 7 2
      services/document-updater/app/coffee/DispatchManager.coffee

+ 7 - 2
services/document-updater/app/coffee/DispatchManager.coffee

@@ -25,8 +25,13 @@ module.exports = DispatchManager =
 					backgroundTask = (cb) ->
 					backgroundTask = (cb) ->
 						UpdateManager.processOutstandingUpdatesWithLock project_id, doc_id, (error) ->
 						UpdateManager.processOutstandingUpdatesWithLock project_id, doc_id, (error) ->
 							# log everything except OpRangeNotAvailable errors, these are normal
 							# log everything except OpRangeNotAvailable errors, these are normal
-							if error? and not (error instanceof Errors.OpRangeNotAvailableError)
-								logger.error err: error, project_id: project_id, doc_id: doc_id, "error processing update"
+							if error? 
+								# downgrade OpRangeNotAvailable and "Delete component" errors so they are not sent to sentry
+								logAsWarning = (error instanceof Errors.OpRangeNotAvailableError) || error.message?.match(/^Delete component/)
+								if logAsWarning
+									logger.warn err: error, project_id: project_id, doc_id: doc_id, "error processing update"
+								else
+									logger.error err: error, project_id: project_id, doc_id: doc_id, "error processing update"
 							cb()
 							cb()
 					RateLimiter.run backgroundTask, callback
 					RateLimiter.run backgroundTask, callback