[Freeipa-devel] [freeipa PR#590][comment] Validate user input for cert-get-requestdata

LiptonB freeipa-github-notification at redhat.com
Wed Mar 15 15:54:20 UTC 2017


  URL: https://github.com/freeipa/freeipa/pull/590
Title: #590: Validate user input for cert-get-requestdata

LiptonB commented:
"""
I don't think one could really add a new helper without modifying the code, so there's probably no need to allow arbitrary strings. Given that, StrEnum seems appropriate.

For the record, https://github.com/freeipa/freeipa/pull/542 is going to be modified to remove the `helper` parameter of `cert-get-requestdata` entirely, though I haven't had a chance to make the change yet.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/590#issuecomment-286787004


More information about the Freeipa-devel mailing list