From 08b96f1b9f419ab250f24f3828db28a247ef11ac Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Sun, 19 Mar 2017 20:03:28 +0100 Subject: [PATCH] Fix wrong HTTP status codes on error pages --- app/controllers/accounts_controller.rb | 2 +- app/controllers/application_controller.rb | 6 +++--- app/controllers/stream_entries_controller.rb | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/app/controllers/accounts_controller.rb b/app/controllers/accounts_controller.rb index 00f8047fde..dc1aeb5ea5 100644 --- a/app/controllers/accounts_controller.rb +++ b/app/controllers/accounts_controller.rb @@ -56,6 +56,6 @@ class AccountsController < ApplicationController end def check_account_suspension - head 410 if @account.suspended? + gone if @account.suspended? end end diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index e4b6d0fafe..ef9364897c 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -51,21 +51,21 @@ class ApplicationController < ActionController::Base def not_found respond_to do |format| format.any { head 404 } - format.html { render 'errors/404', layout: 'error' } + format.html { render 'errors/404', layout: 'error', status: 404 } end end def gone respond_to do |format| format.any { head 410 } - format.html { render 'errors/410', layout: 'error' } + format.html { render 'errors/410', layout: 'error', status: 410 } end end def unprocessable_entity respond_to do |format| format.any { head 422 } - format.html { render 'errors/422', layout: 'error' } + format.html { render 'errors/422', layout: 'error', status: 422 } end end diff --git a/app/controllers/stream_entries_controller.rb b/app/controllers/stream_entries_controller.rb index c43d372ed3..de38b36023 100644 --- a/app/controllers/stream_entries_controller.rb +++ b/app/controllers/stream_entries_controller.rb @@ -50,6 +50,6 @@ class StreamEntriesController < ApplicationController end def check_account_suspension - head 410 if @account.suspended? + gone if @account.suspended? end end