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

[Martin Kosek]

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


Martin