[SSSD] Re: [PATCHES] User lookup and login by Kerberos alias and email

2016-07-27 Thread Jakub Hrozek
On Tue, Jul 26, 2016 at 04:57:19PM +0200, Sumit Bose wrote: > now with patches ... > > On Tue, Jul 26, 2016 at 04:55:37PM +0200, Sumit Bose wrote: > > On Tue, Jul 26, 2016 at 11:12:34AM +0200, Jakub Hrozek wrote: > > > On Fri, Jul 22, 2016 at 09:44:33PM +0200, Sumit Bose wrote: > > > > Hi, > > >

[SSSD] Re: [PATCHES] views: properly override group member names

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 11:36:09AM +0200, Jakub Hrozek wrote: > On Wed, Jul 27, 2016 at 11:11:48AM +0200, Jakub Hrozek wrote: > > On Tue, Jul 26, 2016 at 10:05:21PM +0200, Sumit Bose wrote: > > > On Tue, Jul 26, 2016 at 06:06:48PM +0200, Jakub Hrozek wrote: > > > > On Tue, Jul 26, 2016 at

[SSSD] Re: [PATCHES] AD: netlogon_get_domain_info() allow missing arguments

2016-07-27 Thread Lukas Slebodnik
On (27/07/16 15:57), Jakub Hrozek wrote: >On Mon, Jul 25, 2016 at 04:11:16PM +0200, Lukas Slebodnik wrote: >> On (25/07/16 13:10), Jakub Hrozek wrote: >> >On Wed, Jul 20, 2016 at 03:14:17PM +0200, Sumit Bose wrote: >> >> Hi, >> >> >> >> it is possible that the CLAP/netlogon reply does not contain

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread thierry bordaz
On 07/27/2016 03:36 PM, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 02:55:37PM +0200, thierry bordaz wrote: On 07/27/2016 01:56 PM, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 01:03:59PM +0200, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 12:22:46PM +0200, Lukas Slebodnik wrote: On

[SSSD] Re: [PATCHES] AD: netlogon_get_domain_info() allow missing arguments

2016-07-27 Thread Jakub Hrozek
On Mon, Jul 25, 2016 at 04:11:16PM +0200, Lukas Slebodnik wrote: > On (25/07/16 13:10), Jakub Hrozek wrote: > >On Wed, Jul 20, 2016 at 03:14:17PM +0200, Sumit Bose wrote: > >> Hi, > >> > >> it is possible that the CLAP/netlogon reply does not contain any site > >> data. In this case we should not

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread thierry bordaz
On 07/27/2016 01:56 PM, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 01:03:59PM +0200, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 12:22:46PM +0200, Lukas Slebodnik wrote: On (27/07/16 12:08), Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 12:02:24PM +0200, Jakub Hrozek wrote: On Wed, Jul 27,

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 02:55:37PM +0200, thierry bordaz wrote: > > > On 07/27/2016 01:56 PM, Jakub Hrozek wrote: > > On Wed, Jul 27, 2016 at 01:03:59PM +0200, Jakub Hrozek wrote: > > > On Wed, Jul 27, 2016 at 12:22:46PM +0200, Lukas Slebodnik wrote: > > > > On (27/07/16 12:08), Jakub Hrozek

[SSSD] Re: [PATCH] sssctl: Consistent commands naming

2016-07-27 Thread Michal Židek
On 07/27/2016 11:09 AM, Jakub Hrozek wrote: On Wed, Jul 27, 2016 at 11:03:34AM +0200, Pavel Březina wrote: On 07/26/2016 04:19 PM, Michal Židek wrote: On 07/26/2016 01:19 PM, Pavel Březina wrote: On 07/25/2016 02:12 PM, Michal Židek wrote: Hi, this patches makes the sssctl commands more

[SSSD] Re: [PATCH] Change debug level of config error msgs

2016-07-27 Thread Petr Cech
On 07/27/2016 03:05 PM, Petr Cech wrote: On 07/27/2016 02:32 PM, Michal Židek wrote: Hi, I believe that this patch makes pinpointing of config errors a little easier. Especially when using sssctl tool that currently refuses to start a command when there are syntax errors in sssd.conf, but by

[SSSD] Re: [PATCH] Change debug level of config error msgs

2016-07-27 Thread Petr Cech
On 07/27/2016 02:32 PM, Michal Židek wrote: Hi, I believe that this patch makes pinpointing of config errors a little easier. Especially when using sssctl tool that currently refuses to start a command when there are syntax errors in sssd.conf, but by default it does not print problematic line

[SSSD] [PATCH] Change debug level of config error msgs

