Re: [Freeipa-devel] [PATCH] 262-265 Enable psearch by default

2012-06-08 Thread Martin Kosek
On Tue, 2012-06-05 at 09:32 +0200, Martin Kosek wrote: > On Mon, 2012-06-04 at 23:49 -0400, Rob Crittenden wrote: > > Martin Kosek wrote: > > > On Fri, 2012-05-25 at 17:14 +0200, Martin Kosek wrote: > > >> On Fri, 2012-05-25 at 09:25 -0400, Rob Crittenden wrote: > > >>> Martin Kosek wrote: > >

Re: [Freeipa-devel] [PATCH 0022] fix crash during zone unload when NS is not resolvable

2012-06-08 Thread Martin Kosek
On Thu, 2012-06-07 at 17:51 +0200, Adam Tkac wrote: > On Thu, Jun 07, 2012 at 04:03:46PM +0200, Martin Kosek wrote: > > On Thu, 2012-06-07 at 15:33 +0200, Petr Spacek wrote: > > > Hello, > > > > > > this patch fixes https://fedorahosted.org/bind-dyndb-ldap/ticket/77: > > > bind-dyndb-ldap crashes

[Freeipa-devel] [PATCH] 0060 Clean keytabs before installing new keys into them

2012-06-08 Thread Petr Viktorin
This removes old principals for the newly installed realm from /etc/krb5.keytab before client installation. The ticket also mentioned doing this for server/replica installs, but in that case the keytab is removed and created from scratch. https://fedorahosted.org/freeipa/ticket/2698 -- PetrĀ³ F

Re: [Freeipa-devel] [PATCH] 0059 Fix update plugin order

2012-06-08 Thread Petr Viktorin
On 06/08/2012 04:39 AM, Rob Crittenden wrote: Petr Viktorin wrote: While messing with the ipa-ldap-updater, I found the order method was using an algorithm that could give incorrect results. I'm submitting a fix in an extra patch, as it's largely unrelated and shouldn't be so controversial. Ca

Re: [Freeipa-devel] [PATCH] 0059 Fix update plugin order

2012-06-08 Thread Petr Viktorin
On 06/08/2012 12:31 PM, Petr Viktorin wrote: On 06/08/2012 04:39 AM, Rob Crittenden wrote: Petr Viktorin wrote: While messing with the ipa-ldap-updater, I found the order method was using an algorithm that could give incorrect results. I'm submitting a fix in an extra patch, as it's largely unr

[Freeipa-devel] keys[-1] idiom?

2012-06-08 Thread John Dennis
I frequently see this idiom in the code in plugins and pre/post callbacks: keys[-1] My understanding is that keys in the positional parameter list of the command, correct? keys always appears in a formal parameter list with the parameter "options", which I believe are the keyword arguments o

Re: [Freeipa-devel] [PATCH] 262-265 Enable psearch by default

2012-06-08 Thread Rob Crittenden
Martin Kosek wrote: On Tue, 2012-06-05 at 09:32 +0200, Martin Kosek wrote: On Mon, 2012-06-04 at 23:49 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-05-25 at 17:14 +0200, Martin Kosek wrote: On Fri, 2012-05-25 at 09:25 -0400, Rob Crittenden wrote: Martin Kosek wrote: This se

Re: [Freeipa-devel] [PATCH] 262-265 Enable psearch by default

2012-06-08 Thread Martin Kosek
On Fri, 2012-06-08 at 11:10 -0400, Rob Crittenden wrote: > Martin Kosek wrote: > > On Tue, 2012-06-05 at 09:32 +0200, Martin Kosek wrote: > >> On Mon, 2012-06-04 at 23:49 -0400, Rob Crittenden wrote: > >>> Martin Kosek wrote: > On Fri, 2012-05-25 at 17:14 +0200, Martin Kosek wrote: > > On

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

2012-06-08 Thread Petr Vobornik
For those of you who are only interest in user perspective I prepared a set of screenshots to demonstrate workflow of password reset: http://pvoborni.fedorapeople.org/ux/reset_password_workflow.png Patch depends on mkosek #274. Web UI was missing a way how to reset expired password for normal

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

2012-06-08 Thread Petr Vobornik
and now the patch... On 06/08/2012 05:51 PM, Petr Vobornik wrote: For those of you who are only interest in user perspective I prepared a set of screenshots to demonstrate workflow of password reset: http://pvoborni.fedorapeople.org/ux/reset_password_workflow.png Patch depends on mkosek #274. W

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

2012-06-08 Thread Petr Vobornik
This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for u

Re: [Freeipa-devel] [PATCH] 19-21 Use exop instead of kadmin.local

2012-06-08 Thread Sumit Bose
On Thu, Jun 07, 2012 at 12:09:32PM +0200, Sumit Bose wrote: > now with patches :-) > On Thu, Jun 07, 2012 at 12:07:13PM +0200, Sumit Bose wrote: > > Hi, > > > > this patch fixes https://fedorahosted.org/freeipa/ticket/2513 and as a > > consequence makes https://fedorahosted.org/freeipa/ticket/2516