Re: [SSSD] RFC: Improving the debug messages

2015-10-01 Thread Pavel Březina
On 09/29/2015 07:53 PM, Jakub Hrozek wrote: On Tue, Sep 29, 2015 at 06:15:35PM +0200, Sumit Bose wrote: On Tue, Sep 29, 2015 at 05:57:32PM +0200, Jakub Hrozek wrote: On Mon, Sep 28, 2015 at 10:18:07AM +0200, Sumit Bose wrote: On Mon, Jun 29, 2015 at 11:07:30PM -0400, Dan Lavu wrote: I've

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] RFC: Talloc reports

2015-10-01 Thread Pavel Březina
On 09/29/2015 09:30 PM, Pavel Reichl wrote: On 09/29/2015 05:59 PM, Jakub Hrozek wrote: On Tue, Sep 29, 2015 at 02:12:48PM +0200, Pavel Reichl wrote: On 09/29/2015 02:04 PM, Pavel Březina wrote: On 09/03/2015 12:48 PM, Pavel Březina wrote: Hi, due to recent memory leak issues, I think it

Re: [SSSD] RFC: Improving the debug messages

2015-10-01 Thread Pavel Reichl
On 10/01/2015 09:58 AM, Pavel Březina wrote: On 09/29/2015 07:53 PM, Jakub Hrozek wrote: On Tue, Sep 29, 2015 at 06:15:35PM +0200, Sumit Bose wrote: On Tue, Sep 29, 2015 at 05:57:32PM +0200, Jakub Hrozek wrote: On Mon, Sep 28, 2015 at 10:18:07AM +0200, Sumit Bose wrote: On Mon, Jun 29,

Re: [SSSD] [PATCH] [HBAC]: Better libhbac debuging

2015-10-01 Thread Michal Židek
On 10/01/2015 12:37 PM, Petr Cech wrote: On 09/18/2015 04:30 PM, Petr Cech wrote: Hello, there are fixed patches attached. Bump. The code was already acked by PavelR, the only problem was with the version info change and the version script. I checked the last iteration and both version

Re: [SSSD] Review of umask() in SSSD

2015-10-01 Thread Petr Cech
On 10/01/2015 01:19 PM, Jakub Hrozek wrote: On Thu, Oct 01, 2015 at 12:38:49PM +0200, Petr Cech wrote: Bump. Thanks for reply, Jakub. Why was 077 changed for 0177? This change is something, which I think was discussed earlier in this thread. # pcech: # > I would like to ask you if we

Re: [SSSD] Review of umask() in SSSD

2015-10-01 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 12:38:49PM +0200, Petr Cech wrote: > Bump. Why was 077 changed for 0177? About the name -- shouldn't we say just "SSS_DFL_UMASK" ? We are a security project, therefore restrictive by default :-) ___ sssd-devel mailing list

[SSSD] [PATCH] IPA: Check connection validity explicitly

2015-10-01 Thread Jakub Hrozek
Hi, the attached patch (hopefully) fixes https://fedorahosted.org/sssd/ticket/2810 (gdb) bt #0 0x7f9e8dde21d7 in ipa_get_ad_acct_send (ar=0x7f9e9cc2ff80, override_attrs=0x0, be_req=, ipa_ctx=, ev=0x7f9e9cbf0670, mem_ctx=) at src/providers/ipa/ipa_subdomains_id.c:648 #1

Re: [SSSD] [PATCH] [HBAC]: Better libhbac debuging

2015-10-01 Thread Michal Židek
On 10/01/2015 01:03 PM, Pavel Reichl wrote: On 10/01/2015 12:59 PM, Michal Židek wrote: On 10/01/2015 12:37 PM, Petr Cech wrote: On 09/18/2015 04:30 PM, Petr Cech wrote: Hello, there are fixed patches attached. Bump. The code was already acked by PavelR, the only problem was with the

Re: [SSSD] [PATCH] PAM: only allow missing user name for certificate

2015-10-01 Thread Pavel Reichl
diff --git a/src/responder/pam/pamsrv_cmd.c b/src/responder/pam/pamsrv_cmd.c index 27dddcf43c1ff6eb465e1cb58d6dddf21413dcc4..978c637e22b03d3be1e07e8dc713aa01c4bb22e5 100644 --- a/src/responder/pam/pamsrv_cmd.c +++ b/src/responder/pam/pamsrv_cmd.c @@ -957,11 +957,13 @@ static errno_t

Re: [SSSD] [PATCH] sbus codegen tests: free ctx

2015-10-01 Thread Lukas Slebodnik
On (01/10/15 14:36), Pavel Březina wrote: >https://fedorahosted.org/sssd/ticket/2759 >From ad00807733ed3150d06615de035c6662af067673 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Pavel=20B=C5=99ezina?= >Date: Thu, 1 Oct 2015 14:34:51 +0200 >Subject: [PATCH] sbus codegen tests:

