deimos-ruby 2.0.14 → 2.0.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e8597163d34a0b2afd7ef60bbc2eda8f08c4e441292181dc7d15215ad4b2ebd7
-  data.tar.gz: 364899fa4738dd1df44f98cecc67bdc503f1a679a26274f78b49bd8b29c1ee02
+  metadata.gz: d6c9d3bcbf085821dc0be156389bd223dbce36ba8c34ff452612cf82ca3b3be4
+  data.tar.gz: fc9bf56f6ad566a9e197e50b5b167af620e9c0d6ec0fd9f50999059b7fded6ee
 SHA512:
-  metadata.gz: 54eff5f5b0f375816b3a25516f602985b00440fe522d5495346fcb873d37fb1a822666ad04623c74002a15c15c3a70ba851099096d72eaba8ab9840a460ddbc2
-  data.tar.gz: 9901785951d244c09fc81ca6601c6700149b2f03f8246f167fe594fce778c035a1f81a1a468045930cf94e5787cecffcefdf069132cfe57686ec0172c8a28f5c
+  metadata.gz: 88ac68371a699744e87fcc0c7e8a1294af1b02d4ff6973f95c1667c3754ac8fd0eeda65afddd966a4451058028409e3183e6bca112c553dbf7ac67f72929ffdd
+  data.tar.gz: 30f93b2f6aefbdf3b8554bb0a6b5bac53e6638a88adc75bdded31fde88b783ca6311fb50b652f9b8085d32da6175449181dd8229a1544018e0e1c7c51ddc158f

data/CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## UNRELEASED
 
+## 2.0.15 - 2025-05-29
+
+- Fix: Do not force data to JSON when logging payloads. This allows log formatters to handle the payloads on their own (e.g. adding additional fields).
+- Fix: Add topic to "Publishing Messages" log message.
+
 ## 2.0.14 - 2025-05-26
 
 - Fix: Log actual payloads by default (was logging "null" for the payload).

data/lib/deimos/backends/base.rb

@@ -9,9 +9,10 @@ module Deimos
         # @param messages [Array<Hash>]
         # @return [void]
         def publish(producer_class:, messages:)
+          topic = producer_class.topic
           execute(producer_class: producer_class, messages: messages)
           message = ::Deimos::Logging.messages_log_text(producer_class.karafka_config.payload_log, messages)
-          Deimos::Logging.log_info({message: 'Publishing Messages:'}.merge(message))
+          Deimos::Logging.log_info({message: "Publishing Messages for #{topic}:" }.merge(message))
         end
 
         # @param producer_class [Class<Deimos::Producer>]

data/lib/deimos/logging.rb

@@ -5,10 +5,10 @@ module Deimos
       def log_add(method, msg)
         if Karafka.logger.respond_to?(:tagged)
           Karafka.logger.tagged('Deimos') do |logger|
-            logger.send(method, msg.to_json)
+            logger.send(method, msg)
           end
         else
-          Karafka.logger.send(method, msg.to_json)
+          Karafka.logger.send(method, msg)
         end
       end
 

data/lib/deimos/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Deimos
-  VERSION = '2.0.14'
+  VERSION = '2.0.15'
 end

data/spec/producer_spec.rb

@@ -283,15 +283,15 @@ module ProducerTest
             ]
           )
           expect(Karafka.logger).to have_received(:info).with(match_message({
-            'message' => 'Publishing Messages:',
-            'payloads' => [
+            message: 'Publishing Messages for my-topic-with-id:',
+            payloads: [
               {
-                'payload' => { 'test_id' => 'foo', 'some_int' => 123 },
-                'key' => 'key'
+                payload: { 'test_id' => 'foo', 'some_int' => 123 },
+                key: 'key'
               },
               {
-                'payload' => { 'test_id' => 'foo2', 'some_int' => 123 },
-                'key' => 'key2'
+                payload: { 'test_id' => 'foo2', 'some_int' => 123 },
+                key: 'key2'
               }
             ]
           }))
@@ -309,8 +309,8 @@ module ProducerTest
             ]
           )
           expect(Karafka.logger).to have_received(:info).with(match_message({
-            'message' => 'Publishing Messages:',
-            'payloads_count' => 2
+            message: 'Publishing Messages for my-topic-with-id:',
+            payloads_count: 2
           }))
         end
       end

data/spec/spec_helper.rb

@@ -347,14 +347,13 @@ end
 RSpec::Matchers.define :match_message do |msg|
   match do |actual|
     begin
-      parsed = JSON.parse(actual)
-      parsed['payloads']&.each do |p|
-        p['payload'].delete('timestamp')
-        p['payload'].delete('message_id')
+      return false unless actual.is_a?(Hash)
+
+      actual[:payloads]&.each do |p|
+        p[:payload].delete('timestamp')
+        p[:payload].delete('message_id')
       end
-      expect(parsed).to match(a_hash_including(msg))
-    rescue JSON::ParserError
-      false
+      expect(actual).to match(a_hash_including(msg))
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deimos-ruby
 version: !ruby/object:Gem::Version
-  version: 2.0.14
+  version: 2.0.15
 platform: ruby
 authors:
 - Daniel Orner
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-05-26 00:00:00.000000000 Z
+date: 2025-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: avro_turf