Add support for redirect_back_or_to and respect raise_on_open_redirects config

Author: skryukov

lib/inertia_rails/controller.rb

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

spec/dummy/app/controllers/inertia_test_controller.rb

@@ -50,6 +50,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

@@ -24,6 +24,7 @@
   get 'redirect_with_inertia_errors' => 'inertia_test#redirect_with_inertia_errors'
   post 'redirect_with_inertia_errors' => 'inertia_test#redirect_with_inertia_errors'
   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

@@ -39,11 +39,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