logstash-output-kusto 1.0.5-java → 1.0.6-java

data/lib/logstash-output-kusto_jars.rb

@@ -4,9 +4,7 @@ begin
 rescue LoadError
   require 'io/netty/netty-resolver-dns-native-macos/4.1.68.Final/netty-resolver-dns-native-macos-4.1.68.Final-osx-x86_64.jar'
   require 'com/google/guava/guava/24.1.1-jre/guava-24.1.1-jre.jar'
-  require 'com/microsoft/azure/kusto/kusto-data/3.1.3/kusto-data-3.1.3.jar'
   require 'com/microsoft/azure/azure-storage/8.6.6/azure-storage-8.6.6.jar'
-  require 'com/microsoft/azure/kusto/kusto-ingest/3.1.3/kusto-ingest-3.1.3.jar'
   require 'io/projectreactor/netty/reactor-netty-http/1.0.11/reactor-netty-http-1.0.11.jar'
   require 'net/java/dev/jna/jna/5.5.0/jna-5.5.0.jar'
   require 'io/netty/netty-handler-proxy/4.1.68.Final/netty-handler-proxy-4.1.68.Final.jar'
@@ -15,20 +13,22 @@ rescue LoadError
   require 'org/apache/commons/commons-lang3/3.11/commons-lang3-3.11.jar'
   require 'org/reactivestreams/reactive-streams/1.0.3/reactive-streams-1.0.3.jar'
   require 'com/microsoft/azure/azure-keyvault-core/1.2.4/azure-keyvault-core-1.2.4.jar'
+  require 'com/fasterxml/jackson/core/jackson-core/2.12.5/jackson-core-2.12.5.jar'
   require 'io/github/resilience4j/resilience4j-core/1.7.1/resilience4j-core-1.7.1.jar'
   require 'io/netty/netty-codec-dns/4.1.68.Final/netty-codec-dns-4.1.68.Final.jar'
   require 'com/nimbusds/nimbus-jose-jwt/9.9.3/nimbus-jose-jwt-9.9.3.jar'
   require 'io/netty/netty-transport/4.1.68.Final/netty-transport-4.1.68.Final.jar'
+  require 'com/fasterxml/jackson/core/jackson-annotations/2.12.5/jackson-annotations-2.12.5.jar'
   require 'org/jetbrains/annotations/22.0.0/annotations-22.0.0.jar'
   require 'io/netty/netty-transport-native-unix-common/4.1.68.Final/netty-transport-native-unix-common-4.1.68.Final.jar'
   require 'io/netty/netty-transport-native-epoll/4.1.68.Final/netty-transport-native-epoll-4.1.68.Final-linux-x86_64.jar'
+  require 'com/microsoft/azure/kusto/kusto-ingest/3.2.1/kusto-ingest-3.2.1.jar'
   require 'io/vavr/vavr/0.10.2/vavr-0.10.2.jar'
   require 'io/netty/netty-resolver-dns/4.1.68.Final/netty-resolver-dns-4.1.68.Final.jar'
   require 'com/azure/azure-identity/1.3.7/azure-identity-1.3.7.jar'
   require 'com/fasterxml/jackson/module/jackson-module-jaxb-annotations/2.12.5/jackson-module-jaxb-annotations-2.12.5.jar'
-  require 'org/apache/commons/commons-text/1.9/commons-text-1.9.jar'
+  require 'com/fasterxml/jackson/core/jackson-databind/2.12.5/jackson-databind-2.12.5.jar'
   require 'net/java/dev/jna/jna-platform/5.6.0/jna-platform-5.6.0.jar'
-  require 'com/fasterxml/jackson/core/jackson-core/2.13.0/jackson-core-2.13.0.jar'
   require 'io/netty/netty-resolver/4.1.68.Final/netty-resolver-4.1.68.Final.jar'
   require 'jakarta/activation/jakarta.activation-api/1.2.1/jakarta.activation-api-1.2.1.jar'
   require 'commons-logging/commons-logging/1.2/commons-logging-1.2.jar'
