Replace sentry-raven with sentry-ruby

No need to capture exceptions manually in the low-level error handler
since getsentry/sentry-ruby#2026

Close #242

Gemfile

@@ -18,7 +18,7 @@ gem 'commonmarker'
 gem 'rest-client'
 gem 'octokit'
 gem 'addressable'
-gem 'sentry-raven'
+gem 'sentry-ruby'
 gem 'racksh'
 gem 'warning'
 gem 'rake'

Gemfile.lock

@@ -104,8 +104,8 @@ GEM
     sawyer (0.8.2)
       addressable (>= 2.3.5)
       faraday (> 0.8, < 2.0)
-    sentry-raven (2.9.0)
-      faraday (>= 0.7.6, < 1.0)
+    sentry-ruby (5.10.0)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
     sequel (5.70.0)
     sequel_pg (1.17.1)
       pg (>= 0.18.0, != 1.2.0)
@@ -162,7 +162,7 @@ DEPENDENCIES
   rake
   rest-client
   rubocop (~> 1.54.1)
-  sentry-raven
+  sentry-ruby
   sequel
   sequel_pg
   sinatra

config.ru

@@ -2,9 +2,9 @@
 
 require_relative 'config/sentry'
 
-require 'rack/ssl-enforcer'
+use Sentry::Rack::CaptureExceptions
 
-use Raven::Rack
+require 'rack/ssl-enforcer'
 
 # Connects to the database
 require_relative 'config/app'

config/puma.rb

@@ -17,12 +17,10 @@
 end
 
 lowlevel_error_handler do |ex, env|
-  Raven.capture_exception(
-    ex,
-    :message => ex.message,
-    :extra => { :puma => env },
-    :transaction => "Puma"
-  )
+  if App.test_lowlevel_error_handler?
+    puts "puma lowlevel_error_handler ran with exception=#{ex.inspect}"
+  end
+
   [500, {}, ["An error has occurred, and engineers have been informed.\n"]]
 end
 

config/sentry.rb

@@ -1,13 +1,18 @@
 # frozen_string_literal: true
 
-require "raven"
+require "sentry-ruby"
 require "logger"
 
 require_relative "../lib/app"
 
-Raven.configure do |config|
+Sentry.init do |config|
   config.logger = App.null_logger if App.test?
   config.logger.level = Logger::DEBUG
-  config.processors -= [Raven::Processor::PostData] # send POST data
-  config.processors -= [Raven::Processor::Cookies]  # send cookies
+
+  # https://docs.sentry.io/platforms/ruby/configuration/releases/#release-health
+  config.auto_session_tracking = false
+
+  # send POST data send cookies
+  # https://docs.sentry.io/platforms/ruby/migration/#removed-processors
+  config.send_default_pii = true
 end

test/integration/app_boot_test.rb

@@ -69,8 +69,8 @@ def test_app_lowlevel_error_handler
 
       get_http_response(port: port)
 
-      assert spawn.wait("DEBUG -- sentry: ** [Raven]")
-      assert spawn.wait("excluded from capture: DSN not set")
+      assert spawn.wait("DEBUG -- sentry: ** [Sentry] Initializing the background worker")
+      assert spawn.wait("puma lowlevel_error_handler ran")
     end
   end