Only report to Sentry when it's enabled.

3ee8eb11 · Connor Shea · 2e9bf6a7 · 3ee8eb11
Commit 3ee8eb11 authored Jul 05, 2016 by Connor Shea
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

config/initializers/secure_headers.rb config/initializers/secure_headers.rb +6 -2

No files found.
--- a/config/initializers/secure_headers.rb
+++ b/config/initializers/secure_headers.rb
 require 'gitlab/current_settings'
 include Gitlab::CurrentSettings
-uri = URI.parse(current_application_settings.sentry_dsn)
+if Rails.env.production? && current_application_settings.sentry_enabled
-CSP_REPORT_URI = "#{uri.scheme}://#{uri.host}/api#{uri.path}/csp-report/?sentry_key=#{uri.user}"
+  uri = URI.parse(current_application_settings.sentry_dsn)
+  CSP_REPORT_URI = "#{uri.scheme}://#{uri.host}/api#{uri.path}/csp-report/?sentry_key=#{uri.user}"
+else
+  CSP_REPORT_URI = ''
+end
 SecureHeaders::Configuration.default do |config|
  config.cookies = {