pcgrenier commented on issue #3604: NIFI-6255 NIFI-6287: Hash function for 
expression language and record…
URL: https://github.com/apache/nifi/pull/3604#issuecomment-515164310
 
 
   > I don't think the error has to explicitly list all the available options, 
but I think the docs should. The error can just reference the docs to let 
people know where the definitive list is.
   > 
   > The list of supported algorithms does not change frequently, and the docs 
can be updated in the event the list changes.
   > 
   > There are some examples of our current hash usage in 
[`HashService`](https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/security/util/crypto/HashService.java#L59)
 and 
[`HashAlgorithm`](https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/security/util/crypto/HashAlgorithm.java).
 I believe in the docs for `CryptographicHashContent` the possible values are 
enumerated even though it's a drop-down list in the processor configuration 
rather than a freeform text field.
   
   Ok, I added the list dynamically to the error message, also to the docs.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to