[SSSD] Re: [PATCH] LDAP: Do not print "null" in the DEBUG message

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 10:06:38AM +0100, Pavel Březina wrote: > I'm fine either way. OK, see the attached patch. >From ea49df6b0816845ec667392a170f4176e759731f Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Wed, 9 Mar 2016 18:44:43 +0100 Subject: [PATCH] LDAP: Remove

[SSSD] Re: [PATCH] make async connect generic

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 06:28:50PM +0100, Jakub Hrozek wrote: > On Tue, Mar 08, 2016 at 12:28:55PM -0500, Simo Sorce wrote: > > On Tue, 2016-03-08 at 12:11 -0500, Simo Sorce wrote: > > > On Tue, 2016-03-08 at 17:48 +0100, Jakub Hrozek wrote: > > > > On Tue, Mar 08, 2016 at 10:18:46AM -0500, Simo

[SSSD] Re: [PATCH] memberof: Don't allocate on a NULL context

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 05:37:12PM +0100, Lukas Slebodnik wrote: > On (09/03/16 15:21), Jakub Hrozek wrote: > >On Wed, Mar 09, 2016 at 11:32:39AM +0100, Pavel Březina wrote: > >> On 02/26/2016 02:03 PM, Jakub Hrozek wrote: > >> >On Fri, Feb 26, 2016 at 11:08:45AM +0100, Pavel Březina wrote: > >>

[SSSD] Re: [PATCH] make async connect generic

2016-03-09 Thread Jakub Hrozek
On Tue, Mar 08, 2016 at 12:28:55PM -0500, Simo Sorce wrote: > On Tue, 2016-03-08 at 12:11 -0500, Simo Sorce wrote: > > On Tue, 2016-03-08 at 17:48 +0100, Jakub Hrozek wrote: > > > On Tue, Mar 08, 2016 at 10:18:46AM -0500, Simo Sorce wrote: > > > > Fixing everything else commented before. > > > >

[SSSD] Re: [PATCH SET] TOOL: Invalidation of sudo rules at sss_cache

2016-03-09 Thread Petr Cech
On 03/08/2016 05:57 PM, Petr Cech wrote: On 03/08/2016 05:09 PM, Pavel Reichl wrote: Hello Petr, I just run through the code and I have some code style suggestions, feel free to disagree :-). I will address some suggestions tomorrow. I am doing tests for sysdb_sudo_rules now. Thanks for

[SSSD] [PATCH] UTIL: Allow to append new line in sss_vdebug_fn

