fix(logging): add missing :origin to rails logging

Author: solnic

sentry-rails/lib/sentry/rails/log_subscriber.rb

@@ -27,6 +27,8 @@ module Rails
     #     end
     #   end
     class LogSubscriber < ActiveSupport::LogSubscriber
+      ORIGIN = "auto.logger.rails.log_subscriber"
+
       class << self
         if ::Rails.version.to_f < 6.0
           # Rails 5.x does not provide detach_from
@@ -51,8 +53,9 @@ def detach_from(namespace, notifications = ActiveSupport::Notifications)
       # @param message [String] The log message
       # @param level [Symbol] The log level (:trace, :debug, :info, :warn, :error, :fatal)
       # @param attributes [Hash] Additional structured attributes to include
-      def log_structured_event(message:, level: :info, attributes: {})
-        Sentry.logger.public_send(level, message, **attributes)
+      # @param origin [String] The origin of the log event
+      def log_structured_event(message:, level: :info, attributes: {}, origin: ORIGIN)
+        Sentry.logger.public_send(level, message, **attributes, origin: origin)
       rescue => e
         # Silently handle any errors in logging to avoid breaking the application
         Sentry.configuration.sdk_logger.debug("Failed to log structured event: #{e.message}")

sentry-rails/spec/sentry/rails/log_subscribers/action_controller_subscriber_spec.rb

@@ -35,6 +35,7 @@
         expect(log_event[:attributes][:method][:value]).to eq("GET")
         expect(log_event[:attributes][:path][:value]).to eq("/world")
         expect(log_event[:attributes][:format][:value]).to eq(:html)
+        expect(log_event[:attributes]["sentry.origin"][:value]).to eq("auto.logger.rails.log_subscriber")
       end
 
       it "logs bad requests appropriately" do

sentry-rails/spec/sentry/rails/log_subscribers/action_mailer_subscriber_spec.rb

@@ -38,6 +38,7 @@
         expect(log_event[:attributes][:duration_ms][:value]).to be > 0
         expect(log_event[:attributes][:perform_deliveries][:value]).to be true
         expect(log_event[:attributes][:delivery_method][:value]).to eq(:test)
+        expect(log_event[:attributes]["sentry.origin"][:value]).to eq("auto.logger.rails.log_subscriber")
         expect(log_event[:attributes][:date]).to be_present
       end
 

sentry-rails/spec/sentry/rails/log_subscribers/active_job_subscriber_spec.rb

@@ -26,6 +26,7 @@
         expect(log_event[:level]).to eq("info")
         expect(log_event[:attributes][:job_class][:value]).to eq("NormalJob")
         expect(log_event[:attributes][:duration_ms][:value]).to be > 0
+        expect(log_event[:attributes]["sentry.origin"][:value]).to eq("auto.logger.rails.log_subscriber")
       end
 
       it "logs job enqueue events when jobs are enqueued" do

sentry-rails/spec/sentry/rails/log_subscribers/active_record_subscriber_spec.rb

@@ -25,6 +25,7 @@
         expect(log_event[:level]).to eq("info")
         expect(log_event[:attributes][:sql][:value]).to include("INSERT INTO")
         expect(log_event[:attributes][:duration_ms][:value]).to be > 0
+        expect(log_event[:attributes]["sentry.origin"][:value]).to eq("auto.logger.rails.log_subscriber")
       end
 
       it "logs SELECT queries with proper attributes" do