From 354ed182358877648d20c8acad1cc8802c828335 Mon Sep 17 00:00:00 2001 From: Ernesto Tagwerker Date: Wed, 6 Aug 2025 10:23:21 -0400 Subject: [PATCH 1/3] Add integration to Sentry --- Gemfile | 5 +++++ Gemfile.lock | 8 ++++++++ app/views/layouts/application.html.erb | 1 + config/initializers/sentry.rb | 7 +++++++ 4 files changed, 21 insertions(+) create mode 100644 config/initializers/sentry.rb diff --git a/Gemfile b/Gemfile index 63b0293..3e11698 100644 --- a/Gemfile +++ b/Gemfile @@ -77,3 +77,8 @@ group :test do gem "capybara" gem "selenium-webdriver" end + +group :production do + gem "sentry-ruby" + gem "sentry-rails" +end diff --git a/Gemfile.lock b/Gemfile.lock index 360d185..f59e60e 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -374,6 +374,12 @@ GEM rexml (~> 3.2, >= 3.2.5) rubyzip (>= 1.2.2, < 3.0) websocket (~> 1.0) + sentry-rails (5.26.0) + railties (>= 5.0) + sentry-ruby (~> 5.26.0) + sentry-ruby (5.26.0) + bigdecimal + concurrent-ruby (~> 1.0, >= 1.0.2) sidekiq (7.3.9) base64 connection_pool (>= 2.3.0) @@ -489,6 +495,8 @@ DEPENDENCIES rails (~> 8.0.2) rubocop-rails-omakase selenium-webdriver + sentry-rails + sentry-ruby sidekiq sidekiq-scheduler slack-ruby-block-kit (>= 0.24.0) diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb index 4f1a513..0460337 100644 --- a/app/views/layouts/application.html.erb +++ b/app/views/layouts/application.html.erb @@ -16,6 +16,7 @@ <%# Includes all stylesheet files in app/assets/stylesheets %> <%= stylesheet_link_tag :app, "data-turbo-track": "reload" %> <%= javascript_importmap_tags %> + <%= Sentry.get_trace_propagation_meta.html_safe %> diff --git a/config/initializers/sentry.rb b/config/initializers/sentry.rb new file mode 100644 index 0000000..177a117 --- /dev/null +++ b/config/initializers/sentry.rb @@ -0,0 +1,7 @@ +# frozen_string_literal: true + +Sentry.init do |config| + config.breadcrumbs_logger = [ :active_support_logger ] + config.dsn = ENV["SENTRY_DSN"] + config.traces_sample_rate = 1.0 +end From b95fe4f1f835c42fc2569e996d6b5163fedc0d91 Mon Sep 17 00:00:00 2001 From: amandabizzinotto Date: Wed, 6 Aug 2025 11:42:36 -0300 Subject: [PATCH 2/3] Only initialise sentry in production --- config/initializers/sentry.rb | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/config/initializers/sentry.rb b/config/initializers/sentry.rb index 177a117..0f4d7c3 100644 --- a/config/initializers/sentry.rb +++ b/config/initializers/sentry.rb @@ -1,7 +1,9 @@ # frozen_string_literal: true -Sentry.init do |config| - config.breadcrumbs_logger = [ :active_support_logger ] - config.dsn = ENV["SENTRY_DSN"] - config.traces_sample_rate = 1.0 +if Rails.env.production? + Sentry.init do |config| + config.breadcrumbs_logger = [ :active_support_logger ] + config.dsn = ENV["SENTRY_DSN"] + config.traces_sample_rate = 1.0 + end end From 64d07494ce8f1c4c75e8a212158409813794b36b Mon Sep 17 00:00:00 2001 From: Ernesto Tagwerker Date: Wed, 6 Aug 2025 10:46:10 -0400 Subject: [PATCH 3/3] Use Sentry only in production --- app/views/layouts/application.html.erb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb index 0460337..e91ec44 100644 --- a/app/views/layouts/application.html.erb +++ b/app/views/layouts/application.html.erb @@ -16,7 +16,9 @@ <%# Includes all stylesheet files in app/assets/stylesheets %> <%= stylesheet_link_tag :app, "data-turbo-track": "reload" %> <%= javascript_importmap_tags %> - <%= Sentry.get_trace_propagation_meta.html_safe %> + <% if Rails.env.production? %> + <%= Sentry.get_trace_propagation_meta.html_safe %> + <% end %> @@ -29,7 +31,7 @@ <% end %> <% end %> - + <%= yield %>