Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-12 Thread Jakub Hrozek
On Fri, Oct 09, 2015 at 02:14:12PM +0200, Pavel Březina wrote: > On 10/09/2015 01:34 PM, Sumit Bose wrote: > >On Thu, Oct 08, 2015 at 08:51:41PM +0200, Jakub Hrozek wrote: > >>On Wed, Oct 07, 2015 at 04:28:02PM +0200, Sumit Bose wrote: > >>> From 3d5352977ca07f8a1a6d3bbaa57d11b54fd705d0 Mon Sep 17

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-09 Thread Sumit Bose
On Thu, Oct 08, 2015 at 08:51:41PM +0200, Jakub Hrozek wrote: > On Wed, Oct 07, 2015 at 04:28:02PM +0200, Sumit Bose wrote: > > From 3d5352977ca07f8a1a6d3bbaa57d11b54fd705d0 Mon Sep 17 00:00:00 2001 > > From: Sumit Bose > > Date: Wed, 7 Oct 2015 15:22:34 +0200 > > Subject:

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-09 Thread Pavel Březina
On 10/09/2015 01:34 PM, Sumit Bose wrote: On Thu, Oct 08, 2015 at 08:51:41PM +0200, Jakub Hrozek wrote: On Wed, Oct 07, 2015 at 04:28:02PM +0200, Sumit Bose wrote: From 3d5352977ca07f8a1a6d3bbaa57d11b54fd705d0 Mon Sep 17 00:00:00 2001 From: Sumit Bose Date: Wed, 7 Oct 2015

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-08 Thread Pavel Březina
On 10/07/2015 04:28 PM, Sumit Bose wrote: On Wed, Oct 07, 2015 at 04:22:31PM +0200, Sumit Bose wrote: On Wed, Oct 07, 2015 at 02:42:18PM +0200, Pavel Březina wrote: On 10/06/2015 03:16 PM, Pavel Březina wrote: On 10/01/2015 05:06 PM, Sumit Bose wrote: On Thu, Sep 17, 2015 at 11:06:30AM

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-08 Thread Jakub Hrozek
On Thu, Oct 08, 2015 at 08:34:17PM +0200, Jakub Hrozek wrote: > On Thu, Oct 08, 2015 at 01:21:29PM +0200, Pavel Březina wrote: > > On 10/07/2015 04:28 PM, Sumit Bose wrote: > > >On Wed, Oct 07, 2015 at 04:22:31PM +0200, Sumit Bose wrote: > > >>On Wed, Oct 07, 2015 at 02:42:18PM +0200, Pavel

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-08 Thread Jakub Hrozek
On Wed, Oct 07, 2015 at 04:28:02PM +0200, Sumit Bose wrote: > From 3d5352977ca07f8a1a6d3bbaa57d11b54fd705d0 Mon Sep 17 00:00:00 2001 > From: Sumit Bose > Date: Wed, 7 Oct 2015 15:22:34 +0200 > Subject: [PATCH 3/3] nss: fix UPN lookups for sub-domain users This one should go to

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-08 Thread Jakub Hrozek
On Thu, Oct 08, 2015 at 01:21:29PM +0200, Pavel Březina wrote: > On 10/07/2015 04:28 PM, Sumit Bose wrote: > >On Wed, Oct 07, 2015 at 04:22:31PM +0200, Sumit Bose wrote: > >>On Wed, Oct 07, 2015 at 02:42:18PM +0200, Pavel Březina wrote: > >>>On 10/06/2015 03:16 PM, Pavel Březina wrote: > On

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-07 Thread Sumit Bose
On Wed, Oct 07, 2015 at 04:22:31PM +0200, Sumit Bose wrote: > On Wed, Oct 07, 2015 at 02:42:18PM +0200, Pavel Březina wrote: > > On 10/06/2015 03:16 PM, Pavel Březina wrote: > > >On 10/01/2015 05:06 PM, Sumit Bose wrote: > > >>On Thu, Sep 17, 2015 at 11:06:30AM +0200, Pavel Březina wrote: > >

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-07 Thread Sumit Bose
On Wed, Oct 07, 2015 at 02:42:18PM +0200, Pavel Březina wrote: > On 10/06/2015 03:16 PM, Pavel Březina wrote: > >On 10/01/2015 05:06 PM, Sumit Bose wrote: > >>On Thu, Sep 17, 2015 at 11:06:30AM +0200, Pavel Březina wrote: > >>>On 09/17/2015 10:32 AM, Petr Cech wrote: > Hi Pavel! > >

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-06 Thread Pavel Březina
On 10/01/2015 05:06 PM, Sumit Bose wrote: On Thu, Sep 17, 2015 at 11:06:30AM +0200, Pavel Březina wrote: On 09/17/2015 10:32 AM, Petr Cech wrote: Hi Pavel! There is some code between my last end and this continuation. I was read it and did't find anything wrong. Hi Petr, thank you for you

Re: [SSSD] [PATCH] cache_req: support UPN

2015-10-01 Thread Pavel Březina
On 09/17/2015 11:06 AM, Pavel Březina wrote: On 09/17/2015 10:32 AM, Petr Cech wrote: Hi Pavel! There is some code between my last end and this continuation. I was read it and did't find anything wrong. Hi Petr, thank you for you review. New patches are attached. All comments from the

Re: [SSSD] [PATCH] cache_req: support UPN

2015-09-18 Thread Petr Cech
On 09/17/2015 11:06 AM, Pavel Březina wrote: I wanted to avoid writing types for send_fn and done_fn. I'll do it if you want me to, but I think this is good enough for test. Well. # return run_cache_req(...) but it doesn't provide value. It was there originally then I realized I don't have

Re: [SSSD] [PATCH] cache_req: support UPN

2015-09-17 Thread Petr Cech
Hi Pavel! There is some code between my last end and this continuation. I was read it and did't find anything wrong. On 09/16/2015 04:26 PM, Petr Cech wrote: 0003-cache_req-tests-reduce-code-duplication.patch From e41f96a47f2b0f8d3e07e34af83e9a516d29df34 Mon Sep 17 00:00:00 2001 From:

Re: [SSSD] [PATCH] cache_req: support UPN

2015-09-16 Thread Petr Cech
On 09/14/2015 01:34 PM, Pavel Březina wrote: On 09/14/2015 01:32 PM, Pavel Březina wrote: 0001: Use extra flag also in OOB request. 0002: Provide support for UPN. This add an improvement from NSS code, but I'm not sure if it is desired or not. If you have [domain/AD.PB] in sssd.conf and UPN

Re: [SSSD] [PATCH] cache_req: support UPN

2015-09-16 Thread Jakub Hrozek
On Wed, Sep 16, 2015 at 03:50:37PM +0200, Petr Cech wrote: > On 09/14/2015 01:34 PM, Pavel Březina wrote: > >On 09/14/2015 01:32 PM, Pavel Březina wrote: > >>0001: > >>Use extra flag also in OOB request. > >> > >>0002: > >>Provide support for UPN. This add an improvement from NSS code, but I'm >

Re: [SSSD] [PATCH] cache_req: support UPN

2015-09-14 Thread Pavel Březina
On 09/14/2015 01:32 PM, Pavel Březina wrote: 0001: Use extra flag also in OOB request. 0002: Provide support for UPN. This add an improvement from NSS code, but I'm not sure if it is desired or not. If you have [domain/AD.PB] in sssd.conf and UPN "u...@ad.pb" then NSS responder will not find