Blame SOURCES/0140-dbus-keep-the-polkit-authorization-for-all-clients.patch

06486d
From 9c807b4f4d808201118f15182e12a9e68e12dc36 Mon Sep 17 00:00:00 2001
06486d
From: Jakub Filak <jfilak@redhat.com>
06486d
Date: Thu, 2 Jul 2015 12:19:30 +0200
06486d
Subject: [PATCH] dbus: keep the polkit authorization for all clients
06486d
06486d
I played with the defaults and I've found out that:
06486d
 - allow_any == clients like ssh
06486d
 - allow_active == for local console (X, terminal)
06486d
 - allow_inactive had no effect
06486d
06486d
Anyway, I do not find any reason to prevent any client from getting
06486d
access to all ABRT problems and I also think we should use
06486d
'auth_admin_keep' until we implement the new abrt dbus service.
06486d
06486d
auth_admin_keep - requires authorization for the first request and the
06486d
authorization is kept for a brief period (polkit(8)).
06486d
06486d
commit e98c8766655216db3d9a08b1fa52ba7decf57c46
06486d
Author: Stef Walter <stefw@redhat.com>
06486d
Date:   Mon May 5 08:41:00 2014 +0200
06486d
06486d
dbus: Fix desktop centric polkit policy file
06486d
06486d
In order to allow use of ABRT's DBus API on servers, the polkit
06486d
policy should allow admin usage even when not logged in an active
06486d
seat (ie: monitor and keyboard). Otherwise use from ssh logins and
06486d
Cockpit is prevented.
06486d
06486d
Related: #1224984
06486d
06486d
Signed-off-by: Jakub Filak <jfilak@redhat.com>
06486d
06486d
Conflicts:
06486d
	src/dbus/abrt_polkit.policy
06486d
---
06486d
 src/dbus/abrt_polkit.policy | 8 ++++----
06486d
 1 file changed, 4 insertions(+), 4 deletions(-)
06486d
06486d
diff --git a/src/dbus/abrt_polkit.policy b/src/dbus/abrt_polkit.policy
06486d
index 06008b4..beb615f 100644
06486d
--- a/src/dbus/abrt_polkit.policy
06486d
+++ b/src/dbus/abrt_polkit.policy
06486d
@@ -19,9 +19,9 @@ Copyright (c) 2012 ABRT Team <crash-catcher@fedorahosted.com>
06486d
     <description>Get problems from all users</description>
06486d
     <message>Reading others problems requires authentication</message>
06486d
     <defaults>
06486d
-      <allow_any>no</allow_any>
06486d
+      <allow_any>auth_admin_keep</allow_any>
06486d
       <allow_active>auth_admin_keep</allow_active>
06486d
-      <allow_inactive>no</allow_inactive>
06486d
+      <allow_inactive>auth_admin_keep</allow_inactive>
06486d
     </defaults>
06486d
   </action>
06486d
 
06486d
@@ -30,9 +30,9 @@ Copyright (c) 2012 ABRT Team <crash-catcher@fedorahosted.com>
06486d
     <description>Set value of configuration properties</description>
06486d
     <message>Update configuration values reuquires authentication</message>
06486d
     <defaults>
06486d
-      <allow_any>no</allow_any>
06486d
+      <allow_any>auth_admin_keep</allow_any>
06486d
       <allow_active>auth_admin_keep</allow_active>
06486d
-      <allow_inactive>no</allow_inactive>
06486d
+      <allow_inactive>auth_admin_keep</allow_inactive>
06486d
     </defaults>
06486d
   </action>
06486d
 
06486d
-- 
06486d
2.4.3
06486d