Backport 1a7049906ee6735bae058e9985a3806d08d9746b

Tag the blank log line between dev requests so it's easier to filter out entire requests with grep -v without leaving a stray newline behind. Example:

  # Tag asset requests so it's easy to filter them from dev logs
  config.log_tags = [
    -> request { :assets if request.path.starts_with?(config.assets.prefix) },
    -> request { request.uuid }
  ]

  # Watch the logs, ignoring asset requests
  $ tail -100f log/development.log | grep -v assets

1 files changed, 6 insertions, 3 deletions

diff --git a/railties/lib/rails/rack/logger.rb b/railties/lib/rails/rack/logger.rb
index d0d053e7ed..3f59bb8733 100644
--- a/railties/lib/rails/rack/logger.rb
+++ b/railties/lib/rails/rack/logger.rb
@@ -12,9 +12,6 @@ module Rails
       def call(env)
         request = ActionDispatch::Request.new(env)
 
-        # Put some space between requests in development logs.
-        Rails.logger.info "\n\n" if Rails.env.development?
-
         if Rails.logger.respond_to?(:tagged)
           Rails.logger.tagged(compute_tags(request)) { call_app(request, env) }
         else
@@ -25,6 +22,12 @@ module Rails
     protected
 
       def call_app(request, env)
+        # Put some space between requests in development logs.
+        if Rails.env.development?
+          Rails.logger.info ''
+          Rails.logger.info ''
+        end
+
         Rails.logger.info started_request_message(request)
         @app.call(env)
       ensure