Как перенаправить на 404 в Rails? - PullRequest
468 голосов
/ 05 марта 2010

Я бы хотел «подделать» страницу 404 в Rails. В PHP я бы просто отправил заголовок с кодом ошибки как таковой:

header("HTTP/1.0 404 Not Found");

Как это сделать с Rails?

Ответы [ 10 ]

1033 голосов
/ 13 февраля 2011

Не визуализируйте 404 самостоятельно, нет причин для этого; В Rails эта функциональность уже встроена. Если вы хотите показать страницу 404, создайте метод render_404 (или not_found, как я его назвал) в ApplicationController следующим образом:

def not_found
  raise ActionController::RoutingError.new('Not Found')
end

Rails также обрабатывает AbstractController::ActionNotFound и ActiveRecord::RecordNotFound таким же образом.

Это делает две вещи лучше:

1) Он использует встроенный в Rails обработчик rescue_from для рендеринга страницы 404, и 2) он прерывает выполнение вашего кода, позволяя вам делать такие приятные вещи, как:

  user = User.find_by_email(params[:email]) or not_found
  user.do_something!

без необходимости писать уродливые условные выражения.

В качестве бонуса, это также очень легко обрабатывать в тестах. Например, в интеграционном тесте rspec:

# RSpec 1

lambda {
  visit '/something/you/want/to/404'
}.should raise_error(ActionController::RoutingError)

# RSpec 2+

expect {
  get '/something/you/want/to/404'
}.to raise_error(ActionController::RoutingError)

и минитест:

assert_raises(ActionController::RoutingError) do 
  get '/something/you/want/to/404'
end

ИЛИ см. Дополнительную информацию из Рельсы рендеринга 404 не найдены в действии контроллера

238 голосов
/ 05 марта 2010

HTTP 404 Статус

Чтобы вернуть заголовок 404, просто используйте параметр :status для метода рендеринга.

def action
  # here the code

  render :status => 404
end

Если вы хотите отобразить стандартную страницу 404, вы можете извлечь эту функцию в методе.

def render_404
  respond_to do |format|
    format.html { render :file => "#{Rails.root}/public/404", :layout => false, :status => :not_found }
    format.xml  { head :not_found }
    format.any  { head :not_found }
  end
end

и назовите это в своем действии

def action
  # here the code

  render_404
end

Если вы хотите, чтобы действие отображало страницу с ошибкой и останавливалось, просто используйте оператор return.

def action
  render_404 and return if params[:something].blank?

  # here the code that will never be executed
end

ActiveRecord и HTTP 404

Также помните, что Rails спасает некоторые ошибки ActiveRecord, такие как ActiveRecord::RecordNotFound, отображающий страницу с ошибкой 404.

Это означает, что вам не нужно спасать это действие самостоятельно

def show
  user = User.find(params[:id])
end

User.find повышает ActiveRecord::RecordNotFound, когда пользователя не существует. Это очень мощная функция. Посмотрите на следующий код

def show
  user = User.find_by_email(params[:email]) or raise("not found")
  # ...
end

Вы можете упростить это, делегировав Rails проверку. Просто используйте версию взрыва.

def show
  user = User.find_by_email!(params[:email])
  # ...
end
58 голосов
/ 18 августа 2011

Недавно выбранный ответ, представленный Стивеном Сорокой, близок, но не завершен. Сам тест скрывает тот факт, что это не возвращает истинный 404 - он возвращает статус 200 - «успех». Первоначальный ответ был ближе, но попытался отобразить макет так, как если бы не произошло никакого сбоя. Это исправляет все:

render :text => 'Not Found', :status => '404'

Вот типичный мой тестовый набор для чего-то, что я ожидаю вернуть 404, используя сопоставители RSpec и Shoulda:

describe "user view" do
  before do
    get :show, :id => 'nonsense'
  end

  it { should_not assign_to :user }

  it { should respond_with :not_found }
  it { should respond_with_content_type :html }

  it { should_not render_template :show }
  it { should_not render_with_layout }

  it { should_not set_the_flash }
end

Эта здоровая паранойя позволила мне обнаружить несоответствие типов контента, когда все остальное выглядело замечательно :) Я проверяю все эти элементы: назначенные переменные, код ответа, тип контента ответа, шаблон отображается, макет отображается, флеш-сообщения. *

Я пропущу проверку типа контента в приложениях, которые строго html ... иногда. Ведь «скептик проверяет ВСЕ ящики»:)

http://dilbert.com/strips/comic/1998-01-20/

К вашему сведению: я не рекомендую проверять то, что происходит в контроллере, то есть «should_raise». Что вас волнует, так это результат. Мои тесты, приведенные выше, позволили мне попробовать различные решения, и тесты остаются прежними, независимо от того, вызывает ли решение исключение, специальный рендеринг и т. Д.

18 голосов
/ 19 января 2015

Вы также можете использовать файл рендеринга:

render file: "#{Rails.root}/public/404.html", layout: false, status: 404

Где вы можете использовать макет или нет.

Другой вариант - использовать исключения для управления им:

raise ActiveRecord::RecordNotFound, "Record not found."
12 голосов
/ 12 марта 2014

Выбранный ответ не работает в Rails 3.1+, поскольку обработчик ошибок был перемещен в промежуточное ПО (см. github issue ).

