From 8fbabdef4e845a5c628d846b5a2f7ba32f905edb Mon Sep 17 00:00:00 2001 From: "Jon R. Humphrey" Date: Fri, 2 Jun 2023 10:49:33 +0100 Subject: [PATCH] fix: removed commas from request method list `%w[]` is a non-interpolated array of words, separated by whitespace therefore doesn't need comma separation --- lib/json_rails_logger/json_formatter.rb | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/json_rails_logger/json_formatter.rb b/lib/json_rails_logger/json_formatter.rb index f4f51f7..02d5f18 100644 --- a/lib/json_rails_logger/json_formatter.rb +++ b/lib/json_rails_logger/json_formatter.rb @@ -7,7 +7,7 @@ class JsonFormatter < ::Logger::Formatter method path status duration user_agent accept request_id ].freeze - REQUEST_METHODS = %w[GET, POST, PUT, DELETE, PATCH].freeze + REQUEST_METHODS = %w[GET POST PUT DELETE PATCH].freeze def call(severity, timestamp, _progname, raw_msg) sev = process_severity(severity) @@ -45,7 +45,6 @@ def process_message(raw_msg) msg = normalize_message(raw_msg) return msg unless msg.is_a?(String) - return status_message(msg) if status_message?(msg) return request_type(msg) if request_type?(msg) return user_agent_message(msg) if user_agent_message?(msg) @@ -106,7 +105,6 @@ def request_type(msg) splitted_msg = msg.split method = splitted_msg[0] path = splitted_msg[1] - { method: method, path: path } end