@@ -37,11 +37,10 @@ rescue LoadError
   require 'commons-codec/commons-codec/1.11/commons-codec-1.11.jar'
   require 'com/microsoft/azure/msal4j-persistence-extension/1.1.0/msal4j-persistence-extension-1.1.0.jar'
   require 'com/fasterxml/woodstox/woodstox-core/6.2.4/woodstox-core-6.2.4.jar'
+  require 'org/apache/commons/commons-text/1.10.0/commons-text-1.10.0.jar'
   require 'com/github/stephenc/jcip/jcip-annotations/1.0-1/jcip-annotations-1.0-1.jar'
-  require 'com/fasterxml/jackson/core/jackson-databind/2.13.0/jackson-databind-2.13.0.jar'
   require 'io/netty/netty-transport-native-kqueue/4.1.68.Final/netty-transport-native-kqueue-4.1.68.Final-osx-x86_64.jar'
   require 'io/netty/netty-handler/4.1.68.Final/netty-handler-4.1.68.Final.jar'
-  require 'com/fasterxml/jackson/core/jackson-annotations/2.13.0/jackson-annotations-2.13.0.jar'
   require 'com/google/code/findbugs/jsr305/1.3.9/jsr305-1.3.9.jar'
   require 'org/checkerframework/checker-compat-qual/2.0.0/checker-compat-qual-2.0.0.jar'
   require 'org/apache/httpcomponents/httpclient/4.5.13/httpclient-4.5.13.jar'
@@ -65,6 +64,7 @@ rescue LoadError
   require 'io/vavr/vavr-match/0.10.2/vavr-match-0.10.2.jar'
   require 'com/nimbusds/content-type/2.1/content-type-2.1.jar'
   require 'org/slf4j/slf4j-api/1.8.0-beta4/slf4j-api-1.8.0-beta4.jar'
+  require 'com/microsoft/azure/kusto/kusto-data/3.2.1/kusto-data-3.2.1.jar'
   require 'com/microsoft/azure/msal4j/1.11.0/msal4j-1.11.0.jar'
   require 'com/nimbusds/lang-tag/1.5/lang-tag-1.5.jar'
   require 'com/azure/azure-core/1.21.0/azure-core-1.21.0.jar'
@@ -78,9 +78,7 @@ end
 if defined? Jars
   require_jar 'io.netty', 'netty-resolver-dns-native-macos', 'osx-x86_64', '4.1.68.Final'
   require_jar 'com.google.guava', 'guava', '24.1.1-jre'
-  require_jar 'com.microsoft.azure.kusto', 'kusto-data', '3.1.3'
   require_jar 'com.microsoft.azure', 'azure-storage', '8.6.6'
-  require_jar 'com.microsoft.azure.kusto', 'kusto-ingest', '3.1.3'
   require_jar 'io.projectreactor.netty', 'reactor-netty-http', '1.0.11'
   require_jar 'net.java.dev.jna', 'jna', '5.5.0'
   require_jar 'io.netty', 'netty-handler-proxy', '4.1.68.Final'
@@ -89,20 +87,22 @@ if defined? Jars
   require_jar 'org.apache.commons', 'commons-lang3', '3.11'
   require_jar 'org.reactivestreams', 'reactive-streams', '1.0.3'
   require_jar 'com.microsoft.azure', 'azure-keyvault-core', '1.2.4'
+  require_jar 'com.fasterxml.jackson.core', 'jackson-core', '2.12.5'
   require_jar 'io.github.resilience4j', 'resilience4j-core', '1.7.1'
   require_jar 'io.netty', 'netty-codec-dns', '4.1.68.Final'
   require_jar 'com.nimbusds', 'nimbus-jose-jwt', '9.9.3'
   require_jar 'io.netty', 'netty-transport', '4.1.68.Final'
+  require_jar 'com.fasterxml.jackson.core', 'jackson-annotations', '2.12.5'
   require_jar 'org.jetbrains', 'annotations', '22.0.0'
   require_jar 'io.netty', 'netty-transport-native-unix-common', '4.1.68.Final'
   require_jar 'io.netty', 'netty-transport-native-epoll', 'linux-x86_64', '4.1.68.Final'
