Better swift redirects

librariesio · Jul 22, 2016 · 263d5e2 · 263d5e2
1 parent 2af8c3d
commit 263d5e2
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/config/routes.rb b/config/routes.rb
@@ -158,11 +158,17 @@
     get '/rails/mailers/*path'   => "rails/mailers#preview"
   end
 
+  # legacy swift rename redirects
   get '/swift/', to: redirect("/SwiftPM")
   get '/Swift/', to: redirect("/SwiftPM")
-
-  get '/swift/:name', to: redirect("/SwiftPM/%{name}"), constraints: { :name => /.*/ }
-  get '/Swift/:name', to: redirect("/SwiftPM/%{name}"), constraints: { :name => /.*/ }
+  get '/swift/:name', constraints: { :name => /.*/ }, to: redirect { |params, request|
+    path = "SwiftPM/#{Rack::Utils.escape(params[:name])}"
+    "http://#{request.host_with_port}/#{path}"
+  }
+  get '/swift/:name', constraints: { :name => /.*/ }, to: redirect { |params, request|
+    path = "SwiftPM/#{Rack::Utils.escape(params[:name])}"
+    "http://#{request.host_with_port}/#{path}"
+  }
 
   get '/:platform/:name/suggestions', to: 'project_suggestions#new', as: :project_suggestions, constraints: { :name => /.*/ }
   post '/:platform/:name/suggestions', to: 'project_suggestions#create', constraints: { :name => /.*/ }