Do not convert cache duration to hours

Hours is not fine enough as granularity for good cache control. For this reason, the `Cache-Control` header operates with a granularity of seconds as well. Let's also work with seconds here. This is literally a breaking change, but it will only break caches.
AlchemyCMS · May 15, 2024 · 2a0d06b · 2a0d06b
1 parent 7b1d334
commit 2a0d06b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/alchemy/json_api/nodes_controller.rb b/app/controllers/alchemy/json_api/nodes_controller.rb
@@ -17,7 +17,7 @@ def index
       private
 
       def cache_duration
-        ENV.fetch("ALCHEMY_JSON_API_CACHE_DURATION", 3).to_i.hours
+        ENV.fetch("ALCHEMY_JSON_API_CACHE_DURATION", 3).to_i
       end
 
       def jsonapi_meta(nodes)

diff --git a/app/controllers/alchemy/json_api/pages_controller.rb b/app/controllers/alchemy/json_api/pages_controller.rb
@@ -47,7 +47,7 @@ def render_pages_json(allowed)
       end
 
       def cache_duration
-        ENV.fetch("ALCHEMY_JSON_API_CACHE_DURATION", 3).to_i.hours
+        ENV.fetch("ALCHEMY_JSON_API_CACHE_DURATION", 3).to_i
       end
 
       def caching_options