+  require_jar 'com.microsoft.azure.kusto', 'kusto-ingest', '3.2.1'
   require_jar 'io.vavr', 'vavr', '0.10.2'
   require_jar 'io.netty', 'netty-resolver-dns', '4.1.68.Final'
   require_jar 'com.azure', 'azure-identity', '1.3.7'
   require_jar 'com.fasterxml.jackson.module', 'jackson-module-jaxb-annotations', '2.12.5'
-  require_jar 'org.apache.commons', 'commons-text', '1.9'
+  require_jar 'com.fasterxml.jackson.core', 'jackson-databind', '2.12.5'
   require_jar 'net.java.dev.jna', 'jna-platform', '5.6.0'
-  require_jar 'com.fasterxml.jackson.core', 'jackson-core', '2.13.0'
   require_jar 'io.netty', 'netty-resolver', '4.1.68.Final'
   require_jar 'jakarta.activation', 'jakarta.activation-api', '1.2.1'
   require_jar 'commons-logging', 'commons-logging', '1.2'
@@ -111,11 +111,10 @@ if defined? Jars
   require_jar 'commons-codec', 'commons-codec', '1.11'
   require_jar 'com.microsoft.azure', 'msal4j-persistence-extension', '1.1.0'
   require_jar 'com.fasterxml.woodstox', 'woodstox-core', '6.2.4'
+  require_jar 'org.apache.commons', 'commons-text', '1.10.0'
   require_jar 'com.github.stephenc.jcip', 'jcip-annotations', '1.0-1'
-  require_jar 'com.fasterxml.jackson.core', 'jackson-databind', '2.13.0'
   require_jar 'io.netty', 'netty-transport-native-kqueue', 'osx-x86_64', '4.1.68.Final'
   require_jar 'io.netty', 'netty-handler', '4.1.68.Final'
-  require_jar 'com.fasterxml.jackson.core', 'jackson-annotations', '2.13.0'
   require_jar 'com.google.code.findbugs', 'jsr305', '1.3.9'
   require_jar 'org.checkerframework', 'checker-compat-qual', '2.0.0'
   require_jar 'org.apache.httpcomponents', 'httpclient', '4.5.13'
@@ -139,6 +138,7 @@ if defined? Jars
   require_jar 'io.vavr', 'vavr-match', '0.10.2'
   require_jar 'com.nimbusds', 'content-type', '2.1'
   require_jar 'org.slf4j', 'slf4j-api', '1.8.0-beta4'
+  require_jar 'com.microsoft.azure.kusto', 'kusto-data', '3.2.1'
   require_jar 'com.microsoft.azure', 'msal4j', '1.11.0'
   require_jar 'com.nimbusds', 'lang-tag', '1.5'
   require_jar 'com.azure', 'azure-core', '1.21.0'

data/logstash-output-kusto.gemspec

