Redirect to canonical route in without trailing slash in Rails 3

Question

On Rails 3, I'm trying to redirect from a URL without a trailing slash to the canonical URL that has a slash.

match "/test", :to => redirect("/test/")

However, the route above matches both /test and /test/ causing a redirect loop.

How do I make it match only the version without the slash?

Blake Taylor · Answer 1 · 2014-05-08T21:45:31.097

6

You can force the redirect at the controller level.

# File: app/controllers/application_controller.rb
class ApplicationController < ActionController::Base

  protected

  def force_trailing_slash
    redirect_to request.original_url + '/' unless request.original_url.match(/\/$/)
  end
end

# File: app/controllers/test_controller.rb
class TestController < ApplicationController

  before_filter :force_trailing_slash, only: 'test'  # The magic

  # GET /test/
  def test
    # ...
  end
end

edited May 08 '14 at 21:45

answered Jan 22 '14 at 03:45

Blake Taylor

9,217
5
38
41

1

`original_url` also includes query parameters, so this check catches too much. – cburgmer Feb 10 '16 at 15:22

score 3 · Answer 2 · answered Sep 11 '12 at 06:36

I wanted to do the same to have a cannonical url for a blog, this works

  match 'post/:year/:title', :to => redirect {|env, params| "/post/#{params[:year]}/#{params[:title]}/" }, :constraints => lambda {|r| !r.original_fullpath.end_with?('/')}
  match 'post/:year/:title(/*file_path)' => 'posts#show', :as => :post, :format => false

then I have another rule which deals with the relative paths inside the post. Order is important, so former goes first and generic one goes second.

score 2 · Answer 3 · answered Dec 21 '11 at 13:25

There is an option in ActionDispatch called trailing_slash you can use to force the trailing slash at the end of the URL. I'm not sure if it can be used in the routing definition.

def tes_trailing_slsh
  add_host!
  options = {:controller => 'foo', :trailing_slash => true, :action => 'bar', :id => '33'}
  assert_equal('http://www.basecamphq.com/foo/bar/33/', W.new.url_for(options) )
end

In your case, the best way is to use Rack or your web server to execute the redirect. In Apache, you can add a definition such as

RewriteEngine on
RewriteRule ^(.+[^/])$ $1/  [R=301,L]

To redirect all routes without a trailing slash to the corresponding one with trailing slash.

Or you can use rack-rewrite to perform the same task in your Rails app at Rack level.

rack-rewrite is an interesting option. Though if at all possible I'd prefer a solution in Rails without using additional middleware and without doing it on the webserver side. — Shai Coleman, Dec 21 '11 at 16:41
Actually, when you call `redirect("/test/")` you're using a Rack middleware. ;) — Simone Carletti, Dec 21 '11 at 17:45

score 1 · Answer 4 · answered Feb 25 '21 at 16:34

Bullet-proof solution:

  before_action :force_trailing_slash

  ...

  private

  def force_trailing_slash
    return if trailing_slash?

    url = url_for \
      request.path_parameters
        .merge(request.query_parameters)
        .merge(trailing_slash: true)

    redirect_to url, status: :moved_permanently
  end

  def trailing_slash?
    URI(request.original_url).path.ends_with? '/'
  end

score 0 · Answer 5 · answered Dec 21 '11 at 13:34

0

Maybe it works with

match "/test$", :to => redirect("/test/")

answered Dec 21 '11 at 13:34

BvuRVKyUVlViVIc7

11,641
9
59
111

Redirect to canonical route in without trailing slash in Rails 3

5 Answers5

Linked