Fix redirecting non-functional accounts on public pages (#11978)
Fix #11969
This commit is contained in:
		
							parent
							
								
									3ec80c7aec
								
							
						
					
					
						commit
						de5305a3a5
					
				
					 12 changed files with 16 additions and 0 deletions
				
			
		| 
						 | 
					@ -10,6 +10,7 @@ class AccountsController < ApplicationController
 | 
				
			||||||
  before_action :set_body_classes
 | 
					  before_action :set_body_classes
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
					  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def show
 | 
					  def show
 | 
				
			||||||
    respond_to do |format|
 | 
					    respond_to do |format|
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -2,6 +2,7 @@
 | 
				
			||||||
 | 
					
 | 
				
			||||||
class CustomCssController < ApplicationController
 | 
					class CustomCssController < ApplicationController
 | 
				
			||||||
  skip_before_action :store_current_location
 | 
					  skip_before_action :store_current_location
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  before_action :set_cache_headers
 | 
					  before_action :set_cache_headers
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -9,6 +9,8 @@ class DirectoriesController < ApplicationController
 | 
				
			||||||
  before_action :set_tag, only: :show
 | 
					  before_action :set_tag, only: :show
 | 
				
			||||||
  before_action :set_accounts
 | 
					  before_action :set_accounts
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def index
 | 
					  def index
 | 
				
			||||||
    render :index
 | 
					    render :index
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -8,6 +8,7 @@ class FollowerAccountsController < ApplicationController
 | 
				
			||||||
  before_action :set_cache_headers
 | 
					  before_action :set_cache_headers
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
					  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def index
 | 
					  def index
 | 
				
			||||||
    respond_to do |format|
 | 
					    respond_to do |format|
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -8,6 +8,7 @@ class FollowingAccountsController < ApplicationController
 | 
				
			||||||
  before_action :set_cache_headers
 | 
					  before_action :set_cache_headers
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
					  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def index
 | 
					  def index
 | 
				
			||||||
    respond_to do |format|
 | 
					    respond_to do |format|
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -2,6 +2,7 @@
 | 
				
			||||||
 | 
					
 | 
				
			||||||
class ManifestsController < ApplicationController
 | 
					class ManifestsController < ApplicationController
 | 
				
			||||||
  skip_before_action :store_current_location
 | 
					  skip_before_action :store_current_location
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def show
 | 
					  def show
 | 
				
			||||||
    expires_in 3.minutes, public: true
 | 
					    expires_in 3.minutes, public: true
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -4,6 +4,7 @@ class MediaController < ApplicationController
 | 
				
			||||||
  include Authorization
 | 
					  include Authorization
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_before_action :store_current_location
 | 
					  skip_before_action :store_current_location
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  before_action :authenticate_user!, if: :whitelist_mode?
 | 
					  before_action :authenticate_user!, if: :whitelist_mode?
 | 
				
			||||||
  before_action :set_media_attachment
 | 
					  before_action :set_media_attachment
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -4,6 +4,7 @@ class MediaProxyController < ApplicationController
 | 
				
			||||||
  include RoutingHelper
 | 
					  include RoutingHelper
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_before_action :store_current_location
 | 
					  skip_before_action :store_current_location
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  before_action :authenticate_user!, if: :whitelist_mode?
 | 
					  before_action :authenticate_user!, if: :whitelist_mode?
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -7,6 +7,8 @@ class RemoteFollowController < ApplicationController
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  before_action :set_body_classes
 | 
					  before_action :set_body_classes
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def new
 | 
					  def new
 | 
				
			||||||
    @remote_follow = RemoteFollow.new(session_params)
 | 
					    @remote_follow = RemoteFollow.new(session_params)
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -10,6 +10,8 @@ class RemoteInteractionController < ApplicationController
 | 
				
			||||||
  before_action :set_status
 | 
					  before_action :set_status
 | 
				
			||||||
  before_action :set_body_classes
 | 
					  before_action :set_body_classes
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def new
 | 
					  def new
 | 
				
			||||||
    @remote_follow = RemoteFollow.new(session_params)
 | 
					    @remote_follow = RemoteFollow.new(session_params)
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -19,6 +19,7 @@ class StatusesController < ApplicationController
 | 
				
			||||||
  before_action :set_autoplay, only: :embed
 | 
					  before_action :set_autoplay, only: :embed
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
					  skip_around_action :set_locale, if: -> { request.format == :json }
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!, only: [:show, :embed]
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  content_security_policy only: :embed do |p|
 | 
					  content_security_policy only: :embed do |p|
 | 
				
			||||||
    p.frame_ancestors(false)
 | 
					    p.frame_ancestors(false)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -13,6 +13,8 @@ class TagsController < ApplicationController
 | 
				
			||||||
  before_action :set_body_classes
 | 
					  before_action :set_body_classes
 | 
				
			||||||
  before_action :set_instance_presenter
 | 
					  before_action :set_instance_presenter
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					  skip_before_action :require_functional!
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def show
 | 
					  def show
 | 
				
			||||||
    respond_to do |format|
 | 
					    respond_to do |format|
 | 
				
			||||||
      format.html do
 | 
					      format.html do
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		
		Reference in a new issue