Drop monkey patch now we're using option style redirects

Option style redirects escape path parameters correctly, so we
can drop out monkey patch.

diff --git a/config/initializers/routing_redirect.rb b/config/initializers/routing_redirect.rb
deleted file mode 100644 (file)
index d064472..0000000
--- a/config/initializers/routing_redirect.rb
+++ /dev/null
@@ -1,16 +0,0 @@
-require "action_dispatch/routing/redirection"
-
-#
-# Fix escaping in routes to use path style escaping
-#
-# https://github.com/rails/rails/issues/13110
-#
-module ActionDispatch
-  module Routing
-    class PathRedirect < Redirect
-      def escape(params)
-        Hash[params.map{ |k,v| [k, URI.escape(v)] }]
-      end
-    end
-  end
-end

diff --git a/test/integration/redirect_test.rb b/test/integration/redirect_test.rb
index 5a9ba17ec4276a55b069e4183058df84fd920f6b..1c4af7d4fe81752888294b9218519c1227744d63 100644 (file)
--- a/test/integration/redirect_test.rb
+++ b/test/integration/redirect_test.rb
@@ -31,6 +31,10 @@ class RedirectTest  < ActionDispatch::IntegrationTest
     get "/user/name/edits"
     assert_response :redirect
     assert_redirected_to "/user/name/history"
     get "/user/name/edits"
     assert_response :redirect
     assert_redirected_to "/user/name/history"

+
+    get "/user/name%20with%20spaces/edits"
+    assert_response :redirect
+    assert_redirected_to "/user/name%20with%20spaces/history"

   end
 
   def test_history_feed_redirects
   end
 
   def test_history_feed_redirects


@@ -45,6 +49,10 @@ class RedirectTest  < ActionDispatch::IntegrationTest
     get "/user/name/edits/feed"
     assert_response :redirect
     assert_redirected_to "/user/name/history/feed"
     get "/user/name/edits/feed"
     assert_response :redirect
     assert_redirected_to "/user/name/history/feed"

+
+    get "/user/name%20with%20spaces/edits/feed"
+    assert_response :redirect
+    assert_redirected_to "/user/name%20with%20spaces/history/feed"

   end
 
   def test_browse_redirects
   end
 
   def test_browse_redirects