search

Azure / logstash-output-kusto

Logstash output for Kusto
Apache License 2.0
13 stars 14 forks source link

replacing .empty? with .nil? in test for app_key and app_id #60

Closed gbordier closed 1 year ago

gbordier commented 1 year ago

.empty does not exist on nil class, therefore removing app_key and app_id from the output conf and setting managed_identity does not work (exception)

github-actions[bot] commented 1 year ago

Unit Test Results

  1 files  ±0    1 suites  ±0   3s :stopwatch: ±0s 22 tests ±0  22 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 1f94e0fb. ± Comparison against base commit 7f8fcc83.

github-actions[bot] commented 1 year ago

Unit Test Results

  1 files  ±0    1 suites  ±0   3s :stopwatch: ±0s 22 tests ±0  22 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 1f94e0fb. ± Comparison against base commit 7f8fcc83.

github-actions[bot] commented 1 year ago

Unit Test Results

  1 files  ±0    1 suites  ±0   3s :stopwatch: ±0s 22 tests ±0  22 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 1f94e0fb. ± Comparison against base commit 7f8fcc83.