@@ -1,36 +1,36 @@
-Gem::Specification.new do |s|
-  s.name          = 'logstash-output-kusto' #WATCH OUT: we hardcoded usage of this name in one of the classes.
-  s.version       = '1.0.5'
-  s.licenses      = ['Apache-2.0']
-  s.summary       = 'Writes events to Azure Data Explorer (Kusto)'
-  s.description   = 'This is a logstash output plugin used to write events to an Azure Data Explorer (a.k.a Kusto)'
-  s.homepage      = 'https://github.com/Azure/logstash-output-kusto'
-  s.authors       = ['Tamir Kamara', 'Asaf Mahlev']
-  s.email         = 'nugetkusto@microsoft.com'
-  s.require_paths = ['lib']
-  s.platform = 'java'
-
-  # Files
-  s.files = Dir['lib/**/*', 'spec/**/*', 'vendor/**/*', '*.gemspec', '*.md', 'CONTRIBUTORS', 'Gemfile', 'LICENSE', 'NOTICE.TXT']
-
-  # Tests
-  s.test_files = s.files.grep(%r{^(test|spec|features)/})
-
-  # Special flag to let us know this is actually a logstash plugin
-  s.metadata = { "logstash_plugin" => "true", "logstash_group" => "output" }
-
-  # Gem dependencies
-  s.add_runtime_dependency 'logstash-core-plugin-api', '~> 2.0'
-  s.add_runtime_dependency 'logstash-codec-json_lines'
-  s.add_runtime_dependency 'logstash-codec-line'
-
-  s.add_development_dependency 'logstash-devutils'
-  s.add_development_dependency 'flores'
-  s.add_development_dependency 'logstash-input-generator'
-  s.add_development_dependency 'ruby-maven', '~> 3.3.11'
-  s.add_development_dependency 'rspec_junit_formatter'
-
-  # Jar dependencies
-  s.requirements << "jar 'com.microsoft.azure.kusto, kusto-ingest, 3.1.3'"
-  s.add_runtime_dependency 'jar-dependencies'
-end
+Gem::Specification.new do |s|
+  s.name          = 'logstash-output-kusto' #WATCH OUT: we hardcoded usage of this name in one of the classes.
+  s.version       = '1.0.6'
+  s.licenses      = ['Apache-2.0']
+  s.summary       = 'Writes events to Azure Data Explorer (Kusto)'
+  s.description   = 'This is a logstash output plugin used to write events to an Azure Data Explorer (a.k.a Kusto)'
+  s.homepage      = 'https://github.com/Azure/logstash-output-kusto'
+  s.authors       = ['Tamir Kamara', 'Asaf Mahlev']
+  s.email         = 'nugetkusto@microsoft.com'
+  s.require_paths = ['lib']
+  s.platform = 'java'
+
+  # Files
+  s.files = Dir['lib/**/*', 'spec/**/*', 'vendor/**/*', '*.gemspec', '*.md', 'CONTRIBUTORS', 'Gemfile', 'LICENSE', 'NOTICE.TXT']
+
+  # Tests
+  s.test_files = s.files.grep(%r{^(test|spec|features)/})
+
+  # Special flag to let us know this is actually a logstash plugin
+  s.metadata = { "logstash_plugin" => "true", "logstash_group" => "output" }
+
+  # Gem dependencies
+  s.add_runtime_dependency 'logstash-core-plugin-api', '~> 2.0'
+  s.add_runtime_dependency 'logstash-codec-json_lines'
+  s.add_runtime_dependency 'logstash-codec-line'
+
+  s.add_development_dependency 'logstash-devutils'
+  s.add_development_dependency 'flores'
+  s.add_development_dependency 'logstash-input-generator'
+  s.add_development_dependency 'ruby-maven', '~> 3.3.11'
+  s.add_development_dependency 'rspec_junit_formatter'
+
+  # Jar dependencies
+  s.requirements << "jar 'com.microsoft.azure.kusto, kusto-ingest, 3.2.1'"
+  s.add_runtime_dependency 'jar-dependencies'
+end

data/spec/outputs/kusto/ingestor_spec.rb