Re: [SSSD] [PATCH] [HBAC]: Better libhbac debuging

2015-10-01 Thread Pavel Reichl
On 10/01/2015 12:59 PM, Michal Židek wrote: On 10/01/2015 12:37 PM, Petr Cech wrote: On 09/18/2015 04:30 PM, Petr Cech wrote: Hello, there are fixed patches attached. Bump. The code was already acked by PavelR, the only problem was with the version info change and the version script. I

Re: [SSSD] [PATCHES] DYDNDS: update quality of input for nsupdate

2015-10-01 Thread Pavel Reichl
On 09/30/2015 03:43 PM, Jakub Hrozek wrote: On Tue, Sep 29, 2015 at 11:40:34AM +0200, Pavel Reichl wrote: On 09/21/2015 05:47 PM, Jakub Hrozek wrote: On Mon, Sep 21, 2015 at 01:03:13PM +0200, Pavel Reichl wrote: On 09/21/2015 10:52 AM, Pavel Březina wrote: On 08/28/2015 11:05 AM, Pavel

Re: [SSSD] [PATCH] PAM: only allow missing user name for certificate

2015-10-01 Thread Lukas Slebodnik
On (01/10/15 10:57), Sumit Bose wrote: >Hi, > >this patch aims to fix https://fedorahosted.org/sssd/ticket/2811. It >occurs if the client of sssd_pam does not provide a user name during >pre-auth and certificate/Smartcard authentication is not enabled. > >I relaxed the user name check when adding

[SSSD] [PATCH] sbus codegen tests: free ctx

2015-10-01 Thread Pavel Březina
https://fedorahosted.org/sssd/ticket/2759 >From ad00807733ed3150d06615de035c6662af067673 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date: Thu, 1 Oct 2015 14:34:51 +0200 Subject: [PATCH] sbus codegen tests: free ctx Memory context was not freed therefore

Re: [SSSD] [PATCH] [HBAC]: Better libhbac debuging

2015-10-01 Thread Petr Cech
On 09/18/2015 04:30 PM, Petr Cech wrote: Hello, there are fixed patches attached. Bump. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] Review of umask() in SSSD

2015-10-01 Thread Petr Cech
Bump. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] [PATCH] util: Include disabled domains in link_forest_roots

2015-10-01 Thread Michal Židek
On 09/16/2015 03:56 PM, Michal Židek wrote: On 09/15/2015 04:03 PM, Jakub Hrozek wrote: On Tue, Sep 15, 2015 at 03:53:41PM +0200, Michal Židek wrote: On 09/14/2015 05:43 PM, Jakub Hrozek wrote: On Wed, Sep 09, 2015 at 02:43:59PM +0200, Michal Židek wrote: Hi, patch for ticket

Re: [SSSD] [PATCH] util: Include disabled domains in link_forest_roots

2015-10-01 Thread Lukas Slebodnik
On (01/10/15 13:56), Michal Židek wrote: >On 09/16/2015 03:56 PM, Michal Židek wrote: >>On 09/15/2015 04:03 PM, Jakub Hrozek wrote: >>>On Tue, Sep 15, 2015 at 03:53:41PM +0200, Michal Židek wrote: On 09/14/2015 05:43 PM, Jakub Hrozek wrote: >On Wed, Sep 09, 2015 at 02:43:59PM +0200, Michal

Re: [SSSD] [PATCH] util: Include disabled domains in link_forest_roots

2015-10-01 Thread Michal Židek
On 10/01/2015 03:08 PM, Lukas Slebodnik wrote: On (01/10/15 13:56), Michal Židek wrote: On 09/16/2015 03:56 PM, Michal Židek wrote: On 09/15/2015 04:03 PM, Jakub Hrozek wrote: On Tue, Sep 15, 2015 at 03:53:41PM +0200, Michal Židek wrote: On 09/14/2015 05:43 PM, Jakub Hrozek wrote: On Wed,

Re: [SSSD] [PATCH] SDAP: Relax POSIX check

2015-10-01 Thread Jakub Hrozek
On Fri, Sep 25, 2015 at 07:49:23AM +0200, Lukas Slebodnik wrote: > There is a common agreement (at least from management side) that we want > to improve code coverage This patch is great opportunity to add regression > test. You can choose either integration test or unit test; you should know >

Re: [SSSD] [PATCH] SDAP: Relax POSIX check

