fix: the elasticsearch.update_aliases event must include body

marcosgz · Nov 27, 2023 · df13590 · df13590
1 parent 6489028
commit df13590
Show file tree

Hide file tree

Showing 12 changed files with 12 additions and 12 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-1.x.lock b/gemfiles/Gemfile.elasticsearch-1.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-2.x.lock b/gemfiles/Gemfile.elasticsearch-2.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-5.x.lock b/gemfiles/Gemfile.elasticsearch-5.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-6.x.lock b/gemfiles/Gemfile.elasticsearch-6.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-7.x.lock b/gemfiles/Gemfile.elasticsearch-7.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.elasticsearch-8.x.lock b/gemfiles/Gemfile.elasticsearch-8.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.opensearch-1.x.lock b/gemfiles/Gemfile.opensearch-1.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/gemfiles/Gemfile.opensearch-2.x.lock b/gemfiles/Gemfile.opensearch-2.x.lock
@@ -1,7 +1,7 @@
 PATH
   remote: ..
   specs:
-    esse (0.2.4)
+    esse (0.2.5)
       multi_json
       thor (>= 0.19)
 

diff --git a/lib/esse/cli/event_listener.rb b/lib/esse/cli/event_listener.rb
@@ -63,7 +63,7 @@ def elasticsearch_update_settings(event)
       end
 
       def elasticsearch_update_aliases(event)
-        actions = event[:request][:body][:actions]
+        actions = event[:request].dig(:body, :actions) || []
         removed = actions.select { |a| a.key?(:remove) }
         added = actions.select { |a| a.key?(:add) }
         print_message '[%<runtime>s] Successfuly updated aliases:',

diff --git a/lib/esse/transport/aliases.rb b/lib/esse/transport/aliases.rb
@@ -25,7 +25,7 @@ def update_aliases(body:, **options)
         throw_error_when_readonly!
 
         Esse::Events.instrument('elasticsearch.update_aliases') do |payload|
-          payload[:request] = options
+          payload[:request] = options.merge(body: body)
           payload[:response] = coerce_exception { client.indices.update_aliases(**options, body: body) }
         end
       end

diff --git a/lib/esse/version.rb b/lib/esse/version.rb
@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Esse
-  VERSION = '0.2.4'
+  VERSION = '0.2.5'
 end