Call dup not to overwrite original value

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
fluent · Jan 24, 2020 · abb50a8 · abb50a8
1 parent 38f80c5
commit abb50a8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/fluent/config/section.rb b/lib/fluent/config/section.rb
@@ -58,6 +58,10 @@ def to_h
         @params
       end
 
+      def dup
+        Section.new(@params.dup, @corresponding_config_element.dup)
+      end
+
       def +(other)
         Section.new(self.to_h.merge(other.to_h))
       end

diff --git a/lib/fluent/plugin_helper/http_server.rb b/lib/fluent/plugin_helper/http_server.rb
@@ -115,7 +115,7 @@ def stop
       private
 
       def _http_server_overwrite_config(config, opts)
-        conf = config
+        conf = config.dup
         Fluent::PluginHelper::Server::SERVER_TRANSPORT_PARAMS.map(&:to_s).each do |param|
           if opts.key?(param)
             conf[param] = opts[param]