Re: [Freeipa-devel] [PATCH] 879 ensure ssl socket is shut down

2011-09-20 Thread Jan Cholasta
On 19.9.2011 23:32, Rob Crittenden wrote: httplib makes a copy of the nss file descriptor but doesn't close it when the response code != 200 so we need to close it ourselves. rob Can we be sure that httplib's behavior is consistent and won't change? I would rather try to close the fd

Re: [Freeipa-devel] [PATCH] 878 ignore restorecon errors

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 01:04 +0300, Alexander Bokovoy wrote: On Mon, 19 Sep 2011, Rob Crittenden wrote: According to the SELinux devs the return value from restorecon does not necessarily reflect success/failure and recommended ignoring it. This does that. ACK. I was about to submit similar

[Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
Remove an invalid instance name passed to dirsrv service so that it is correctly stopped. https://fedorahosted.org/freeipa/ticket/1800 From b2e5ba49e22509327bcc521370c11d1cbb9d8de4 Mon Sep 17 00:00:00 2001 From: Martin Kosek mko...@redhat.com Date: Tue, 20 Sep 2011 10:11:35 +0200 Subject:

Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Alexander Bokovoy
On Tue, 20 Sep 2011, Martin Kosek wrote: Remove an invalid instance name passed to dirsrv service so that it is correctly stopped. https://fedorahosted.org/freeipa/ticket/1800 ACK. -- / Alexander Bokovoy ___ Freeipa-devel mailing list

Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 11:47 +0300, Alexander Bokovoy wrote: On Tue, 20 Sep 2011, Martin Kosek wrote: Remove an invalid instance name passed to dirsrv service so that it is correctly stopped. https://fedorahosted.org/freeipa/ticket/1800 ACK. Pushed to master, ipa-2-1. Martin

Re: [Freeipa-devel] [PATCH] 124 ipactl does not stop dirsrv

2011-09-20 Thread Martin Kosek
On Tue, 2011-09-20 at 10:52 +0200, Martin Kosek wrote: On Tue, 2011-09-20 at 11:47 +0300, Alexander Bokovoy wrote: On Tue, 20 Sep 2011, Martin Kosek wrote: Remove an invalid instance name passed to dirsrv service so that it is correctly stopped.

Re: [Freeipa-devel] [PATCH] #1728 New schema for IPAv3 required attributes

2011-09-20 Thread Sumit Bose
On Mon, Sep 19, 2011 at 12:34:36PM -0400, Simo Sorce wrote: Attached find a patch for new attributes and objectclasses for the IPA v3 goal of configuring trust relationships between freeipa and windows domains. I think everything is ok, I just started to wonder if it is maybe safer to always

Re: [Freeipa-devel] [PATCH] 25 Create Tool for Enabling Disabling Managed Entry

2011-09-20 Thread Martin Kosek
On Fri, 2011-09-16 at 16:37 +, JR Aquino wrote: On Sep 16, 2011, at 2:11 AM, Martin Kosek wrote: On Thu, 2011-09-15 at 17:25 +, JR Aquino wrote: On Sep 15, 2011, at 1:47 AM, Martin Kosek wrote: On Thu, 2011-09-15 at 00:47 +, JR Aquino wrote: On Jul 22, 2011, at 7:05 AM,

Re: [Freeipa-devel] [PATCH] 876 normalize user principal

2011-09-20 Thread Martin Kosek
On Fri, 2011-09-16 at 10:16 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Normalize and validate user principals in user and passwd plugins. The uid in the principal should be lower-case. rob With updated API.txt rob This works fine. I would just suggest improving the way how

Re: [Freeipa-devel] [PATCH] #1728 New schema for IPAv3 required attributes

2011-09-20 Thread Simo Sorce
On Tue, 2011-09-20 at 12:36 +0200, Sumit Bose wrote: On Mon, Sep 19, 2011 at 12:34:36PM -0400, Simo Sorce wrote: Attached find a patch for new attributes and objectclasses for the IPA v3 goal of configuring trust relationships between freeipa and windows domains. I think everything is

Re: [Freeipa-devel] [PATCH] #1728 New schema for IPAv3 required attributes

2011-09-20 Thread Sumit Bose
On Tue, Sep 20, 2011 at 08:47:58AM -0400, Simo Sorce wrote: On Tue, 2011-09-20 at 12:36 +0200, Sumit Bose wrote: On Mon, Sep 19, 2011 at 12:34:36PM -0400, Simo Sorce wrote: Attached find a patch for new attributes and objectclasses for the IPA v3 goal of configuring trust relationships

Re: [Freeipa-devel] [PATCH] 273 Removed HBAC rule type.

2011-09-20 Thread Petr Vobornik
On 09/16/2011 06:24 PM, Endi Sukma Dewata wrote: HBAC rule type has been removed from the list page and details page because it is no longer supported in IPA 3.0. Ticket #1795 This should be pushed to master branch only. ___ Freeipa-devel mailing

Re: [Freeipa-devel] [PATCH] #1728 New schema for IPAv3 required attributes

2011-09-20 Thread Simo Sorce
On Tue, 2011-09-20 at 14:59 +0200, Sumit Bose wrote: On Tue, Sep 20, 2011 at 08:47:58AM -0400, Simo Sorce wrote: On Tue, 2011-09-20 at 12:36 +0200, Sumit Bose wrote: On Mon, Sep 19, 2011 at 12:34:36PM -0400, Simo Sorce wrote: Attached find a patch for new attributes and objectclasses for

Re: [Freeipa-devel] Structured DNS record API proposal

2011-09-20 Thread Endi Sukma Dewata
On 9/20/2011 6:15 AM, Martin Kosek wrote: ACK. Proposal looks like it will work fairly easily with the UI. We'll have to make some chagnes due to the Add doing something different based on the type, but that is the case anyway. Yes, I was thinking how can we integrate this new API to WebUI.

Re: [Freeipa-devel] Still failing on 5.7 with the same error........

2011-09-20 Thread JR Aquino
On Sep 19, 2011, at 10:16 PM, JR Aquino wrote: We're having significant reproducible problems with rhel 5.7 + FreeIPA master... I'm not sure if it is localized to us or even which side is responsible for the error... Has anyone had success with rhel 5.7's repo included FreeIPA client

[Freeipa-devel] [PATCH] 280 Fixed problem displaying special characters.

2011-09-20 Thread Endi Sukma Dewata
Some jQuery objects in various locations have been modified to use text() to show values obtained from the server (except messages). The text() will automatically encode special characters. Ticket #1798 -- Endi S. Dewata From 2af9922e6e271f1f48a47d20296facee5d6cb049 Mon Sep 17 00:00:00 2001

Re: [Freeipa-devel] Still failing on 5.7 with the same error........

2011-09-20 Thread Rob Crittenden
JR Aquino wrote: On Sep 19, 2011, at 10:16 PM, JR Aquino wrote: We're having significant reproducible problems with rhel 5.7 + FreeIPA master... I'm not sure if it is localized to us or even which side is responsible for the error... Has anyone had success with rhel 5.7's repo included