Skip to content
  • Alex Ghiculescu's avatar
    55cd9ae8
    Fix double logging in `ActiveRecord::QueryLog` · 55cd9ae8
    Alex Ghiculescu authored
    Fixes https://github.com/rails/rails/issues/46103
    
    An issue exists if you set `config.active_record.query_log_tags` to an array that includes `:controller`, `:namespaced_controller`, `:action`, or `:job`; the relevant item will get duplicated in the log line. This occured because the relevant railties would add the item to `config.active_record.query_log_tags` again during setup. This PR fixes that by only adding those items to the config if they aren't already set.
    
    So this PR just fixes that bug, and tweaks the changelog entry rather than adding another one for the same bug.
    55cd9ae8
    Fix double logging in `ActiveRecord::QueryLog`
    Alex Ghiculescu authored
    Fixes https://github.com/rails/rails/issues/46103
    
    An issue exists if you set `config.active_record.query_log_tags` to an array that includes `:controller`, `:namespaced_controller`, `:action`, or `:job`; the relevant item will get duplicated in the log line. This occured because the relevant railties would add the item to `config.active_record.query_log_tags` again during setup. This PR fixes that by only adding those items to the config if they aren't already set.
    
    So this PR just fixes that bug, and tweaks the changelog entry rather than adding another one for the same bug.
Loading