2016-03-09 Thread Lukas Slebodnik
ehlo, I read log files from latest 1.13 today and it was a small challenge due to missing line feed after some ldb messages. LS >From c9f7e99fb933706663948efc659937299ec54077 Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Wed, 9 Mar 2016 17:14:16 +0100 Subject: [PATCH

[SSSD] Re: [PATCH] memberof: Don't allocate on a NULL context

2016-03-09 Thread Lukas Slebodnik
On (09/03/16 15:21), Jakub Hrozek wrote: >On Wed, Mar 09, 2016 at 11:32:39AM +0100, Pavel Březina wrote: >> On 02/26/2016 02:03 PM, Jakub Hrozek wrote: >> >On Fri, Feb 26, 2016 at 11:08:45AM +0100, Pavel Březina wrote: >> >>On 02/24/2016 03:19 PM, Jakub Hrozek wrote: >> >>>Hi, >> >>> >> >>>the

[SSSD] Re: [PATCH] IPA SUDO: download externalUser attribute

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 01:28:49PM +0100, Lukas Slebodnik wrote: > On (09/03/16 10:23), Jakub Hrozek wrote: > >On Tue, Mar 08, 2016 at 11:03:40PM +0100, Lukas Slebodnik wrote: > >> On (08/03/16 18:21), Jakub Hrozek wrote: > >> >On Fri, Mar 04, 2016 at 02:01:54PM +0100, Pavel Březina wrote: > >> >>

[SSSD] Re: [PATCH] memberof: Don't allocate on a NULL context

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 11:32:39AM +0100, Pavel Březina wrote: > On 02/26/2016 02:03 PM, Jakub Hrozek wrote: > >On Fri, Feb 26, 2016 at 11:08:45AM +0100, Pavel Březina wrote: > >>On 02/24/2016 03:19 PM, Jakub Hrozek wrote: > >>>Hi, > >>> > >>>the attached patch fixes: > >>>

[SSSD] Re: [PATCH] Move two options in sssd.conf to the correct section

2016-03-09 Thread Jakub Hrozek
On Wed, Mar 09, 2016 at 11:35:53AM +0100, Pavel Březina wrote: > On 02/23/2016 09:14 PM, Jakub Hrozek wrote: > >On Tue, Feb 23, 2016 at 03:00:56PM +0100, Jakub Hrozek wrote: > >>On Tue, Feb 23, 2016 at 01:29:42PM +0100, Pavel Březina wrote: > >>>On 02/23/2016 12:09 PM, Jakub Hrozek wrote: >

[SSSD] Re: [PATCH] cache_req improvements

2016-03-09 Thread Sumit Bose
On Tue, Mar 01, 2016 at 01:05:48PM +0100, Pavel Březina wrote: > On 02/26/2016 01:47 PM, Jakub Hrozek wrote: > >On Wed, Feb 24, 2016 at 12:41:24PM +0100, Pavel Březina wrote: > From f61d0192b8254247802167ea385b52f65d4e175d Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Pavel=20B=C5=99ezina?=

[SSSD] Re: [PATCH] IPA SUDO: download externalUser attribute

2016-03-09 Thread Lukas Slebodnik
On (09/03/16 10:23), Jakub Hrozek wrote: >On Tue, Mar 08, 2016 at 11:03:40PM +0100, Lukas Slebodnik wrote: >> On (08/03/16 18:21), Jakub Hrozek wrote: >> >On Fri, Mar 04, 2016 at 02:01:54PM +0100, Pavel Březina wrote: >> >> This allows configuration with id_provider = proxy >> >> and sudo_provider

[SSSD] Re: [PATCH] Move two options in sssd.conf to the correct section

2016-03-09 Thread Pavel Březina
On 02/23/2016 09:14 PM, Jakub Hrozek wrote: On Tue, Feb 23, 2016 at 03:00:56PM +0100, Jakub Hrozek wrote: On Tue, Feb 23, 2016 at 01:29:42PM +0100, Pavel Březina wrote: On 02/23/2016 12:09 PM, Jakub Hrozek wrote: Reported in https://bugzilla.redhat.com/show_bug.cgi?id=1310141 Ack.

[SSSD] Re: [PATCH] memberof: Don't allocate on a NULL context

2016-03-09 Thread Pavel Březina
On 02/26/2016 02:03 PM, Jakub Hrozek wrote: On Fri, Feb 26, 2016 at 11:08:45AM +0100, Pavel Březina wrote: On 02/24/2016 03:19 PM, Jakub Hrozek wrote: Hi, the attached patch fixes: https://fedorahosted.org/sssd/ticket/2959 It was confirmed by the original reporter. The bug was there

[SSSD] Re: [PATCH] IPA SUDO: download externalUser attribute

2016-03-09 Thread Jakub Hrozek
On Tue, Mar 08, 2016 at 11:03:40PM +0100, Lukas Slebodnik wrote: > On (08/03/16 18:21), Jakub Hrozek wrote: > >On Fri, Mar 04, 2016 at 02:01:54PM +0100, Pavel Březina wrote: > >> This allows configuration with id_provider = proxy > >> and sudo_provider = ipa when someone needs to fetch > >> rules

[SSSD] Re: [PATCH] LDAP: Do not print "null" in the DEBUG message

2016-03-09 Thread Pavel Březina
On 03/08/2016 10:57 PM, Lukas Slebodnik wrote: On (08/03/16 18:14), Jakub Hrozek wrote: On Tue, Mar 08, 2016 at 09:34:29AM +0100, Lukas Slebodnik wrote: On (25/02/16 11:06), Jakub Hrozek wrote: On Wed, Feb 24, 2016 at 06:05:11PM +0100, Lukas Slebodnik wrote: On (24/02/16 16:43), Jakub Hrozek