You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by GitBox <gi...@apache.org> on 2021/02/22 18:47:08 UTC

[GitHub] [nifi] thenatog commented on pull request #4788: NIFI-8132 Replaced framework uses of MD5 with SHA-256

thenatog commented on pull request #4788:
URL: https://github.com/apache/nifi/pull/4788#issuecomment-783590319


   This is a +1 from me. I think that the benefits of moving away from MD5 outweigh potential risks of people using parts of NiFi framework that aren't considered the official API. Hopefully it would also be a small change for them to correct if they are using these specific methods. From what I can tell from the code, comments above, and my own testing, we don't foresee any issues that could arise from the actual change of the hashing function away from MD5. Once we have this in main we'll have more people using/testing NiFi before a release which can help identify issues, if any.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org