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

MartinBasti freeipa-github-notification at redhat.com
Wed Mar 15 13:59:10 UTC 2017


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

MartinBasti commented:
"""
I have no context about how exactly certrequest is supposed to work, but IMO it was done in that way to allow dynamically adding more helpers as plugins, that's why it is Str and not SrEnum, but code doesn't look it may support that.

@LiptonB do you remember why Str param was used?

@Akasurde Right now there is no backward compatibility because 4.5 will be first release that contains this feature.
"""

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


More information about the Freeipa-devel mailing list