2016-07-27 Thread Michal Židek
Hi, I believe that this patch makes pinpointing of config errors a little easier. Especially when using sssctl tool that currently refuses to start a command when there are syntax errors in sssd.conf, but by default it does not print problematic line number. Compare: ldb: unable to dlopen

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 01:03:59PM +0200, Jakub Hrozek wrote: > On Wed, Jul 27, 2016 at 12:22:46PM +0200, Lukas Slebodnik wrote: > > On (27/07/16 12:08), Jakub Hrozek wrote: > > >On Wed, Jul 27, 2016 at 12:02:24PM +0200, Jakub Hrozek wrote: > > >> On Wed, Jul 27, 2016 at 11:54:16AM +0200, Lukas

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 12:22:46PM +0200, Lukas Slebodnik wrote: > On (27/07/16 12:08), Jakub Hrozek wrote: > >On Wed, Jul 27, 2016 at 12:02:24PM +0200, Jakub Hrozek wrote: > >> On Wed, Jul 27, 2016 at 11:54:16AM +0200, Lukas Slebodnik wrote: > >> > ehlo, > >> > > >> > attached patch fixes acces

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Lukas Slebodnik
On (27/07/16 12:08), Jakub Hrozek wrote: >On Wed, Jul 27, 2016 at 12:02:24PM +0200, Jakub Hrozek wrote: >> On Wed, Jul 27, 2016 at 11:54:16AM +0200, Lukas Slebodnik wrote: >> > ehlo, >> > >> > attached patch fixes acces denied after activating user in 389ds. >> > Jakub had some comments/ideas in

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 12:02:24PM +0200, Jakub Hrozek wrote: > On Wed, Jul 27, 2016 at 11:54:16AM +0200, Lukas Slebodnik wrote: > > ehlo, > > > > attached patch fixes acces denied after activating user in 389ds. > > Jakub had some comments/ideas in ticket but I think it's better to discuss > >

[SSSD] Re: [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 11:54:16AM +0200, Lukas Slebodnik wrote: > ehlo, > > attached patch fixes acces denied after activating user in 389ds. > Jakub had some comments/ideas in ticket but I think it's better to discuss > about virtual attributes and timestamp cache on mailing list. Yes, so the

[SSSD] [PATCH] SYSDB: Check changed virtual attributes before modified timestamp

2016-07-27 Thread Lukas Slebodnik
ehlo, attached patch fixes acces denied after activating user in 389ds. Jakub had some comments/ideas in ticket but I think it's better to discuss about virtual attributes and timestamp cache on mailing list. LS >From f1c5ded4457717cd516b676dfe7a6d315a7ce70b Mon Sep 17 00:00:00 2001 From: Lukas

[SSSD] Re: [PATCHES] views: properly override group member names

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 11:11:48AM +0200, Jakub Hrozek wrote: > On Tue, Jul 26, 2016 at 10:05:21PM +0200, Sumit Bose wrote: > > On Tue, Jul 26, 2016 at 06:06:48PM +0200, Jakub Hrozek wrote: > > > On Tue, Jul 26, 2016 at 05:25:11PM +0200, Jakub Hrozek wrote: > > > > On Tue, Jul 26, 2016 at

[SSSD] Re: [PATCH] PROVIDER: Conversion empty string from D-Bus to NULL

2016-07-27 Thread Petr Cech
On 07/27/2016 10:57 AM, Pavel Březina wrote: On 07/27/2016 10:26 AM, Petr Cech wrote: Hi list, there is patch for [1] attached. [1] https://fedorahosted.org/sssd/ticket/3084 If you prefer two commits--one for reverting and second with the patch--don't hesitate to tell me it. :-) Regards --

[SSSD] Re: [PATCHES] views: properly override group member names

2016-07-27 Thread Jakub Hrozek
On Tue, Jul 26, 2016 at 10:05:21PM +0200, Sumit Bose wrote: > On Tue, Jul 26, 2016 at 06:06:48PM +0200, Jakub Hrozek wrote: > > On Tue, Jul 26, 2016 at 05:25:11PM +0200, Jakub Hrozek wrote: > > > On Tue, Jul 26, 2016 at 01:51:56PM +0200, Sumit Bose wrote: > > > > > > The third patch adds a sysdb

[SSSD] Re: [PATCH] sssctl: Consistent commands naming

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 11:03:34AM +0200, Pavel Březina wrote: > On 07/26/2016 04:19 PM, Michal Židek wrote: > > On 07/26/2016 01:19 PM, Pavel Březina wrote: > > > On 07/25/2016 02:12 PM, Michal Židek wrote: > > > > Hi, > > > > > > > > this patches makes the sssctl commands more similar to > > >

[SSSD] Re: [PATCH] sssctl: Consistent commands naming

2016-07-27 Thread Pavel Březina
On 07/26/2016 04:19 PM, Michal Židek wrote: On 07/26/2016 01:19 PM, Pavel Březina wrote: On 07/25/2016 02:12 PM, Michal Židek wrote: Hi, this patches makes the sssctl commands more similar to ipa tool commands. I also think this pattern makes it easier to remember the commands. Note that in

[SSSD] Re: [PATCH] PROVIDER: Conversion empty string from D-Bus to NULL

2016-07-27 Thread Pavel Březina
On 07/27/2016 10:26 AM, Petr Cech wrote: Hi list, there is patch for [1] attached. [1] https://fedorahosted.org/sssd/ticket/3084 If you prefer two commits--one for reverting and second with the patch--don't hesitate to tell me it. :-) Regards -- Petr^4 Čech

[SSSD] Re: [PATCH] PROVIDER: Conversion empty string from D-Bus to NULL

2016-07-27 Thread Jakub Hrozek
On Wed, Jul 27, 2016 at 10:26:30AM +0200, Petr Cech wrote: > Hi list, > > there is patch for [1] attached. > > [1] https://fedorahosted.org/sssd/ticket/3084 > > If you prefer two commits--one for reverting and second with the > patch--don't hesitate to tell me it. :-) Pavel is best suited to