logstash-output-elasticsearch 11.22.13-java → 12.0.0-java

data/spec/unit/outputs/elasticsearch/http_client_spec.rb

@@ -270,83 +270,6 @@ describe LogStash::Outputs::ElasticSearch::HttpClient do
 
        end
      end
-    context "the 'user-agent' header" do
-      let(:pool) { double("pool") }
-      let(:compression_level) { 6 }
-      let(:base_options) { super().merge( :client_settings => {:compression_level => compression_level}) }
-      let(:actions) { [
-        ["index", {:_id=>nil, :_index=>"logstash"}, {"message_1"=> message_1}],
-        ["index", {:_id=>nil, :_index=>"logstash"}, {"message_2"=> message_2}],
-        ["index", {:_id=>nil, :_index=>"logstash"}, {"message_3"=> message_3}],
-      ]}
-      let(:message_1) { "hello" }
-      let(:message_2_size) { 1_000 }
-      let(:message_2) { SecureRandom.alphanumeric(message_2_size / 2 ) * 2 }
-      let(:message_3_size) { 1_000 }
-      let(:message_3) { "m" * message_3_size }
-      let(:messages_size) { message_1.size + message_2.size + message_3.size }
-      let(:action_overhead) { 42 + 16 + 2 } # header plus doc key size plus new line overhead per action
-
-      let(:response) do
-        response = double("response")
-        allow(response).to receive(:code).and_return(response)
-        allow(response).to receive(:body).and_return({"errors" => false}.to_json)
-        response
-      end
-
-      before(:each) do
-        subject.instance_variable_set("@pool", pool)
-      end
-
-      it "carries bulk request's uncompressed size" do
-        expect(pool).to receive(:post) do |path, params, body|
-          headers = params.fetch(:headers, {})
-          expect(headers["X-Elastic-Event-Count"]).to eq("3")
-          expect(headers["X-Elastic-Uncompressed-Request-Length"]).to eq (messages_size + (action_overhead * 3)).to_s
-        end.and_return(response)
-
-        subject.send(:bulk, actions)
-      end
-      context "without compression" do
-        let(:compression_level) { 0 }
-        it "carries bulk request's uncompressed size" do
-          expect(pool).to receive(:post) do |path, params, body|
-            headers = params.fetch(:headers, {})
-            expect(headers["X-Elastic-Event-Count"]).to eq("3")
-            expect(headers["X-Elastic-Uncompressed-Request-Length"]).to eq (messages_size + (action_overhead * 3)).to_s
-          end.and_return(response)
-          subject.send(:bulk, actions)
-        end
-      end
-
-      context "with compressed messages over 20MB" do
-        let(:message_2_size) { 21_000_000 }
-        it "carries bulk request's uncompressed size" do
-          # only the first, tiny, message is sent first
-          expect(pool).to receive(:post) do |path, params, body|
-            headers = params.fetch(:headers, {})
-            expect(headers["X-Elastic-Uncompressed-Request-Length"]).to eq (message_1.size + action_overhead).to_s
-            expect(headers["X-Elastic-Event-Count"]).to eq("1")
-          end.and_return(response)
-
-          # huge message_2 is sent afterwards alone
-          expect(pool).to receive(:post) do |path, params, body|
-            headers = params.fetch(:headers, {})
-            expect(headers["X-Elastic-Uncompressed-Request-Length"]).to eq (message_2.size + action_overhead).to_s
-            expect(headers["X-Elastic-Event-Count"]).to eq("1")
-          end.and_return(response)
-
-          # finally medium message_3 is sent alone as well
-          expect(pool).to receive(:post) do |path, params, body|
-            headers = params.fetch(:headers, {})
-            expect(headers["X-Elastic-Uncompressed-Request-Length"]).to eq (message_3.size + action_overhead).to_s
-            expect(headers["X-Elastic-Event-Count"]).to eq("1")
-          end.and_return(response)
-
-          subject.send(:bulk, actions)
-        end
-      end
-    end
   end
 
   describe "sniffing" do

data/spec/unit/outputs/elasticsearch_spec.rb

@@ -777,7 +777,7 @@ describe LogStash::Outputs::ElasticSearch do
       end
 
       before(:each) do
