@ teracyhq#462 | should make sure settings is immutable after being b…

…uilt
hoatle · Oct 27, 2018 · f7acd55 · f7acd55
1 parent 18bcf8d
commit f7acd55
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/teracy-dev/config/manager.rb b/lib/teracy-dev/config/manager.rb
@@ -37,7 +37,7 @@ def configure(settings, config, type:)
 
         TeracyDev::Util.multi_sort(@items, weight: :desc, id: :asc).each do |item|
           configurator = item[:configurator]
-          configurator.configure(settings, config, type: type)
+          configurator.configure(Util.deep_copy(settings).freeze, config, type: type)
         end
       end
 

diff --git a/lib/teracy-dev/loader.rb b/lib/teracy-dev/loader.rb
@@ -174,12 +174,12 @@ def configure(settings, config, type:)
     def configure_vagrant(settings)
       Vagrant.configure("2") do |common|
 
-        configure(Util.deep_copy(settings).freeze, common, type: 'common')
+        configure(settings, common, type: 'common')
         settings['nodes'].each do |node_settings|
           primary = node_settings['primary'] ||= false
           autostart = node_settings['autostart'] === false ? false : true
           common.vm.define node_settings['name'], primary: primary, autostart: autostart do |node|
-            configure(Util.deep_copy(node_settings).freeze, node, type: 'node')
+            configure(node_settings, node, type: 'node')
           end
         end
       end