Make sure to dup the headers coming from respond_with

koudelka · Aug 27, 2011 · b3ee0fe · b3ee0fe
1 parent 6841627
commit b3ee0fe
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/cramp/abstract.rb b/lib/cramp/abstract.rb
@@ -44,7 +44,7 @@ def send_headers
     end
 
     def build_headers
-      status, headers = respond_to?(:respond_with, true) ? respond_with : [200, {'Content-Type' => 'text/html'}]
+      status, headers = respond_to?(:respond_with, true) ? respond_with.dup : [200, {'Content-Type' => 'text/html'}]
       headers['Connection'] ||= 'keep-alive'
       [status, headers]
     end

diff --git a/lib/cramp/action.rb b/lib/cramp/action.rb
@@ -38,10 +38,12 @@ def build_headers
         [status, headers.merge(self.default_sse_headers)]
       when :chunked
         status, headers = respond_to?(:respond_with, true) ? respond_with : [200, {}]
+
+        headers = headers.merge(self.default_chunked_headers)
         headers['Content-Type'] ||= 'text/html'
         headers['Cache-Control'] ||= 'no-cache'
 
-        [status, headers.merge(self.default_chunked_headers)]
+        [status, headers]
       else
         super
       end