-        allow(subject.client).to receive(:bulk_send).with(instance_of(StringIO), instance_of(Array), instance_of(Hash)) do |stream, actions, headers|
+        allow(subject.client).to receive(:bulk_send).with(instance_of(StringIO), instance_of(Array)) do |stream, actions|
           expect( stream.string ).to include '"foo":"bar1"'
           expect( stream.string ).to include '"foo":"bar2"'
         end.and_return(bulk_response, {"errors"=>false}) # let's make it go away (second call) to not retry indefinitely
@@ -915,12 +915,7 @@ describe LogStash::Outputs::ElasticSearch do
       allow(elasticsearch_output_instance.client.pool).to receive(:post) do |path, params, body|
         if body.length > max_bytes
           max_bytes *= 2 # ensure a successful retry
-          raise ::LogStash::Outputs::ElasticSearch::HttpClient::Pool::BadResponseCodeError.new(
-            413,
-            "test-url",
-            body,
-            ""
-          )
+          double("Response", :code => 413, :body => "")
         else
           double("Response", :code => 200, :body => '{"errors":false,"items":[{"index":{"status":200,"result":"created"}}]}')
         end
@@ -1130,81 +1125,6 @@ describe LogStash::Outputs::ElasticSearch do
     end
   end
 
-  describe "SSL deprecated settings" do
-    let(:base_options) { {"ssl" => "true"} }
-
-    context "with client certificate" do
-      let(:do_register) { true }
-      let(:cacert) { Stud::Temporary.file.path }
-      let(:options) { base_options.merge(
-        "cacert" => cacert,
-        "ssl_certificate_verification" => false
-      ) }
-
-      after :each do
-        File.delete(cacert)
-      end
-
-      it "should map new configs into params" do
-        expect(subject.params).to match hash_including(
-                                          "ssl_enabled" => true,
-                                          "ssl_verification_mode" => "none",
-                                          "ssl_certificate_authorities" => [cacert]
-                                        )
-      end
-
-      it "should set new configs variables" do
-        expect(subject.instance_variable_get(:@ssl_enabled)).to eql(true)
-        expect(subject.instance_variable_get(:@ssl_verification_mode)).to eql("none")
-        expect(subject.instance_variable_get(:@ssl_certificate_authorities)).to eql([cacert])
-      end
-    end
-
-    context "with java stores" do
-      let(:do_register) { true }
-      let(:keystore) { Stud::Temporary.file.path }
-      let(:truststore) { Stud::Temporary.file.path }
-      let(:options) { base_options.merge(
-        "keystore" => keystore,
-        "keystore_password" => "keystore",
-        "truststore" => truststore,
-        "truststore_password" => "truststore",
-        "ssl_certificate_verification" => true
-      ) }
-
-      let(:spy_http_client_builder!) do
-        allow(described_class::HttpClientBuilder).to receive(:build).with(any_args).and_call_original
-        allow(described_class::HttpClientBuilder).to receive(:setup_ssl).with(any_args).and_return({})
-      end
-
-      after :each do
-        File.delete(keystore)
-        File.delete(truststore)
-      end
-
-      it "should map new configs into params" do
-        expect(subject.params).to match hash_including(
-                                          "ssl_enabled" => true,
-                                          "ssl_keystore_path" => keystore,
-                                          "ssl_truststore_path" => truststore,
-                                          "ssl_verification_mode" => "full"
-                                        )
-
-        expect(subject.params["ssl_keystore_password"].value).to eql("keystore")
-        expect(subject.params["ssl_truststore_password"].value).to eql("truststore")
-      end
-
-      it "should set new configs variables" do
-        expect(subject.instance_variable_get(:@ssl_enabled)).to eql(true)
-        expect(subject.instance_variable_get(:@ssl_keystore_path)).to eql(keystore)
-        expect(subject.instance_variable_get(:@ssl_keystore_password).value).to eql("keystore")
-        expect(subject.instance_variable_get(:@ssl_truststore_path)).to eql(truststore)
-        expect(subject.instance_variable_get(:@ssl_truststore_password).value).to eql("truststore")
-        expect(subject.instance_variable_get(:@ssl_verification_mode)).to eql("full")
-      end
-    end
-  end
-
   describe "retry_on_conflict" do
     let(:num_retries) { 123 }
     let(:event) { LogStash::Event.new("myactionfield" => "update", "message" => "blah") }

