pier_logging 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3b5d51eccb3353d9b1e898eb5c6f6d5d14dedcb183df51c7ed6300554e702f9
-  data.tar.gz: 55a389c152833b3b94889af6fecaddc73682d33a5b48ac495fdf414f7e8886fe
+  metadata.gz: a55ccf237c0b0f40cdc5a97c3b1409112de8343a8b1070f1b70d814ef4ed4dad
+  data.tar.gz: acd0969b17612d4f42d1c8229699c5176e2aa8ebfdf13a5fd8c07b64b94602d3
 SHA512:
-  metadata.gz: 1246e42cf2aa2632506cb52aa90159998582f80ed89dda9915834abbff04f726d84fcd819c99dcf9cc0a73d56db03ee0ba597ba29fafa547028ab3f52f75e1b5
-  data.tar.gz: a4d3294e81e52f535a08f8fe883efdcf93acc8f98d27e12c879290db862dcb748458adadb4e7541b5cafcded08e99b74a681d820106a8c7dc9567260124ea2b0
+  metadata.gz: 56f1b5ca14007773f0604a3bf8720f81f5950892120e260a9a7c6a9231680edf4ece11c5702714e29a07ccae6c9490d8b406a992c78af8e6c1e85a887fa05deb
+  data.tar.gz: 79c17490ba3109bdbce3a06dd8fa45f92b6537a350e274273d5856208b99f878e9d085d4d194f7c8b62490af5b051b568d1a9eb74552993bb55c0962815a92d0

data/lib/pier_logging.rb

@@ -54,13 +54,15 @@ module PierLogging
   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 @@ module PierLogging
       @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'" 

data/lib/pier_logging/request_logger.rb

@@ -57,7 +57,7 @@ module PierLogging
           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 @@ module PierLogging
       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

data/lib/pier_logging/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pier_logging
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - Mauricio Banduk