deimos-ruby 2.0.13 → 2.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c18442a9ba85a0b6de4c3cdaee151727d7e7ba9ca70de8129b08a91ad1a100c
-  data.tar.gz: 7db213f9a31a8775f6f23fd10098d284df59f5dbf4b040d0719b9fa10f5af6e6
+  metadata.gz: e8597163d34a0b2afd7ef60bbc2eda8f08c4e441292181dc7d15215ad4b2ebd7
+  data.tar.gz: 364899fa4738dd1df44f98cecc67bdc503f1a679a26274f78b49bd8b29c1ee02
 SHA512:
-  metadata.gz: 532e1eed51b376e568c1e7e9b424a65a29a7e339e8db91e3f3761328ff826e3ad45a8c12bddbb4155707610df3ab36bf90e1acfe6411b6829be6fe8e45453e75
-  data.tar.gz: 29ca80e081b8c0699ee8b132c31d7ab27eb91f63dca808a03f8c7e3905a70a7f520964abc0631995b033c7bfc72d3581dd7d5dacd93d234b970ebbc92bd99837
+  metadata.gz: 54eff5f5b0f375816b3a25516f602985b00440fe522d5495346fcb873d37fb1a822666ad04623c74002a15c15c3a70ba851099096d72eaba8ab9840a460ddbc2
+  data.tar.gz: 9901785951d244c09fc81ca6601c6700149b2f03f8246f167fe594fce778c035a1f81a1a468045930cf94e5787cecffcefdf069132cfe57686ec0172c8a28f5c

data/CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## UNRELEASED
 
+## 2.0.14 - 2025-05-26
+
+- Fix: Log actual payloads by default (was logging "null" for the payload).
+
 ## 2.0.13 - 2025-05-26
 
 - Fix: Outbox producer would crash if no Deimos producer was defined for the topic.

data/lib/deimos/logging.rb

@@ -42,7 +42,7 @@ module Deimos
         if m.respond_to?(:payload)
           m.payload
         elsif m[:label]
-          m.dig(:label, :raw_payload)
+          m.dig(:label, :original_payload)
         else
           m[:payload]
         end

data/lib/deimos/version.rb

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

data/spec/producer_spec.rb

@@ -272,6 +272,50 @@ module ProducerTest
                                                       }, '456', '4561')
     end
 
+    describe 'payload logging' do
+      context 'with default / full' do
+        it 'should log full payload' do
+          allow(Karafka.logger).to receive(:info)
+          MyProducerWithID.publish_list(
+            [
+              { 'test_id' => 'foo', 'some_int' => 123, :payload_key => 'key' },
+              { 'test_id' => 'foo2', 'some_int' => 123, :payload_key => 'key2' },
+            ]
+          )
+          expect(Karafka.logger).to have_received(:info).with(match_message({
+            'message' => 'Publishing Messages:',
+            'payloads' => [
+              {
+                'payload' => { 'test_id' => 'foo', 'some_int' => 123 },
+                'key' => 'key'
+              },
+              {
+                'payload' => { 'test_id' => 'foo2', 'some_int' => 123 },
+                'key' => 'key2'
+              }
+            ]
+          }))
+        end
+      end
+
+      context 'with count' do
+        it 'should log only count' do
+          Deimos.karafka_config_for(topic: 'my-topic-with-id').payload_log :count
+          allow(Karafka.logger).to receive(:info)
+          MyProducerWithID.publish_list(
+            [
+              { 'test_id' => 'foo', 'some_int' => 123, :payload_key => 'key' },
+              { 'test_id' => 'foo2', 'some_int' => 123, :payload_key => 'key' }
+            ]
+          )
+          expect(Karafka.logger).to have_received(:info).with(match_message({
+            'message' => 'Publishing Messages:',
+            'payloads_count' => 2
+          }))
+        end
+      end
+    end
+
     context 'with Schema Class payloads' do
       it 'should fail on invalid message with error handler' do
         expect(Deimos::ProducerMiddleware).to receive(:call).and_raise('OH NOES')

data/spec/spec_helper.rb

@@ -343,3 +343,18 @@ RSpec.shared_context('with publish_backend') do
     end
   end
 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')
+      end
+      expect(parsed).to match(a_hash_including(msg))
+    rescue JSON::ParserError
+      false
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: deimos-ruby
 version: !ruby/object:Gem::Version
-  version: 2.0.13
+  version: 2.0.14
 platform: ruby
 authors:
 - Daniel Orner