data/spec/unit/outputs/elasticsearch_ssl_spec.rb

@@ -195,3 +195,25 @@ describe "SSL options" do
   end
 end
 
+# Move outside the SSL options describe block that has the after hook
+describe "SSL obsolete settings" do
+  let(:base_settings) { { "hosts" => "localhost", "pool_max" => 1, "pool_max_per_route" => 1 } }
+  [
+    {name: 'ssl', replacement: 'ssl_enabled'},
+    {name: 'ssl_certificate_verification', replacement: 'ssl_verification_mode'},
+    {name: 'cacert', replacement: 'ssl_certificate_authorities'},
+    {name: 'truststore', replacement: 'ssl_truststore_path'},
+    {name: 'keystore', replacement: 'ssl_keystore_path'},
+    {name: 'truststore_password', replacement: 'ssl_truststore_password'},
+    {name: 'keystore_password', replacement: 'ssl_keystore_password'}
+  ].each do |obsolete_setting|
+    context "with option #{obsolete_setting[:name]}" do
+      let(:settings) { base_settings.merge(obsolete_setting[:name] => "value") }
+
+      it "emits an error about the setting being obsolete" do
+        error_text = /The setting `#{obsolete_setting[:name]}` in plugin `elasticsearch` is obsolete and is no longer available. (Use|Set) '#{obsolete_setting[:replacement]}' instead/i
+        expect { LogStash::Outputs::ElasticSearch.new(settings) }.to raise_error LogStash::ConfigurationError, error_text
+      end
+    end
+  end
+end

data/spec/unit/outputs/error_whitelist_spec.rb

@@ -4,6 +4,7 @@ require_relative "../../../spec/es_spec_helper"
 describe "whitelisting error types in expected behavior" do
   let(:template) { '{"template" : "not important, will be updated by :index"}' }
   let(:event1) { LogStash::Event.new("somevalue" => 100, "@timestamp" => "2014-11-17T20:37:17.223Z") }
+  let(:action1) { ["index", {:_id=>1, :routing=>nil, :_index=>"logstash-2014.11.17", :_type=> doc_type }, event1] }
   let(:settings) { {"manage_template" => true, "index" => "logstash-2014.11.17", "template_overwrite" => true, "hosts" => get_host_port() } }
 
   subject { LogStash::Outputs::ElasticSearch.new(settings) }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-output-elasticsearch
 version: !ruby/object:Gem::Version
-  version: 11.22.13
+  version: 12.0.0
 platform: java
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-01 00:00:00.000000000 Z
+date: 2024-12-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
@@ -326,7 +326,9 @@ files:
 - spec/integration/outputs/update_spec.rb
 - spec/spec_helper.rb
 - spec/support/elasticsearch/api/actions/delete_ilm_policy.rb
+- spec/support/elasticsearch/api/actions/get_alias.rb
 - spec/support/elasticsearch/api/actions/get_ilm_policy.rb
+- spec/support/elasticsearch/api/actions/put_alias.rb
 - spec/support/elasticsearch/api/actions/put_ilm_policy.rb
 - spec/unit/http_client_builder_spec.rb
 - spec/unit/outputs/elasticsearch/data_stream_support_spec.rb
@@ -413,7 +415,9 @@ test_files:
 - spec/integration/outputs/update_spec.rb
 - spec/spec_helper.rb
 - spec/support/elasticsearch/api/actions/delete_ilm_policy.rb
+- spec/support/elasticsearch/api/actions/get_alias.rb
 - spec/support/elasticsearch/api/actions/get_ilm_policy.rb
+- spec/support/elasticsearch/api/actions/put_alias.rb
 - spec/support/elasticsearch/api/actions/put_ilm_policy.rb
 - spec/unit/http_client_builder_spec.rb
 - spec/unit/outputs/elasticsearch/data_stream_support_spec.rb