Re: [Freeipa-devel] [PATCH] 658 don't allow attrs=None

2011-01-10 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/22/2010 04:22 PM, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: Setting an empty set of target attributes should raise an exception. It is possible to create an ACI with attributes and then try to set that to None via a

Re: [Freeipa-devel] [PATCH] 658 don't allow attrs=None

2011-01-10 Thread Rob Crittenden
Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 12/22/2010 04:22 PM, Rob Crittenden wrote: Rob Crittenden wrote: Rob Crittenden wrote: Setting an empty set of target attributes should raise an exception. It is possible to create an ACI with attributes and then try to

Re: [Freeipa-devel] [PATCH] 658 don't allow attrs=None

2010-12-22 Thread Rob Crittenden
Rob Crittenden wrote: Setting an empty set of target attributes should raise an exception. It is possible to create an ACI with attributes and then try to set that to None via a mod command later. We need to catch this and raise an exception. ticket 647 rob I'm going to withdraw this patch

Re: [Freeipa-devel] [PATCH] 658 don't allow attrs=None

2010-12-22 Thread Rob Crittenden
Rob Crittenden wrote: Rob Crittenden wrote: Setting an empty set of target attributes should raise an exception. It is possible to create an ACI with attributes and then try to set that to None via a mod command later. We need to catch this and raise an exception. ticket 647 rob I'm going