Add hide_request_body_for_paths config (#23)

Author: banduk

lib/pier_logging.rb

@@ -54,13 +54,15 @@ def formatter=(formatter)
   end
 
   class RequestLoggerConfiguration
-    attr_reader :enabled, :user_info_getter, :hide_response_body_for_paths, 
-                :log_response, :hide_request_headers, :correlation_id_getter
+    attr_reader :enabled, :user_info_getter, :hide_request_body_for_paths, :hide_response_body_for_paths, 
+                :log_request_body, :log_response, :hide_request_headers, :correlation_id_getter
 
     def initialize
       @user_info_getter = ->(_ = nil) { nil }
       @enabled = false
+      @hide_request_body_for_paths = nil
       @hide_response_body_for_paths = nil
+      @log_request_body = true
       @log_response = true
       @hide_request_headers = nil
       @correlation_id_getter = ->(_ = nil, _ = nil) { nil }
@@ -71,11 +73,24 @@ def user_info_getter=(proc)
       @user_info_getter = proc
     end
 
+    def log_request_body=(log_request_body)
+      raise ArgumentError, "Config 'log_request_body' must be a 'boolean'" unless !!log_request_body == log_request_body
+      @log_request_body = log_request_body
+    end
+
     def log_response=(log_response)
       raise ArgumentError, "Config 'log_response' must be a 'boolean'" unless !!log_response == log_response
       @log_response = log_response
     end
 
+    def hide_request_body_for_paths=(hide_request_body_for_paths)
+      unless (hide_request_body_for_paths.is_a? Array) && (hide_request_body_for_paths.all?{|item| item.is_a? Regexp})
+        raise ArgumentError, "Config 'hide_request_body_for_paths' must be an 'Array of Regexps'" 
+      end
+      
+      @hide_request_body_for_paths = hide_request_body_for_paths
+    end
+
     def hide_response_body_for_paths=(hide_response_body_for_paths)
       unless (hide_response_body_for_paths.is_a? Array) && (hide_response_body_for_paths.all?{|item| item.is_a? Regexp})
         raise ArgumentError, "Config 'hide_response_body_for_paths' must be an 'Array of Regexps'" 

lib/pier_logging/request_logger.rb

@@ -57,7 +57,7 @@ def log(args)
           headers: request_headers,
           href: request.url,
           query_string: request.query_string,
-          body: parse_body(request.body)
+          body: request_body(request.path, request.body)
         },
         response: {
           status: status,
@@ -83,11 +83,20 @@ def get_request_headers_from_env(env)
       headers
     end
 
+    def request_body(request_path, body)
+      return nil unless PierLogging.request_logger_configuration.log_request_body
+
+      hide_request_body_for_paths = PierLogging.request_logger_configuration.hide_request_body_for_paths
+      return nil if hide_request_body_for_paths&.any?{ |path|request_path =~ path }
+
+      parse_body(body)
+    end
+    
     def response_body(request_path, body)
       return nil unless PierLogging.request_logger_configuration.log_response
 
       hide_response_body_for_paths = PierLogging.request_logger_configuration.hide_response_body_for_paths
-      return nil if hide_response_body_for_paths && hide_response_body_for_paths.any?{ |path|request_path =~ path }
+      return nil if hide_response_body_for_paths&.any?{ |path|request_path =~ path }
 
       parse_body(body)
     end

lib/pier_logging/version.rb

@@ -1,3 +1,3 @@
 module PierLogging
-  VERSION = "0.3.2"
+  VERSION = "0.3.3"
 end