Re: [Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-07-12 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2011-07-11 at 17:45 -0400, Rob Crittenden wrote: Martin Kosek wrote: On Fri, 2011-06-24 at 16:37 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly

Re: [Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-07-11 Thread Martin Kosek
On Fri, 2011-06-24 at 16:37 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly every command other than add query is True so all rules were ignored on the primary key. This

Re: [Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-07-11 Thread Rob Crittenden
Martin Kosek wrote: On Fri, 2011-06-24 at 16:37 -0400, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly every command other than add query is True so all rules were ignored on the

[Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-06-24 Thread Rob Crittenden
This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly every command other than add query is True so all rules were ignored on the primary key. This meant that to enforce whitespace we would need to define a validator for each one. I decided instead to

Re: [Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-06-24 Thread Rob Crittenden
Rob Crittenden wrote: This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly every command other than add query is True so all rules were ignored on the primary key. This meant that to enforce whitespace we would need to define a validator for each one. I

Re: [Freeipa-devel] [PATCH] 808 don't allow leading/trailing whitespace in strings

2011-06-24 Thread Rob Crittenden
Rob Crittenden wrote: Rob Crittenden wrote: This started as a problem in allowing leading/trailing whitespaces on primary keys. In nearly every command other than add query is True so all rules were ignored on the primary key. This meant that to enforce whitespace we would need to define a