[Freeipa-devel] [freeipa PR#667][comment] idrange-add: properly handle empty --dom-name option

stlaz freeipa-github-notification at redhat.com
Wed Apr 5 06:48:10 UTC 2017


  URL: https://github.com/freeipa/freeipa/pull/667
Title: #667: idrange-add: properly handle empty --dom-name option

stlaz commented:
"""
@flo-renaud That's completely OK :)
I thought we could probably add an assert to `CIDict.__contains__()` method since I realize the issue was somewhere else than this fixed check, but the current situation fails verbosely enough so that's probably fine.
ACK.
"""

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


More information about the Freeipa-devel mailing list