Don't let backtrace_cleanup_callback affect abs_path

At some point, rails added a [default gem filter to their
`BacktraceCleaner`](https://github.com/rails/rails/blob/a8709e6ea26eca73a652af4fdd0a9f7db5352af4/activesupport/lib/active_support/backtrace_cleaner.rb#L118-L125)
which messed up our linecache/context lines parsing logic since we get
paths like
```
activesupport (7.1.2) lib/active_support/callbacks.rb
```
instead of raw paths.

This PR cleans up handling this case by making a clear distinction
between `abs_path` and `filename` as was intended in the original relay
protocol and we were never populating these properly.

Author: sl0thentr0py

sentry-ruby/lib/sentry/backtrace.rb

@@ -18,6 +18,9 @@ class Line
       # org.jruby.runtime.callsite.CachingCallSite.call(CachingCallSite.java:170)
       JAVA_INPUT_FORMAT = /^(.+)\.([^\.]+)\(([^\:]+)\:(\d+)\)$/
 
+      # The absolute path in the backtrace
+      attr_reader :abs_path
+
       # The file portion of the line (such as app/models/user.rb)
       attr_reader :file
 
@@ -35,25 +38,33 @@ class Line
       # Parses a single line of a given backtrace
       # @param [String] unparsed_line The raw line from +caller+ or some backtrace
       # @return [Line] The parsed backtrace line
-      def self.parse(unparsed_line, in_app_pattern = nil)
-        ruby_match = unparsed_line.match(RUBY_INPUT_FORMAT)
-        if ruby_match
-          _, file, number, _, method = ruby_match.to_a
-          file.sub!(/\.class$/, RB_EXTENSION)
-          module_name = nil
-        else
+      def self.parse(unparsed_line, in_app_pattern = nil, cleaned_line = nil)
+        if RUBY_PLATFORM == "java"
           java_match = unparsed_line.match(JAVA_INPUT_FORMAT)
-          _, module_name, method, file, number = java_match.to_a
+          _, module_name, method, abs_path, number = java_match.to_a
+          file = nil
+        else
+          ruby_match = unparsed_line.match(RUBY_INPUT_FORMAT)
+          if ruby_match
+            _, abs_path, number, _, method = ruby_match.to_a
+            abs_path.sub!(/\.class$/, RB_EXTENSION)
+            module_name = nil
+          end
+
+          cleaned_match = cleaned_line.match(RUBY_INPUT_FORMAT) if cleaned_line
+          file = cleaned_match[1] if cleaned_match
         end
-        new(file, number, method, module_name, in_app_pattern)
+
+        new(abs_path, number, method, module_name, in_app_pattern, file)
       end
 
-      def initialize(file, number, method, module_name, in_app_pattern)
-        @file = file
+      def initialize(abs_path, number, method, module_name, in_app_pattern, file = nil)
+        @abs_path = abs_path
         @module_name = module_name
         @number = number.to_i
         @method = method
         @in_app_pattern = in_app_pattern
+        @file = file
       end
 
       def in_app
@@ -86,14 +97,14 @@ def inspect
     def self.parse(backtrace, project_root, app_dirs_pattern, &backtrace_cleanup_callback)
       ruby_lines = backtrace.is_a?(Array) ? backtrace : backtrace.split(/\n\s*/)
 
-      ruby_lines = backtrace_cleanup_callback.call(ruby_lines) if backtrace_cleanup_callback
+      cleaned_lines = backtrace ? backtrace_cleanup_callback.call(ruby_lines) : []
 
       in_app_pattern ||= begin
         Regexp.new("^(#{project_root}/)?#{app_dirs_pattern}")
       end
 
-      lines = ruby_lines.to_a.map do |unparsed_line|
-        Line.parse(unparsed_line, in_app_pattern)
+      lines = ruby_lines.to_a.zip(cleaned_lines).map do |unparsed_line, cleaned_line|
+        Line.parse(unparsed_line, in_app_pattern, cleaned_line)
       end
 
       new(lines)

sentry-ruby/lib/sentry/interfaces/stacktrace.rb

@@ -31,12 +31,13 @@ def initialize(project_root, line, strip_backtrace_load_path = true)
         @project_root = project_root
         @strip_backtrace_load_path = strip_backtrace_load_path
 
-        @abs_path = line.file
+        @abs_path = line.abs_path
         @function = line.method if line.method
         @lineno = line.number
         @in_app = line.in_app
         @module = line.module_name if line.module_name
-        @filename = compute_filename
+        # if the backtrace_cleanup_callback already gave us a nice file, we don't do the stripping again
+        @filename = line.file || compute_filename
       end
 
       def to_s