Merge branch 'rs-sidekiq-sentry-context' into 'master'

Add a `sidekiq` tag to Sentry Raven context This will let us filter errors that were/were not in the Sidekiq environment. Source: http://stackoverflow.com/a/28370539/223897 Closes #15092 See merge request !3634

Merge branch 'rs-sidekiq-sentry-context' into 'master'
dfe4c69e · Stan Hu · a2303417 · 2378ec0d · dfe4c69e
Commit dfe4c69e authored Apr 10, 2016 by Stan Hu
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 0 deletions

application_controller.rb app/controllers/application_controller.rb +10 -0

No files found.
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -47,6 +47,16 @@ class ApplicationController < ActionController::Base
        email: current_user.email,
        username: current_user.username,
      )
+
+      Raven.tags_context(program: sentry_program_context)
+    end
+  end
+
+  def sentry_program_context
+    if Sidekiq.server?
+      'sidekiq'
+    else
+      'rails'
    end
  end