URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
tomaskrizek commented:
"""
ACK, there is no disagreement on the freeipa-devel.
I'm already working on replacing SHA1 with SHA256 given the [recent
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
stlaz commented:
"""
I don't have a problem adding it in this PR, it needs to be resolved according
to the outcome of the freeipa-devel thread anyway.
> On 21 Feb 2017, at 18:54, Tomas Krizek
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
tomaskrizek commented:
"""
Btw, I think sha256 can be added in a separate PR. Let's just wait if there are
any concerns about removing md5 on the freeipa-devel.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
MartinBasti commented:
"""
+1 for sha256
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/482#issuecomment-281352849
--
Manage your subscription for the Freeipa-devel mailing
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
tomaskrizek commented:
"""
@stlaz I think it'd be good to discuss this change on freeipa-devel. Also,
since we're removing md5, I'd consider adding sha256.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
stlaz commented:
"""
@rcritten thanks for noticing the discrepancy in the previous version of the
commit, it was a leftover from previous implementation.
I reworked the commit to remove MD5