@@ -1,121 +1,121 @@
-# encoding: utf-8
-require_relative "../../spec_helpers.rb"
-require 'logstash/outputs/kusto'
-require 'logstash/outputs/kusto/ingestor'
-
-describe LogStash::Outputs::Kusto::Ingestor do
-
-  let(:ingest_url) { "https://ingest-sdkse2etest.eastus.kusto.windows.net/" }
-  let(:app_id) { "myid" }
-  let(:app_key) { LogStash::Util::Password.new("mykey") }
-  let(:app_tenant) { "mytenant" }
-  let(:database) { "mydatabase" }
-  let(:table) { "mytable" }
-  let(:proxy_host) { "localhost" }
-  let(:proxy_port) { 80 }
-  let(:proxy_protocol) { "http" }
-  let(:json_mapping) { "mymapping" }
-  let(:delete_local) { false }
-  let(:logger) { spy('logger') }
-
-  describe '#initialize' do
-
-    it 'does not throw an error when initializing' do
-      # note that this will cause an internal error since connection is being tried.
-      # however we still want to test that all the java stuff is working as expected
-      expect { 
-        ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol, logger)
-        ingestor.stop
-      }.not_to raise_error
-    end
-    
-    dynamic_name_array = ['/a%{name}/', '/a %{name}/', '/a- %{name}/', '/a- %{name}']
-
-    context 'doesnt allow database to have some dynamic part' do
-      dynamic_name_array.each do |test_database|
-        it "with database: #{test_database}" do
-          expect {
-            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, test_database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
-            ingestor.stop
-          }.to raise_error(LogStash::ConfigurationError)          
-        end
-      end
-    end
-
-    context 'doesnt allow table to have some dynamic part' do
-      dynamic_name_array.each do |test_table|
-        it "with database: #{test_table}" do
-          expect {
-            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, test_table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
-            ingestor.stop
-          }.to raise_error(LogStash::ConfigurationError)          
-        end
-      end
-    end
-
-    context 'doesnt allow mapping to have some dynamic part' do
-      dynamic_name_array.each do |json_mapping|
-        it "with database: #{json_mapping}" do
-          expect {
-            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
-            ingestor.stop
-          }.to raise_error(LogStash::ConfigurationError)          
-        end
-      end
-    end
-
-    context 'proxy protocol has to be http or https' do
-      it "with proxy protocol: socks" do
-        expect {
-          ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,'socks',logger)
-          ingestor.stop
-        }.to raise_error(LogStash::ConfigurationError)          
-      end
-    end
-
-  end
-
-  # describe 'receiving events' do
-
-  #   context 'with non-zero flush interval' do
-  #     let(:temporary_output_file) { Stud::Temporary.pathname }
-
-  #     let(:event_count) { 100 }
-  #     let(:flush_interval) { 5 }
-
-  #     let(:events) do
-  #       event_count.times.map do |idx|
-  #         LogStash::Event.new('subject' => idx)
-  #       end
-  #     end
-
-  #     let(:output) { described_class.new(options.merge( {'path' => temporary_output_file, 'flush_interval' => flush_interval, 'delete_temp_files' => false } )) }
-
-  #     before(:each) { output.register }
-      
-  #     after(:each) do
-  #       output.close
-  #       File.exist?(temporary_output_file) && File.unlink(temporary_output_file)
-  #       File.exist?(temporary_output_file + '.kusto') && File.unlink(temporary_output_file + '.kusto')
-  #     end
-
-  #     it 'eventually flushes without receiving additional events' do
-  #       output.multi_receive_encoded(events)
-
-  #       # events should not all be flushed just yet...
-  #       expect(File.read(temporary_output_file)).to satisfy("have less than #{event_count} lines") do |contents|
-  #         contents && contents.lines.count < event_count
-  #       end
-
-  #       # wait for the flusher to run...
-  #       sleep(flush_interval + 1)
-
-  #       # events should all be flushed
-  #       expect(File.read(temporary_output_file)).to satisfy("have exactly #{event_count} lines") do |contents|
-  #         contents && contents.lines.count == event_count
-  #       end
-  #     end
-  #   end
-
-  # end
-end
+# encoding: utf-8
+require_relative "../../spec_helpers.rb"
+require 'logstash/outputs/kusto'
+require 'logstash/outputs/kusto/ingestor'
+
+describe LogStash::Outputs::Kusto::Ingestor do
+
+  let(:ingest_url) { "https://ingest-sdkse2etest.eastus.kusto.windows.net/" }
+  let(:app_id) { "myid" }
+  let(:app_key) { LogStash::Util::Password.new("mykey") }
+  let(:app_tenant) { "mytenant" }
+  let(:database) { "mydatabase" }
+  let(:table) { "mytable" }
+  let(:proxy_host) { "localhost" }
+  let(:proxy_port) { 80 }
+  let(:proxy_protocol) { "http" }
+  let(:json_mapping) { "mymapping" }
+  let(:delete_local) { false }
+  let(:logger) { spy('logger') }
+
+  describe '#initialize' do
+
+    it 'does not throw an error when initializing' do
+      # note that this will cause an internal error since connection is being tried.
+      # however we still want to test that all the java stuff is working as expected
+      expect { 
+        ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol, logger)
+        ingestor.stop
+      }.not_to raise_error
+    end
+    
+    dynamic_name_array = ['/a%{name}/', '/a %{name}/', '/a- %{name}/', '/a- %{name}']
+
+    context 'doesnt allow database to have some dynamic part' do
+      dynamic_name_array.each do |test_database|
+        it "with database: #{test_database}" do
+          expect {
+            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, test_database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
+            ingestor.stop
+          }.to raise_error(LogStash::ConfigurationError)          
+        end
+      end
+    end
+
+    context 'doesnt allow table to have some dynamic part' do
+      dynamic_name_array.each do |test_table|
+        it "with database: #{test_table}" do
+          expect {
+            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, test_table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
+            ingestor.stop
+          }.to raise_error(LogStash::ConfigurationError)          
+        end
+      end
+    end
+
+    context 'doesnt allow mapping to have some dynamic part' do
+      dynamic_name_array.each do |json_mapping|
+        it "with database: #{json_mapping}" do
+          expect {
+            ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,proxy_protocol,logger)
+            ingestor.stop
+          }.to raise_error(LogStash::ConfigurationError)          
+        end
+      end
+    end
+
+    context 'proxy protocol has to be http or https' do
+      it "with proxy protocol: socks" do
+        expect {
+          ingestor = described_class.new(ingest_url, app_id, app_key, app_tenant, database, table, json_mapping, delete_local, proxy_host, proxy_port,'socks',logger)
+          ingestor.stop
+        }.to raise_error(LogStash::ConfigurationError)          
+      end
+    end
+
+  end
+
+  # describe 'receiving events' do
+
+  #   context 'with non-zero flush interval' do
+  #     let(:temporary_output_file) { Stud::Temporary.pathname }
+
+  #     let(:event_count) { 100 }
+  #     let(:flush_interval) { 5 }
+
+  #     let(:events) do
+  #       event_count.times.map do |idx|
+  #         LogStash::Event.new('subject' => idx)
+  #       end
+  #     end
+
+  #     let(:output) { described_class.new(options.merge( {'path' => temporary_output_file, 'flush_interval' => flush_interval, 'delete_temp_files' => false } )) }
+
+  #     before(:each) { output.register }
+      
+  #     after(:each) do
+  #       output.close
+  #       File.exist?(temporary_output_file) && File.unlink(temporary_output_file)
+  #       File.exist?(temporary_output_file + '.kusto') && File.unlink(temporary_output_file + '.kusto')
+  #     end
+
+  #     it 'eventually flushes without receiving additional events' do
+  #       output.multi_receive_encoded(events)
+
+  #       # events should not all be flushed just yet...
+  #       expect(File.read(temporary_output_file)).to satisfy("have less than #{event_count} lines") do |contents|
+  #         contents && contents.lines.count < event_count
+  #       end
+
+  #       # wait for the flusher to run...
+  #       sleep(flush_interval + 1)
+
+  #       # events should all be flushed
+  #       expect(File.read(temporary_output_file)).to satisfy("have exactly #{event_count} lines") do |contents|
+  #         contents && contents.lines.count == event_count
+  #       end
+  #     end
+  #   end
+
+  # end
+end

