URL: https://github.com/freeipa/freeipa/pull/677
Title: #677: cert: defer cert-find result post-processing
HonzaCholasta commented:
"""
master:
* eb6d4c3037d0cc269a7924745f1cbd8f647e6e1a cert: defer cert-find result
post-processing
ipa-4-5:
* 49f9d799c171c7ae2ac546a33a353c2c40b4719c cert:
URL: https://github.com/freeipa/freeipa/pull/677
Title: #677: cert: defer cert-find result post-processing
HonzaCholasta commented:
"""
That might require backporting [issue
6564](https://pagure.io/freeipa/issue/6564) as well.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/677
Title: #677: cert: defer cert-find result post-processing
stlaz commented:
"""
We may need these changes in 4.5 and 4.4, too since `cert-find` is rather
broken there, too.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/677
Title: #677: cert: defer cert-find result post-processing
stlaz commented:
"""
The patched IPA works better than the current 4.4 and 4.5 branches in terms of
options logic, that's good.
From the code I am not sure which searches we do miss,
URL: https://github.com/freeipa/freeipa/pull/677
Title: #677: cert: defer cert-find result post-processing
stlaz commented:
"""
What worries me the most is that the tests are green even though this is
potentially a serious problem.
"""
See the full comment at
