Re: [Freeipa-devel] [PATCH] 147 Set network.http.sendRefererHeader to 2 on browser config

2012-06-20 Thread Endi Sukma Dewata
Sorry for the late response. I have some questions below. On 6/11/2012 1:38 PM, Rob Crittenden wrote: Petr Vobornik wrote: This patch adds a functionality which sets Firefox network.http.sendRefererHeader configuration option to value '2' which enables it. Possible values: http://kb.mozillazin

[Freeipa-devel] Updated 389-ds-base released

2012-06-20 Thread Rob Crittenden
An update of 389-ds-base has been released which should resolve the problems that IPA was having. 389-ds-base-1.2.11.5-1.fc17 corrects the problems we were seeing with managed entries. Don't forget to remove 389-ds-base from excludes in your yum.conf and/or use yum versionlock delete 389-ds-ba

Re: [Freeipa-devel] [PATCH] 1023 tool for configuring automount

2012-06-20 Thread Rob Crittenden
Rob Crittenden wrote: Rob Crittenden wrote: Here is a tool that can be used to configure automount in an IPA client. It can use either SSSD or autofs for automount. It also configures NFSv4 on the client so secure maps will work. rebased patch rebase again rob >From b4e857bc1e4b4d3bda7e06ad

Re: [Freeipa-devel] [PATCH] 158 Separate reset password page

2012-06-20 Thread Endi Sukma Dewata
On 6/19/2012 1:22 AM, Petr Vobornik wrote: Updated patch attached. Sorry for late update, I was working on trust UI and didn't want to switch context. No problem. It would be nicer if this page can use the same fonts as the login dialog in patch #157, but that can be fixed separately. ACK.

Re: [Freeipa-devel] [PATCH] 157 Added password reset capabilities to unauthorized dialog

2012-06-20 Thread Endi Sukma Dewata
ACK, but feel free to make additional minor changes as suggested below before push. On 6/19/2012 9:01 AM, Petr Vobornik wrote: In general I like simplifying the dialog so I took most of your suggestions and implemented them following way: Login -

Re: [Freeipa-devel] [PATCH] 0056 Framework for admin/install tools, with ipa-ldap-updater

2012-06-20 Thread Rob Crittenden
Petr Viktorin wrote: On 06/04/2012 04:56 PM, Petr Viktorin wrote: Currently, FreeIPA's install/admin scripts are long pieces of code that aren't very reusable, importable, or testable. They have been extended over time with features such as logging and error handling, but since each tool was ext

Re: [Freeipa-devel] [PATCH] 26 Fix '--random' param behaviour for host plugin

2012-06-20 Thread Ondrej Hamada
On 06/15/2012 07:36 AM, Martin Kosek wrote: On Thu, 2012-06-14 at 16:35 -0400, Rob Crittenden wrote: Ondrej Hamada wrote: Improved options checking so that host-mod operation is not changing password for enrolled host when '--random' option is used. https://fedorahosted.org/freeipa/ticket/2799

Re: [Freeipa-devel] [PATCH] 1027 add logging to ipa-upgradeconfig

2012-06-20 Thread Rob Crittenden
Petr Viktorin wrote: On 06/18/2012 10:43 PM, Rob Crittenden wrote: Add some additional logging to ipa-upgradeconfig and have it update /var/log/ipaupgrade.log so we can see what an upgrade has already done. rob I don't think the indentation works too well: 2012-06-19T12:52:47Z INFO Verifyin

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-20 Thread Simo Sorce
On Wed, 2012-06-20 at 12:47 +0200, Martin Kosek wrote: > On Wed, 2012-06-20 at 11:10 +0200, Petr Viktorin wrote: > > On 06/12/2012 02:39 PM, Petr Viktorin wrote: > > > On 06/12/2012 02:38 PM, Simo Sorce wrote: > > >> On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: > > >>> This will make old

Re: [Freeipa-devel] [PATCH] 27 Case sensitive renaming of objects

2012-06-20 Thread Martin Kosek
On Wed, 2012-06-20 at 15:21 +0200, Ondrej Hamada wrote: > https://fedorahosted.org/freeipa/ticket/2620 > > When renaming object its case sensitivity is obeyed. This was DS bug. > Unit tests were corrected and minimal DS version was updated in spec > file. > > The minimal version for F16 and lower

Re: [Freeipa-devel] [PATCH] 0050 Fail on unknown Command options

2012-06-20 Thread Martin Kosek
On Tue, 2012-06-12 at 14:38 +0200, Petr Viktorin wrote: > On 06/07/2012 11:37 AM, Petr Vobornik wrote: > > On 05/28/2012 04:16 PM, Martin Kosek wrote: > >> On Mon, 2012-05-28 at 15:46 +0200, Petr Vobornik wrote: > >>> On 05/25/2012 09:20 AM, Petr Vobornik wrote: > On 05/16/2012 02:11 PM, Marti

[Freeipa-devel] [PATCH] 27 Case sensitive renaming of objects

2012-06-20 Thread Ondrej Hamada
https://fedorahosted.org/freeipa/ticket/2620 When renaming object its case sensitivity is obeyed. This was DS bug. Unit tests were corrected and minimal DS version was updated in spec file. The minimal version for F16 and lower is still in koji only, so we may wait with pushing these patch. --

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-20 Thread Martin Kosek
On Wed, 2012-06-20 at 11:10 +0200, Petr Viktorin wrote: > On 06/12/2012 02:39 PM, Petr Viktorin wrote: > > On 06/12/2012 02:38 PM, Simo Sorce wrote: > >> On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: > >>> This will make older clients usable if new output items get added to > >>> commands

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-20 Thread Petr Viktorin
On 06/12/2012 02:39 PM, Petr Viktorin wrote: On 06/12/2012 02:38 PM, Simo Sorce wrote: On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not