Don't set Cache-Control for responses that don't use streams

Cache-Control HTTP header with `no-cache`, even if the response wasn't a
stream. To fix this, we only set HTTP headers if there is an actual
stream.

Closes ruby-grape#2087

CHANGELOG.md

@@ -6,7 +6,7 @@
 
 #### Fixes
 
-* Your contribution here.
+* [#2083](https://github.com/ruby-grape/grape/pull/2083): Don't set Cache-Control for endpoints that aren't streams - [@stanhu](https://github.com/stanhu).
 
 ### 1.4.0 (2020/07/10)
 

lib/grape/dsl/inside_route.rb

@@ -328,6 +328,8 @@ def sendfile(value = nil)
       # * https://github.com/rack/rack/blob/99293fa13d86cd48021630fcc4bd5acc9de5bdc3/lib/rack/chunked.rb
       # * https://github.com/rack/rack/blob/99293fa13d86cd48021630fcc4bd5acc9de5bdc3/lib/rack/etag.rb
       def stream(value = nil)
+        return if value.nil? && !instance_variable_defined?(:@stream)
+
         header 'Content-Length', nil
         header 'Transfer-Encoding', nil
         header 'Cache-Control', 'no-cache' # Skips ETag generation (reading the response up front)
@@ -339,7 +341,7 @@ def stream(value = nil)
         elsif !value.is_a?(NilClass)
           raise ArgumentError, 'Stream object must respond to :each.'
         else
-          instance_variable_defined?(:@stream) ? @stream : nil
+          @stream
         end
       end
 

spec/grape/dsl/inside_route_spec.rb

@@ -419,6 +419,7 @@ def initialize
 
     it 'returns default' do
       expect(subject.stream).to be nil
+      expect(subject.header['Cache-Control']).to eq nil
     end
   end