Merge branch 'revert-sentry-js' into 'master'

Revert "Merge branch 'sentry-js' into 'master'"

This reverts commit 0f309794, reversing
changes made to 1e596fef.

See merge request !3851

CHANGELOG

@@ -90,7 +90,6 @@ v 8.7.0 (unreleased)
   - Fix repository cache invalidation issue when project is recreated with an empty repo (Stan Hu)
   - Fix: Allow empty recipients list for builds emails service when pushed is added (Frank Groeneveld)
   - Improved markdown forms
-  - Show JavaScript errors in sentry
   - Diff design updates (colors, button styles, etc)
   - Copying and pasting a diff no longer pastes the line numbers or +/-
   - Add null check to formData when updating profile content to fix Firefox bug

app/assets/javascripts/application.js.coffee

@@ -55,7 +55,6 @@
 #= require_tree .
 #= require fuzzaldrin-plus
 #= require cropper
-#= require raven
 
 window.slugify = (text) ->
   text.replace(/[^-a-zA-Z0-9]+/g, '_').toLowerCase()

app/assets/javascripts/raven_config.js.coffee

-@raven =
-  init: ->
-    if gon.sentry_dsn?
-      Raven.config(gon.sentry_dsn, {
-        includePaths: [/gon.relative_url_root/]
-        ignoreErrors: [
-          # Random plugins/extensions
-          'top.GLOBALS',
-          # See: http://blog.errorception.com/2012/03/tale-of-unfindable-js-error. html
-          'originalCreateNotification',
-          'canvas.contentDocument',
-          'MyApp_RemoveAllHighlights',
-          'http://tt.epicplay.com',
-          'Can\'t find variable: ZiteReader',
-          'jigsaw is not defined',
-          'ComboSearch is not defined',
-          'http://loading.retry.widdit.com/',
-          'atomicFindClose',
-          # ISP "optimizing" proxy - `Cache-Control: no-transform` seems to
-          # reduce this. (thanks @acdha)
-          # See http://stackoverflow.com/questions/4113268
-          'bmi_SafeAddOnload',
-          'EBCallBackMessageReceived',
-          # See http://toolbar.conduit.com/Developer/HtmlAndGadget/Methods/JSInjection.aspx
-          'conduitPage'
-        ],
-        ignoreUrls: [
-          # Chrome extensions
-          /extensions\//i,
-          /^chrome:\/\//i,
-          # Other plugins
-          /127\.0\.0\.1:4001\/isrunning/i,  # Cacaoweb
-          /webappstoolbarba\.texthelp\.com\//i,
-          /metrics\.itunes\.apple\.com\.edgesuite\.net\//i
-        ]
-      }).install()
-
-    if gon.current_user_id
-      Raven.setUserContext({
-        id: gon.current_user_id
-      })
-
-$ ->
-  raven.init()

lib/gitlab/gon_helper.rb

@@ -7,7 +7,6 @@ module Gitlab
       gon.max_file_size          = current_application_settings.max_attachment_size
       gon.relative_url_root      = Gitlab.config.gitlab.relative_url_root
       gon.user_color_scheme      = Gitlab::ColorSchemes.for_user(current_user).css_class
-      gon.sentry_dsn             = ApplicationSetting.current.sentry_dsn if Rails.env.production?
 
       if current_user
         gon.current_user_id = current_user.id