Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-03-01 Thread Rob Crittenden
Jan Cholasta wrote: On 29.2.2012 19:45, Rob Crittenden wrote: Jan Cholasta wrote: On 20.2.2012 22:56, Rob Crittenden wrote: Rob Crittenden wrote: The variable name rdnattr can be misleading. It is only used to give the name of hte RDN in something that can be renamed. Compare this to

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-29 Thread Jan Cholasta
On 20.2.2012 22:56, Rob Crittenden wrote: Rob Crittenden wrote: The variable name rdnattr can be misleading. It is only used to give the name of hte RDN in something that can be renamed. Compare this to something like netgroups where the DN has no visible relationship to the content of the

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-29 Thread Rob Crittenden
Jan Cholasta wrote: On 20.2.2012 22:56, Rob Crittenden wrote: Rob Crittenden wrote: The variable name rdnattr can be misleading. It is only used to give the name of hte RDN in something that can be renamed. Compare this to something like netgroups where the DN has no visible relationship to

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-20 Thread Rob Crittenden
Rob Crittenden wrote: The variable name rdnattr can be misleading. It is only used to give the name of hte RDN in something that can be renamed. Compare this to something like netgroups where the DN has no visible relationship to the content of the object (ipaUniqueId). Only those objects that

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-15 Thread Jan Cholasta
On 14.2.2012 22:16, Rob Crittenden wrote: Jan Cholasta wrote: On 14.2.2012 16:44, Jan Cholasta wrote: On 7.2.2012 20:25, Rob Crittenden wrote: Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote:

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-15 Thread Jan Cholasta
On 15.2.2012 15:33, Rob Crittenden wrote: Jan Cholasta wrote: On 14.2.2012 22:16, Rob Crittenden wrote: Jan Cholasta wrote: On 14.2.2012 16:44, Jan Cholasta wrote: On 7.2.2012 20:25, Rob Crittenden wrote: Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-14 Thread Jan Cholasta
On 7.2.2012 20:25, Rob Crittenden wrote: Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-14 Thread Jan Cholasta
On 14.2.2012 16:44, Jan Cholasta wrote: On 7.2.2012 20:25, Rob Crittenden wrote: Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-14 Thread Rob Crittenden
Jan Cholasta wrote: On 14.2.2012 16:44, Jan Cholasta wrote: On 7.2.2012 20:25, Rob Crittenden wrote: Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Martin Kosek
On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is being created, namely the key itself. This is to allow us to manage an otherwise

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Jan Cholasta
Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is being created, namely the key itself. This is

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is being created, namely the key itself. This is to allow us to manage an

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Rob Crittenden
Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is being created, namely the

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Jan Cholasta
Dne 7.2.2012 15:15, Rob Crittenden napsal(a): Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-07 Thread Rob Crittenden
Rob Crittenden wrote: Jan Cholasta wrote: Dne 7.2.2012 09:27, Martin Kosek napsal(a): On Mon, 2012-02-06 at 11:52 -0500, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is

Re: [Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-06 Thread Martin Kosek
On Fri, 2012-02-03 at 16:58 -0500, Rob Crittenden wrote: There is some validation that we only need to apply when an entry is being created, namely the key itself. This is to allow us to manage an otherwise illegal entry that finds its way into the system (i.e. migration). Consider this.

[Freeipa-devel] [PATCH] 940 apply some validation to some classes only

2012-02-03 Thread Rob Crittenden
There is some validation that we only need to apply when an entry is being created, namely the key itself. This is to allow us to manage an otherwise illegal entry that finds its way into the system (i.e. migration). Consider this. We migrate a group with a space in it. This isn't allowed in