From bd5c50e8c2693bc2d04c7077cff0ebd6100b4c52 Mon Sep 17 00:00:00 2001 From: glaszig Date: Fri, 16 Aug 2024 00:27:54 -0300 Subject: [PATCH] leave cache-control header alone in its entirety if disable_caching = false closes #616 --- lib/mini_profiler.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/mini_profiler.rb b/lib/mini_profiler.rb index 3eb012bc..c70344a2 100644 --- a/lib/mini_profiler.rb +++ b/lib/mini_profiler.rb @@ -425,14 +425,14 @@ def inject_profiler(env, status, headers, body) # Rack::ETag has already inserted some nonesense in the chain content_type = headers[Rack::CONTENT_TYPE] + headers['x-miniprofiler-original-cache-control'] = headers[Rack::CACHE_CONTROL] unless headers[Rack::CACHE_CONTROL].nil? + if config.disable_caching headers.delete(Rack::ETAG) headers.delete('date') || headers.delete('Date') + headers[Rack::CACHE_CONTROL] = "no-store, must-revalidate, private, max-age=0" end - headers['x-miniprofiler-original-cache-control'] = headers[Rack::CACHE_CONTROL] unless headers[Rack::CACHE_CONTROL].nil? - headers[Rack::CACHE_CONTROL] = "#{"no-store, " if config.disable_caching}must-revalidate, private, max-age=0" - # inject header if headers.is_a? Hash headers['x-miniprofiler-ids'] = ids_comma_separated(env)