discourse
/
docker_manager
mirror of https://github.com/discourse/docker_manager.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: Redirect to login for anon user. 

https://meta.discourse.org/t/link-to-upgrade-shows-500-error/70993/7
This commit is contained in:
Guo Xiang Tan 2017-10-02 18:44:11 +08:00
parent a6539a6cf5
commit f117e28158
3 changed files with 55 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/docker_manager/admin_controller.rb
View File

@ -20,7 +20,7 @@ module DockerManager
if (version < expected_version) || (ruby_version < expected_ruby_version) if (version < expected_version) || (ruby_version < expected_ruby_version)
render text: <<HTML render text: <<~HTML
<html><head></head><body> <html><head></head><body>
<h2>You are running an old version of the Discourse image.</h2> <h2>You are running an old version of the Discourse image.</h2>
<p> <p>

4
app/controllers/docker_manager/application_controller.rb
View File

@ -1,6 +1,5 @@
module DockerManager module DockerManager
class ApplicationController < ActionController::Base class ApplicationController < ActionController::Base
helper DockerManager::ApplicationHelper helper DockerManager::ApplicationHelper
include CurrentUser include CurrentUser
@ -17,7 +16,8 @@ module DockerManager
protected protected
def ensure_admin def ensure_admin
raise Discourse::InvalidAccess.new unless current_user && current_user.admin? return redirect_to '/login' if !current_user
return render(plain: I18n.t('invalid_access'), status: 404) if !current_user.admin?
end end
end end

33
spec/requests/admin_controller_spec.rb Normal file
View File

@ -0,0 +1,33 @@
require 'rails_helper'
RSpec.describe DockerManager::AdminController do
describe 'anonymous user' do
it 'should redirect to login page' do
get '/admin/upgrade'
expect(response.status).to eq(302)
expect(response).to redirect_to('/login')
end
end
describe 'when user is not an admin' do
it 'should redirect to login page' do
sign_in(Fabricate(:user))
get '/admin/upgrade'
expect(response.status).to eq(404)
expect(response.body).to eq(I18n.t('invalid_access'))
end
end
describe 'when user is an admin' do
it 'should return the right response' do
sign_in(Fabricate(:admin))
get '/admin/upgrade'
expect(response.status).to eq(200)
end
end
end