data/spec/outputs/kusto_spec.rb

@@ -1,56 +1,56 @@
-# encoding: utf-8
-require 'logstash/outputs/kusto'
-require 'logstash/codecs/plain'
-require 'logstash/event'
-
-describe LogStash::Outputs::Kusto do
-
-  let(:options) { { "path" => "./kusto_tst/%{+YYYY-MM-dd-HH-mm}",
-    "ingest_url" => "https://ingest-sdkse2etest.eastus.kusto.windows.net/",
-    "app_id" => "myid",
-    "app_key" => "mykey",
-    "app_tenant" => "mytenant",
-    "database" => "mydatabase",
-    "table" => "mytable",
-    "json_mapping" => "mymapping",
-    "proxy_host" => "localhost",
-    "proxy_port" => 3128,
-    "proxy_protocol" => "https"
-  } }
-
-  describe '#register' do
-
-    it 'doesnt allow the path to start with a dynamic string' do
-      kusto = described_class.new(options.merge( {'path' => '/%{name}'} ))
-      expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
-      kusto.close
-    end
-
-    it 'path must include a dynamic string to allow file rotation' do
-      kusto = described_class.new(options.merge( {'path' => '/{name}'} ))
-      expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
-      kusto.close
-    end
-
-
-    dynamic_name_array = ['/a%{name}/', '/a %{name}/', '/a- %{name}/', '/a- %{name}']
-
-    context 'doesnt allow the root directory to have some dynamic part' do
-      dynamic_name_array.each do |test_path|
-         it "with path: #{test_path}" do
-           kusto = described_class.new(options.merge( {'path' => test_path} ))
-           expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
-           kusto.close
-         end
-       end
-    end
-
-    it 'allow to have dynamic part after the file root' do
-      kusto = described_class.new(options.merge({'path' => '/tmp/%{name}'}))
-      expect { kusto.register }.not_to raise_error
-      kusto.close
-    end
-
-  end
-
-end
+# encoding: utf-8
+require 'logstash/outputs/kusto'
+require 'logstash/codecs/plain'
+require 'logstash/event'
+
+describe LogStash::Outputs::Kusto do
+
+  let(:options) { { "path" => "./kusto_tst/%{+YYYY-MM-dd-HH-mm}",
+    "ingest_url" => "https://ingest-sdkse2etest.eastus.kusto.windows.net/",
+    "app_id" => "myid",
+    "app_key" => "mykey",
+    "app_tenant" => "mytenant",
+    "database" => "mydatabase",
+    "table" => "mytable",
+    "json_mapping" => "mymapping",
+    "proxy_host" => "localhost",
+    "proxy_port" => 3128,
+    "proxy_protocol" => "https"
+  } }
+
+  describe '#register' do
+
+    it 'doesnt allow the path to start with a dynamic string' do
+      kusto = described_class.new(options.merge( {'path' => '/%{name}'} ))
+      expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
+      kusto.close
+    end
+
+    it 'path must include a dynamic string to allow file rotation' do
+      kusto = described_class.new(options.merge( {'path' => '/{name}'} ))
+      expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
+      kusto.close
+    end
+
+
+    dynamic_name_array = ['/a%{name}/', '/a %{name}/', '/a- %{name}/', '/a- %{name}']
+
+    context 'doesnt allow the root directory to have some dynamic part' do
+      dynamic_name_array.each do |test_path|
+         it "with path: #{test_path}" do
+           kusto = described_class.new(options.merge( {'path' => test_path} ))
+           expect { kusto.register }.to raise_error(LogStash::ConfigurationError)
+           kusto.close
+         end
+       end
+    end
+
+    it 'allow to have dynamic part after the file root' do
+      kusto = described_class.new(options.merge({'path' => '/tmp/%{name}'}))
+      expect { kusto.register }.not_to raise_error
+      kusto.close
+    end
+
+  end
+
+end

data/spec/spec_helpers.rb

@@ -1,21 +1,21 @@
-# encoding: utf-8
-require "logstash/devutils/rspec/spec_helper"
-require "logstash/logging/logger"
-
-LogStash::Logging::Logger::configure_logging("debug")
-
-RSpec.configure do |config|
-  # register around filter that captures stdout and stderr
-  config.around(:each) do |example|
-    $stdout = StringIO.new
-    $stderr = StringIO.new
-
-    example.run
-
-    example.metadata[:stdout] = $stdout.string
-    example.metadata[:stderr] = $stderr.string
-
-    $stdout = STDOUT
-    $stderr = STDERR
-  end
-end
+# encoding: utf-8
+require "logstash/devutils/rspec/spec_helper"
+require "logstash/logging/logger"
+
+LogStash::Logging::Logger::configure_logging("debug")
+
+RSpec.configure do |config|
+  # register around filter that captures stdout and stderr
+  config.around(:each) do |example|
+    $stdout = StringIO.new
+    $stderr = StringIO.new
+
+    example.run
+
+    example.metadata[:stdout] = $stdout.string
+    example.metadata[:stderr] = $stderr.string
+
+    $stdout = STDOUT
+    $stderr = STDERR
+  end
+end