[Freeipa-devel] [PATCH] 929 add option for ipaconfigstrig

[Rob Crittenden]

Martin Kosek wrote:
> On Tue, 2012-02-07 at 08:56 -0500, Rob Crittenden wrote:
>> Convert from a freeform string into a enumeration.
>>
>> Only values currently allowed are AllowLMhash and AllowNThash.
>>
>> To add more than one value on the command-line specify --ipaconfigstring
>> multiple times.
>>
>> rob
>
> NACK
>
> 1) The patch needs rebasing (conflicts in VERSION file)
>
> 2) I think we should add csv=True to this parameter so that user can
> specify pwd plugin features in a CSV format:
>
> # ipa config-mod --ipaconfigstring=AllowLMhash,AllowNThash
>
> It is much more user friendly than:
>
> # ipa config-mod --ipaconfigstring=AllowLMhash --ipaconfigstring=AllowNThash

Sure, good idea. Updated patch attached.

rob

-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-rcrit-929-2-ipaconfigstring.patch
Type: application/mbox
Size: 4272 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120208/bdbd4af4/attachment.mbox>