Re: [Freeipa-devel] [PATCH] 155 Fix ipa-managed-entries bind procedure

2011-11-15 Thread Martin Kosek
On Tue, 2011-11-15 at 14:12 -0500, Rob Crittenden wrote: > Martin Kosek wrote: > > Make sure that when Directory Manager password is entered, > > we directly do a simple bind instead of trying binding via GSSAPI. > > Also capture ldap.INVALID_CREDENTIALS exception and provide nice > > error message

Re: [Freeipa-devel] [PATCH] 155 Fix ipa-managed-entries bind procedure

2011-11-15 Thread Rob Crittenden
Martin Kosek wrote: Make sure that when Directory Manager password is entered, we directly do a simple bind instead of trying binding via GSSAPI. Also capture ldap.INVALID_CREDENTIALS exception and provide nice error message than crash. https://fedorahosted.org/freeipa/ticket/1927 ACK ___

Re: [Freeipa-devel] [PATCH] 1 Do lazy initializiation ipalib

2011-11-15 Thread Rob Crittenden
Jan Cholasta wrote: Dne 9.11.2011 11:59, Alexander Bokovoy napsal(a): On Wed, 09 Nov 2011, Jan Cholasta wrote: would't suffer from a facelift - currently it is messy, hard to read code, with bits nobody ever used or uses anymore, some of the docstrings and comments aren't correct, etc.) A revi

Re: [Freeipa-devel] [PATCH] 170 Remove redundant information from API.txt

2011-11-15 Thread Martin Kosek
On Tue, 2011-11-15 at 17:42 +0200, Alexander Bokovoy wrote: > On Tue, 15 Nov 2011, Martin Kosek wrote: > > > Some Param or Output attributes do not cause API incompatibility > > (e.g. doc, label or callables) and does not need to be included > > in API.txt. When these attributes are modified, a lo

Re: [Freeipa-devel] [PATCH] 170 Remove redundant information from API.txt

2011-11-15 Thread Alexander Bokovoy
On Tue, 15 Nov 2011, Martin Kosek wrote: > Some Param or Output attributes do not cause API incompatibility > (e.g. doc, label or callables) and does not need to be included > in API.txt. When these attributes are modified, a lot of bogus > changes may get in API.txt - making the real API changes

[Freeipa-devel] [PATCH] 170 Remove redundant information from API.txt

2011-11-15 Thread Martin Kosek
Some Param or Output attributes do not cause API incompatibility (e.g. doc, label or callables) and does not need to be included in API.txt. When these attributes are modified, a lot of bogus changes may get in API.txt - making the real API changes less detectable. https://fedorahosted.org/freeipa

Re: [Freeipa-devel] [PATCH] 169 Fix LDAP object parameter encoding

2011-11-15 Thread Martin Kosek
On Tue, 2011-11-15 at 12:33 +0200, Alexander Bokovoy wrote: > On Tue, 15 Nov 2011, Martin Kosek wrote: > > > Or maybe we should document them better? Right now there is no > > > documentation for a majority of those flags. > > > > > > > I renamed it to 'virtual_attribute' (as an opposite to phys

Re: [Freeipa-devel] [PATCH] 169 Fix LDAP object parameter encoding

2011-11-15 Thread Alexander Bokovoy
On Tue, 15 Nov 2011, Martin Kosek wrote: > > Or maybe we should document them better? Right now there is no > > documentation for a majority of those flags. > > > > I renamed it to 'virtual_attribute' (as an opposite to physical LDAP > attribute), I hope it makes more sense. > > You are right t

Re: [Freeipa-devel] [PATCH] 169 Fix LDAP object parameter encoding

2011-11-15 Thread Martin Kosek
On Mon, 2011-11-14 at 19:10 +0200, Alexander Bokovoy wrote: > On Mon, 14 Nov 2011, Martin Kosek wrote: > > Tests are now OK, there were a lot of errors related to permissions and > > hosts (--random was always on). > > > > I was also thinking if the encoding step shouldn't be done as late as > > p