Вот решение, которое я нашел, которым я очень доволен.

В ApplicationController:

  unless Rails.application.config.consider_all_requests_local
    rescue_from Exception, with: :handle_exception
  end

  def not_found
    raise ActionController::RoutingError.new('Not Found')
  end

  def handle_exception(exception=nil)
    if exception
      logger = Logger.new(STDOUT)
      logger.debug "Exception Message: #{exception.message} \n"
      logger.debug "Exception Class: #{exception.class} \n"
      logger.debug "Exception Backtrace: \n"
      logger.debug exception.backtrace.join("\n")
      if [ActionController::RoutingError, ActionController::UnknownController, ActionController::UnknownAction].include?(exception.class)
        return render_404
      else
        return render_500
      end
    end
  end

  def render_404
    respond_to do |format|
      format.html { render template: 'errors/not_found', layout: 'layouts/application', status: 404 }
      format.all { render nothing: true, status: 404 }
    end
  end

  def render_500
    respond_to do |format|
      format.html { render template: 'errors/internal_server_error', layout: 'layouts/application', status: 500 }
      format.all { render nothing: true, status: 500}
    end
  end

и application.rb:

config.after_initialize do |app|
  app.routes.append{ match '*a', :to => 'application#not_found' } unless config.consider_all_requests_local
end

А в моих ресурсах (показывать, редактировать, обновлять, удалять):

@resource = Resource.find(params[:id]) or not_found

Это, безусловно, можно улучшить, но, по крайней мере, у меня есть разные представления для not_found и internal_error без переопределения основных функций Rails.

7 голосов
/ 01 апреля 2014

это поможет вам ...

Контроллер приложений

class ApplicationController < ActionController::Base
  protect_from_forgery
  unless Rails.application.config.consider_all_requests_local             
    rescue_from ActionController::RoutingError, ActionController::UnknownController, ::AbstractController::ActionNotFound, ActiveRecord::RecordNotFound, with: lambda { |exception| render_error 404, exception }
  end

  private
    def render_error(status, exception)
      Rails.logger.error status.to_s + " " + exception.message.to_s
      Rails.logger.error exception.backtrace.join("\n") 
      respond_to do |format|
        format.html { render template: "errors/error_#{status}",status: status }
        format.all { render nothing: true, status: status }
      end
    end
end

Ошибка контроллера

class ErrorsController < ApplicationController
  def error_404
    @not_found_path = params[:not_found]
  end
end

Вид / ошибки / error_404.html.haml

.site
  .services-page 
    .error-template
      %h1
        Oops!
      %h2
        404 Not Found
      .error-details
        Sorry, an error has occured, Requested page not found!
        You tried to access '#{@not_found_path}', which is not a valid page.
      .error-actions
        %a.button_simple_orange.btn.btn-primary.btn-lg{href: root_path}
          %span.glyphicon.glyphicon-home
          Take Me Home
1 голос
/ 04 ноября 2018

Я хотел выдать «обычный» 404 для любого вошедшего в систему пользователя, который не является администратором, поэтому я закончил тем, что написал что-то подобное в Rails 5:

class AdminController < ApplicationController
  before_action :blackhole_admin

  private

  def blackhole_admin
    return if current_user.admin?

    raise ActionController::RoutingError, 'Not Found'
  rescue ActionController::RoutingError
    render file: "#{Rails.root}/public/404", layout: false, status: :not_found
  end
end
1 голос
/ 01 ноября 2015
<%= render file: 'public/404', status: 404, formats: [:html] %>

просто добавьте это на страницу, которую вы хотите отобразить на странице ошибки 404, и все готово.

0 голосов
/ 25 июля 2015

Если вы хотите обрабатывать разные 404 по-разному, подумайте о том, чтобы перехватить их в своих контроллерах. Это позволит вам выполнять такие действия, как отслеживание числа 404, сгенерированного различными группами пользователей, поддержка взаимодействия с пользователями, чтобы выяснить, что пошло не так / какая часть пользовательского опыта может потребоваться настроить, провести A / B-тестирование и т. Д.

Я поместил здесь базовую логику в ApplicationController, но она также может быть помещена в более конкретные контроллеры, чтобы иметь специальную логику только для одного контроллера.

Причина, по которой я использую if с ENV ['RESCUE_404'], заключается в том, что я могу проверить поднятие AR :: RecordNotFound изолированно. В тестах я могу установить для этого ENV var значение false, и мой rescue_from не сработает. Таким образом, я могу проверить поднятие отдельно от условной логики 404.

class ApplicationController < ActionController::Base

  rescue_from ActiveRecord::RecordNotFound, with: :conditional_404_redirect if ENV['RESCUE_404']

private

  def conditional_404_redirect
    track_404(@current_user)
    if @current_user.present?
      redirect_to_user_home          
    else
      redirect_to_front
    end
  end

end
0 голосов
/ 15 апреля 2014

Чтобы проверить обработку ошибок, вы можете сделать что-то вроде этого:

feature ErrorHandling do
  before do
    Rails.application.config.consider_all_requests_local = false
    Rails.application.config.action_dispatch.show_exceptions = true
  end

  scenario 'renders not_found template' do
    visit '/blah'
    expect(page).to have_content "The page you were looking for doesn't exist."
  end
end
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...