fix memory leak where config array grew unbounded

joelhawksley · Jul 1, 2024 · 6f9cfe2 · 6f9cfe2
1 parent 1d42e04
commit 6f9cfe2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/apis/home_assistant_api.rb b/app/apis/home_assistant_api.rb
@@ -79,7 +79,7 @@ def package_present?
   def unavailable_door_sensors
     out = []
 
-    Timeframe::Application.config.local["home_assistant"]["exterior_door_sensors"].concat(
+    (Timeframe::Application.config.local["home_assistant"]["exterior_door_sensors"] +
       Timeframe::Application.config.local["home_assistant"]["exterior_door_locks"]
     ).each do |entity_id|
       if data.find { _1[:entity_id] == entity_id }&.fetch(:state) == "unavailable"

diff --git a/config/application.rb b/config/application.rb
@@ -28,7 +28,7 @@ def self.redis
       end
     end
 
-    config.local = YAML.load_file(Rails.root.join("config.yml"))
+    config.local = YAML.load_file(Rails.root.join("config.yml")).freeze
 
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration should go into files in config/initializers