2015-10-01 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 08:31:50PM +0200, Lukas Slebodnik wrote: > On (01/10/15 20:13), Jakub Hrozek wrote: > >On Fri, Sep 25, 2015 at 07:49:23AM +0200, Lukas Slebodnik wrote: > >> There is a common agreement (at least from management side) that we want > >> to improve code coverage This patch is

Re: [SSSD] [PATCH] SDAP: Relax POSIX check

2015-10-01 Thread Lukas Slebodnik
On (01/10/15 20:13), Jakub Hrozek wrote: >On Fri, Sep 25, 2015 at 07:49:23AM +0200, Lukas Slebodnik wrote: >> There is a common agreement (at least from management side) that we want >> to improve code coverage This patch is great opportunity to add regression >> test. You can choose either

Re: [SSSD] [PATCH] SDAP: Relax POSIX check

2015-10-01 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 08:39:15PM +0200, Jakub Hrozek wrote: > On Thu, Oct 01, 2015 at 08:31:50PM +0200, Lukas Slebodnik wrote: > > On (01/10/15 20:13), Jakub Hrozek wrote: > > >On Fri, Sep 25, 2015 at 07:49:23AM +0200, Lukas Slebodnik wrote: > > >> There is a common agreement (at least from

[SSSD] FYI: master branch is tracking sssd-1-14

2015-10-01 Thread Jakub Hrozek
sssd-1-13 has a separate branch. I would prefer to push mostly bugfixes and RFEs requires by downstream to sssd-1-13 and keep most of the churn in master. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

Re: [SSSD] [PATCH] [HBAC]: Better libhbac debuging

2015-10-01 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 01:40:34PM +0200, Michal Židek wrote: > On 10/01/2015 01:03 PM, Pavel Reichl wrote: > > > > > >On 10/01/2015 12:59 PM, Michal Židek wrote: > >>On 10/01/2015 12:37 PM, Petr Cech wrote: > >>>On 09/18/2015 04:30 PM, Petr Cech wrote: > Hello, > > there are fixed

[SSSD] Add Catalan support in sssd

2015-10-01 Thread Robert Antoni Buj i Gelonch
Hi, This patch adds Catalan support. Both /src/man/po/ca.po & po/ca.po files exit but they are not used to generate and distribute the translations. Regards, Robert -- http://about.me/rbuj 0001-add-Catalan.patch Description: Binary data ___

Re: [SSSD] Add Catalan support in sssd

2015-10-01 Thread Jakub Hrozek
On Thu, Oct 01, 2015 at 09:28:48PM +0200, Robert Antoni Buj i Gelonch wrote: > Hi, > > This patch adds Catalan support. Both /src/man/po/ca.po & po/ca.po files > exit but they are not used to generate and distribute the translations. > > Regards, > Robert > > -- > http://about.me/rbuj Oops,

Re: [SSSD] CI: Already fixed bug for TODO

2015-10-01 Thread Lukas Slebodnik
On (01/10/15 08:44), Lukas Slebodnik wrote: >On (30/09/15 17:39), Nikolai Kondrashov wrote: >>On 09/30/2015 08:20 AM, Lukas Slebodnik wrote: >>>On (29/09/15 17:35), Nikolai Kondrashov wrote: On 09/29/2015 05:13 PM, Nikolai Kondrashov wrote: >On 09/29/2015 03:54 PM, Lukas Slebodnik wrote:

Re: [SSSD] CI: Already fixed bug for TODO

2015-10-01 Thread Lukas Slebodnik
On (30/09/15 17:39), Nikolai Kondrashov wrote: >On 09/30/2015 08:20 AM, Lukas Slebodnik wrote: >>On (29/09/15 17:35), Nikolai Kondrashov wrote: >>>On 09/29/2015 05:13 PM, Nikolai Kondrashov wrote: On 09/29/2015 03:54 PM, Lukas Slebodnik wrote: >ehlo, > >I touched the CI script and

Re: [SSSD] RFC: Talloc reports

2015-10-01 Thread Pavel Reichl
On 10/01/2015 09:52 AM, Pavel Březina wrote: On 09/29/2015 09:30 PM, Pavel Reichl wrote: On 09/29/2015 05:59 PM, Jakub Hrozek wrote: On Tue, Sep 29, 2015 at 02:12:48PM +0200, Pavel Reichl wrote: On 09/29/2015 02:04 PM, Pavel Březina wrote: On 09/03/2015 12:48 PM, Pavel Březina wrote:

[SSSD] [PATCH] PAM: only allow missing user name for certificate

2015-10-01 Thread Sumit Bose
Hi, this patch aims to fix https://fedorahosted.org/sssd/ticket/2811. It occurs if the client of sssd_pam does not provide a user name during pre-auth and certificate/Smartcard authentication is not enabled. I relaxed the user name check when adding the Smartcard support and added tests as well