Merge pull request #146 from skryukov/support-redirect-back-or-to

Respect `raise_on_open_redirects` Rails 7.0+ config

Author: bknoles

lib/inertia_rails/controller.rb

@@ -67,16 +67,7 @@ def default_render
 
     def redirect_to(options = {}, response_options = {})
       capture_inertia_errors(response_options)
-      super(options, response_options)
-    end
-
-    def redirect_back(fallback_location:, allow_other_host: true, **options)
-      capture_inertia_errors(options)
-      super(
-        fallback_location: fallback_location,
-        allow_other_host: allow_other_host,
-        **options,
-      )
+      super
     end
 
     private

spec/dummy/app/controllers/inertia_test_controller.rb

@@ -58,6 +58,13 @@ def redirect_back_with_inertia_errors
     )
   end
 
+  def redirect_back_or_to_with_inertia_errors
+    redirect_back_or_to(
+      empty_test_path,
+      inertia: { errors: { go: 'back!' } }
+    )
+  end
+
   def error_404
     render inertia: 'ErrorComponent', status: 404
   end

spec/dummy/config/routes.rb

@@ -25,6 +25,7 @@
   post 'redirect_with_inertia_errors' => 'inertia_test#redirect_with_inertia_errors'
   post 'redirect_with_inertia_error_object' => 'inertia_test#redirect_with_inertia_error_object'
   post 'redirect_back_with_inertia_errors' => 'inertia_test#redirect_back_with_inertia_errors'
+  post 'redirect_back_or_to_with_inertia_errors' => 'inertia_test#redirect_back_or_to_with_inertia_errors'
   get 'error_404' => 'inertia_test#error_404'
   get 'error_500' => 'inertia_test#error_500'
   get 'content_type_test' => 'inertia_test#content_type_test'

spec/inertia/response_spec.rb

@@ -48,11 +48,32 @@
             redirect_back_with_inertia_errors_path,
             headers: {
               'X-Inertia' => true,
-              'HTTP_REFERER' => "http://example.com/current-path"
+              'HTTP_REFERER' => "http://www.example.com/current-path"
             }
           )
           expect(response.status).to eq 302
-          expect(response.headers['Location']).to eq('http://example.com/current-path')
+          expect(response.headers['Location']).to eq('http://www.example.com/current-path')
+          expect(session[:inertia_errors]).to include({ go: 'back!' })
+        end
+      end
+    end
+  end
+
+  describe 'redirect_back_or_to' do
+    context 'with an [:inertia][:errors] option' do
+      context 'with a post request' do
+        it 'adds :inertia_errors to the session' do
+          skip("Requires Rails 7.0 or higher") if Rails.version < '7'
+
+          post(
+            redirect_back_or_to_with_inertia_errors_path,
+            headers: {
+              'X-Inertia' => true,
+              'HTTP_REFERER' => "http://www.example.com/current-path"
+            }
+          )
+          expect(response.status).to eq 302
+          expect(response.headers['Location']).to eq('http://www.example.com/current-path')
           expect(session[:inertia_errors]).to include({ go: 'back!' })
         end
       end