Merge pull request #16 from philiplambok-mekari/KRED-2061

KRED-2061: Improve regex logic in filtering logs

Author: philiplambok-mekari

.ruby-version

@@ -0,0 +1 @@
+2.7.2

README.md

@@ -29,6 +29,8 @@ midtrans = MidtransApi::Client.new(
   server_key: 'YOUR-SERVER-KEY',
   sandbox: true|false,
   notification_url: 'https://example.com/callback',
+  filtered_logs: %[word1 word2],
+  logger: Logger.new(STDOUT),
   timeout: 30 # by default will be 60 (seconds)
 )
 

lib/midtrans_api/client.rb

@@ -59,9 +59,9 @@ def initialize(options = {})
             filtered_logs = options[:filtered_logs]
             if filtered_logs.respond_to?(:each)
               filtered_logs.each do |filter|
-                log.filter(%r{(#{filter}=)([\w+-.?@:/]+)}, '\1[FILTERED]')
-                log.filter(%r{(#{filter}":")([\w+-.?@:/]+)}, '\1[FILTERED]')
-                log.filter(%r{(#{filter}":)([\w+-.?@:/]+)}, '\1[FILTERED]')
+                log.filter(/(#{filter}":\s*")(.*?)(")/i, '\1[FILTERED]\3')
+                log.filter(/(#{filter}":\s*)(\d+(?:\.\d+)?|true|false)/i, '\1[FILTERED]')
+                log.filter(/(#{filter}":\s*)(\[.*?\])/i, '\1[FILTERED]')
               end
             end
           end