Merge pull request #38690 from abhaynikam/38685-add-option-to-disable-sql-color

Disable colorize logging of SQL and reduce unnecessary invocation of sql_color matching

activerecord/lib/active_record/log_subscriber.rb

@@ -40,7 +40,7 @@ def sql(event)
       end
 
       name = colorize_payload_name(name, payload[:name])
-      sql  = color(sql, sql_color(sql), true)
+      sql  = color(sql, sql_color(sql), true) if colorize_logging
 
       debug "  #{name}  #{sql}#{binds}"
     end

activerecord/test/cases/log_subscriber_test.rb

@@ -96,6 +96,16 @@ def test_basic_query_logging_coloration
     end
   end
 
+  def test_logging_sql_coloration_disabled
+    logger = TestDebugLogSubscriber.new
+    logger.colorize_logging = false
+
+    SQL_COLORINGS.each do |verb, color_regex|
+      logger.sql(Event.new(0.9, sql: verb.to_s))
+      assert_no_match(/#{REGEXP_BOLD}#{color_regex}#{verb}#{REGEXP_CLEAR}/i, logger.debugs.last)
+    end
+  end
+
   def test_basic_payload_name_logging_coloration_generic_sql
     logger = TestDebugLogSubscriber.new
     logger.colorize_logging = true