[PATCH conductor 11/13] require Privilege::View on config_server.provider_account to test connection

Mo Morsi mmorsi at redhat.com
Thu May 17 13:40:05 UTC 2012


---
 src/app/controllers/config_servers_controller.rb |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/src/app/controllers/config_servers_controller.rb b/src/app/controllers/config_servers_controller.rb
index 0fe1108..896c6af 100644
--- a/src/app/controllers/config_servers_controller.rb
+++ b/src/app/controllers/config_servers_controller.rb
@@ -37,12 +37,14 @@ class ConfigServersController < ApplicationController
 
   def test
     config_server = ConfigServer.find(params[:id])
+    provider_account = config_server.provider_account
+    require_privilege(Privilege::VIEW, provider_account)
+
     if not config_server.connection_valid?
       flash[:error] = config_server.connection_error_msg
     else
       flash[:notice] = t('config_servers.flash.notice.test_successful')
     end
-    provider_account = config_server.provider_account
     provider = provider_account.provider
     redirect_to provider_provider_account_path(provider, provider_account)
   end
-- 
1.7.6.5




More information about the aeolus-devel mailing list