Do not log the binding values for binary columns.

They tend to be large and not very useful in the log.

1 files changed, 11 insertions, 1 deletions

diff --git a/activerecord/lib/active_record/log_subscriber.rb b/activerecord/lib/active_record/log_subscriber.rb
index ca79950049..11ec2eaf79 100644
--- a/activerecord/lib/active_record/log_subscriber.rb
+++ b/activerecord/lib/active_record/log_subscriber.rb
@@ -20,6 +20,16 @@ module ActiveRecord
       @odd_or_even = false
     end
 
+    def render_bind(column, value)
+      if column.type == :binary
+        rendered_value = "<#{value.bytesize} bytes of binary data>"
+      else
+        rendered_value = value
+      end
+
+      [column.name, rendered_value]
+    end
+
     def sql(event)
       self.class.runtime += event.duration
       return unless logger.debug?
@@ -34,7 +44,7 @@ module ActiveRecord
 
       unless (payload[:binds] || []).empty?
         binds = "  " + payload[:binds].map { |col,v|
-          [col.name, v]
+          render_bind(col, v)
         }.inspect
       end