Merge pull request #118 from osbre/vary-header

Set the correct `Vary` header

Author: bknoles

lib/inertia_rails/renderer.rb

@@ -17,7 +17,7 @@ def initialize(component, controller, request, response, render_method, props: n
 
     def render
       if @request.headers['X-Inertia']
-        @response.set_header('Vary', 'Accept')
+        @response.set_header('Vary', 'X-Inertia')
         @response.set_header('X-Inertia', 'true')
         @render_method.call json: page, status: @response.status, content_type: Mime[:json]
       else

spec/inertia/rendering_spec.rb

@@ -49,7 +49,7 @@
 
     it 'has the proper headers' do
       expect(response.headers['X-Inertia']).to eq 'true'
-      expect(response.headers['Vary']).to eq 'Accept'
+      expect(response.headers['Vary']).to eq 'X-Inertia'
       expect(response.headers['Content-Type']).to eq 'application/json; charset=utf-8'
     end
 

spec/inertia/ssr_spec.rb

@@ -42,7 +42,7 @@
       it 'allows inertia to take over when inertia headers are passed' do
         get props_path, headers: {'X-Inertia' => true, 'X-Inertia-Version' => '1.0'}
 
-        expect(response.headers['Vary']).to eq 'Accept'
+        expect(response.headers['Vary']).to eq 'X-Inertia'
         expect(response.headers['Content-Type']).to eq 'application/json; charset=utf-8'
       end
     end