ref: refs/heads/master commit 25165997f0f039846a8bb4c250aab840f049ca4d Author: Stefan Schubert <schubi@suse.de> Date: Tue Aug 11 14:43:34 2009 +0200 improved logging --- .../app/controllers/permissions_controller.rb | 17 +++++++++++------ 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/webservice/app/controllers/permissions_controller.rb b/webservice/app/controllers/permissions_controller.rb index bd40e83..63389e8 100644 --- a/webservice/app/controllers/permissions_controller.rb +++ b/webservice/app/controllers/permissions_controller.rb @@ -25,13 +25,18 @@ class PermissionsController < ApplicationController suse_string = /org.opensuse.yast..*/ # get users or all actions - ret = if user_id - Scr.instance.execute(["polkit-auth", "--user", user_id, "--explicit"]) - else - Scr.instance.execute(["polkit-action"]) - end rescue nil - raise RuntimeError unless ret && ret[:exit] == 0 + begin + ret = if user_id + Scr.instance.execute(["polkit-auth", "--user", user_id, "--explicit"]) + else + Scr.instance.execute(["polkit-action"]) + end + rescue Exception => e + ret = nil + logger.error "SCR.execute has raised the exception: #{e.inspect}" + end + raise RuntimeError unless ret && ret[:exit] == 0 ret[:stdout].scan(suse_string) do |p| next unless filter.blank? or p.include?(filter) yield p -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org