gritz-core 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc43859887cac5ffa9e137f2cab8911a4a6ef79ed4c522380d55869446c360a9
-  data.tar.gz: adfbe678ac9b1af0f507cad8fb69c16bfea7acbe4a422bacfce1e6b90747f34b
+  metadata.gz: 365ed02137d42c5182e6f09a10544c189d7fe0d70fc0faffd2864b752e13e40c
+  data.tar.gz: 797d64d766c36257ef700d7647045d317c4027ed4b7295f959f54fec2139849e
 SHA512:
-  metadata.gz: 0c1d9ef947490cfa7c5832cca9d1c20f0c7e093798e56c4d3b0b261c80279ae1a55f2b747ea2f9c42af99c126ac91a3c55098d18a186d61a5ab362e1caad0b1d
-  data.tar.gz: 6e859a121f10896fb4e38f8cbf669e33ce504637026aa2269d338f003a64f02190b8299b59e145a4909bd5cf6cbd0d62fbab3fac630952154e880cd900cf0014
+  metadata.gz: 8cdc413c2a357fb5e98ce5c397ae5e1b69d136f89752d479d4960367fc0a2cf6d99dabeebede608aa2f7c8f948ee542640ff4f269323a7aa203d5ef07c027296
+  data.tar.gz: 3da9357cc46ca9f495f997803929d489e3ca691074cdad0b1e33f49457bcbc35d58ad37b3331b754b08967a3ff5376dfd60f6a02943a8e6e3e8fac34f78a35a7

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.6.1
+
+- Avoid formatting suppressed RPC completion logs, preserving responses, status mapping and metrics when INFO logging is disabled.
+
 ## 0.6.0
 
 - Run Async workers through the same supervisor, CLI and real-server test helper as Native workers.

data/lib/gritz/core/version.rb

@@ -2,6 +2,6 @@
 
 module Gritz
   module Core
-    VERSION = "0.6.0"
+    VERSION = "0.6.1"
   end
 end

data/lib/gritz/middleware/logging.rb

@@ -22,22 +22,23 @@ module Gritz
         code = :internal
         raise
       ensure
-        duration = (Process.clock_gettime(Process::CLOCK_MONOTONIC) - started) * 1000
-        fields = { request_id: context.request_id, service: context.method.service,
-                   method: context.method.name, code: code, duration_ms: duration.round(3),
-                   peer: context.peer, worker: context.worker, pid: Process.pid, bytes_in: context.bytes_in, bytes_out: context.bytes_out }
-        fields.merge!(context.store.fetch(:gritz_error, {}))
-        fields = prepare(fields, context.log_redact)
-        message = if context.log_format == :logfmt
-                    fields.map do |key, value|
-                      value = value.to_s if value.is_a?(Symbol)
-                      value = JSON.generate(value) if value.is_a?(Hash) || value.is_a?(Array)
-                      "#{key}=#{JSON.generate(value)}"
-                    end.join(" ")
-                  else
-                    JSON.generate(fields)
-                  end
-        context.logger.info(message)
+        context.logger.info do
+          duration = (Process.clock_gettime(Process::CLOCK_MONOTONIC) - started) * 1000
+          fields = { request_id: context.request_id, service: context.method.service,
+                     method: context.method.name, code: code, duration_ms: duration.round(3),
+                     peer: context.peer, worker: context.worker, pid: Process.pid, bytes_in: context.bytes_in, bytes_out: context.bytes_out }
+          fields.merge!(context.store.fetch(:gritz_error, {}))
+          fields = prepare(fields, context.log_redact)
+          if context.log_format == :logfmt
+            fields.map do |key, value|
+              value = value.to_s if value.is_a?(Symbol)
+              value = JSON.generate(value) if value.is_a?(Hash) || value.is_a?(Array)
+              "#{key}=#{JSON.generate(value)}"
+            end.join(" ")
+          else
+            JSON.generate(fields)
+          end
+        end
       end
 
       private

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gritz-core
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - Yudai Takada