opened
aldebaran30701
username_0
The postToNifi method I think we can change it to support SHA256 based on https://nifi.apache.org/docs/nifi-docs/components/org.apache.nifi/nifi-standard-nar/1.11.1/org.apache.nifi.processors.standard.HashContent/.

A global project search may need to be done to ensure other usages of HashContent.md5() are not hiding somewhere. 

Testing to ensure no behavioural changes are made and any subsequent methods which read from the hashed values still function as intended.

### Steps to Reproduce

N/A

**Expected behaviour:** 

**Actual behaviour:** 

**Reproduces how often:** 

### Additional Information

Although this change is a deprecation, it is useful to implement since the md5 is not cryptographically secure.
Update MD5 usages with Cryptographically secure SHA256
issue
created
cygnus-x-1
username_1
Do we care about the use of MD5 hashes for file comparisons in ```ConstellationLoggerHelper.createMD5Hash``` and ```WebServer.getDigest```?
comment
created
arcturus2
username_2
In the `ConstellationLoggerHelper` I need some sort of hashing algorithm, doesn't have to be MD5 if that's what the issue is. I don't have an opinion on the `WebServer.getDigest`.
comment
created
cygnus-x-1
username_1
```WebServer.getDigest``` seems to be doing essentially the same thing as ```ConstellationLoggerHelper.createMD5Hash``` - I want to know if we're happy with continuing to use MD5 hashes for file comparisons as it is _probably_ fine in terms of collisions and almost certainly faster.
comment
created
arcturus2
username_2
Thinking about it further, I'd vote to replace it with SHA256 so that we are more future proof and don't have to worry about when it is removed from the JDK.

Agreed in terms of speed it will be better to stay with MD5. For my use case I don't call `createMD5Hash` often so I can take the performance hit.
comment
created
cygnus-x-1
username_1
Okay, will do.
comment
closed
arcturus2
username_2
issue