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

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to