logstash-output-dynatrace 0.7.1 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 81b4a8c2170a81fb234ff285214b21e531d8b33a304ccbe25b643a10d05a7c28
-  data.tar.gz: 70e9196d107abd6655ea8394fac223713bd8120c5305c5f22075d402f897d65d
+  metadata.gz: 6dc320145268685ae7585b0ffc68beb54ae932b3e9c5261e1ca6106803c487df
+  data.tar.gz: 223b65b82ffc6ed512d270eb879126a9f9fbe046f9dfc9e66ae9a37be2e60fcf
 SHA512:
-  metadata.gz: 6c67bc08595859467e6c32da87b60e0da67e99f3df4eb8b59d1ecedd8b96676429fe908b4e0678ea4f74736445fab3880a25461d439ef8fef3ce6100b9bc7ada
-  data.tar.gz: c0c78831f7436a02ae3faafab913a725af81d9bf8a14111b7de5d4e46bac3b473f22b325ef27280f5bb4262279a86dadf6ec2b19250b89819e90b7e7f5b8c32b
+  metadata.gz: 55a4b11c59aae913134c5972dff31b13c3e3eaa8761db6703a88e081df6d369b39d6be5f924219e30312b819a54c30d7aced19d98b24f6bcf4e988561929ce50
+  data.tar.gz: 401c4c51154584cee22c94f557a5869b8782759b5287498571e2f1ae345150f48625417126140bb54676560a3d75f7659b30ca392b2db2470dae7815b49904d2

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.7.2
+
+- Log response bodies when the response is `200` indicating the request was only partially accepted
+ 
 ## 0.7.1
 
 - Fix an error where max payload size was calculated using character count instead of bytes

data/lib/logstash/outputs/dynatrace.rb

@@ -160,6 +160,10 @@ module LogStash
         end
       end
 
+      def log_partial_success_response(response)
+        @logger.warn("Encountered partial success response", code: response.code, body: response.body)
+      end
+
       def log_error_response(response, ingest_endpoint_url, event)
         log_failure(
           "Encountered non-2xx HTTP code #{response.code}",
@@ -267,6 +271,8 @@ module LogStash
         response = client.post(ingest_endpoint_url, body: event, headers: headers)
 
         if response_success?(response)
+          # some events were not accepted but we don't know which ones or why
+          log_partial_success_response(response) if response_partial_success?(response)
           [:success, event, attempt]
         elsif retryable_response?(response)
           log_retryable_response(response)
@@ -315,6 +321,10 @@ module LogStash
         response.code >= 200 && response.code <= 299
       end
 
+      def response_partial_success?(response)
+        response.code == 200
+      end
+
       def retryable_response?(response)
         RETRYABLE_CODES.include?(response.code)
       end

data/spec/outputs/dynatrace_spec.rb

@@ -168,6 +168,19 @@ describe LogStash::Outputs::Dynatrace do
       end
     end
 
+    context 'with partial success responses' do
+      let(:ingest_endpoint_url) { "http://localhost:#{port}/partial" }
+
+      before do
+        allow(subject).to receive(:log_partial_success_response)
+      end
+
+      it 'should warn on partial success' do
+        subject.multi_receive([event])
+        expect(subject).to have_received(:log_partial_success_response)
+      end
+    end
+
     context 'with retryable failing requests' do
       let(:ingest_endpoint_url) { "http://localhost:#{port}/retry" }
       let(:api_key) { 'placeholder-key' }

data/spec/spec_helper.rb

@@ -98,7 +98,12 @@ class TestApp < Sinatra::Base
 
   multiroute(%w[get post put patch delete], '/good') do
     self.class.last_request = request
-    [200, 'YUP']
+    [204, 'YUP']
+  end
+
+  multiroute(%w[get post put patch delete], '/partial') do
+    self.class.last_request = request
+    [200, 'partial success']
   end
 
   multiroute(%w[get post put patch delete], '/bad') do

data/version.yaml

@@ -1 +1 @@
-logstash-output-dynatrace: '0.7.1'
+logstash-output-dynatrace: '0.7.2'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-dynatrace
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.2
 platform: ruby
 authors:
 - Dynatrace Open Source Engineering
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-23 00:00:00.000000000 Z
+date: 2025-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: logstash-codec-json