Merge pull request #1019 from transitland/sentry-context

when logging exceptions to Sentry, include context
transitland · Mar 10, 2017 · d219956 · d219956
2 parents b16d73f + 6ff785b
commit d219956
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 3 deletions.
diff --git a/app/workers/feed_eater_schedule_worker.rb b/app/workers/feed_eater_schedule_worker.rb
@@ -23,7 +23,14 @@ def perform(feed_onestop_id, feed_version_sha1, feed_schedule_import_id, trip_id
       exception_log = "\n#{e}\n#{e.backtrace}\n"
       log exception_log, :error
       feed_schedule_import.failed(exception_log: exception_log)
-      Raven.capture_exception(e) if defined?(Raven)
+      if defined?(Raven)
+        Raven.capture_exception(e, {
+          tags: {
+            'feed_onestop_id' => feed_onestop_id,
+            'feed_version_sha1' => feed_version_sha1
+          }
+        })
+      end
     else
       log "FeedEaterScheduleWorker #{feed_onestop_id}: Saving successful schedule import"
       feed_schedule_import.succeeded

diff --git a/app/workers/feed_eater_worker.rb b/app/workers/feed_eater_worker.rb
@@ -48,7 +48,14 @@ def perform(feed_onestop_id, feed_version_sha1=nil, import_level=0)
       log exception_log, :error
       log "FeedEaterWorker #{feed_onestop_id}: Saving failed feed import"
       feed_version_import.failed(exception_log: exception_log)
-      Raven.capture_exception(e) if defined?(Raven)
+      if defined?(Raven)
+        Raven.capture_exception(e, {
+          tags: {
+            'feed_onestop_id' => feed_onestop_id,
+            'feed_version_sha1' => feed_version.try(:sha1)
+          }
+        })
+      end
     else
       # Enqueue FeedEaterScheduleWorker jobs, or save successful import.
       if import_level < 2

diff --git a/app/workers/feed_fetcher_worker.rb b/app/workers/feed_fetcher_worker.rb
@@ -19,7 +19,13 @@ def perform(feed_onestop_id)
       #   SignalException, and SyntaxError
       log e.message, :error
       log e.backtrace, :error
-      Raven.capture_exception(e) if defined?(Raven)
+      if defined?(Raven)
+        Raven.capture_exception(e, {
+          tags: {
+            'feed_onestop_id' => feed_onestop_id
+          }
+        })
+      end
     end
   end
 end