Rework to use a callback

Author: solnic

sentry-rails/lib/sentry/rails/configuration.rb

@@ -33,6 +33,12 @@ class Configuration
         end
       end
     end
+
+    after(:configured) do
+      if enable_logs && rails.structured_logging.enabled != false
+        rails.structured_logging.enabled = true
+      end
+    end
   end
 
   module Rails
@@ -183,7 +189,7 @@ def initialize(parent_config = nil)
         @db_query_source_threshold_ms = 100
         @active_support_logger_subscription_items = Sentry::Rails::ACTIVE_SUPPORT_LOGGER_SUBSCRIPTION_ITEMS_DEFAULT.dup
         @active_job_report_on_retry_error = false
-        @structured_logging = StructuredLoggingConfiguration.new(parent_config)
+        @structured_logging = StructuredLoggingConfiguration.new
       end
     end
 
@@ -192,30 +198,24 @@ class StructuredLoggingConfiguration
       # @return [Boolean]
       attr_accessor :enabled
 
-      private :enabled
-
       # Hash of components to subscriber classes for structured logging
       # @return [Hash<Symbol, Class>]
       attr_accessor :subscribers
 
-      # @return [Sentry::Configuration]
-      attr_reader :parent_config
-
       DEFAULT_SUBSCRIBERS = {
         active_record: Sentry::Rails::LogSubscribers::ActiveRecordSubscriber,
         action_controller: Sentry::Rails::LogSubscribers::ActionControllerSubscriber
       }.freeze
 
-      def initialize(parent_config)
-        @enabled = nil
+      def initialize
+        @enabled = false
         @subscribers = DEFAULT_SUBSCRIBERS.dup
-        @parent_config = parent_config
       end
 
       # Returns true if structured logging should be enabled.
       # @return [Boolean]
       def enabled?
-        enabled.nil? ? parent_config.enable_logs : enabled
+        enabled
       end
     end
   end