exceptionfactory commented on pull request #5061:
URL: https://github.com/apache/nifi/pull/5061#issuecomment-867676830


   > > When attempting to verify the most recent updates at runtime, the 
`ListSFTP` worked as expected with the default blank values for the new 
properties. However, when specifying a value for `Ciphers Allowed`, NiFi marked 
the processor as invalid with the following message:
   > > `'Ciphers Allowed' validated against 'aes256-cbc' is invalid because 
'Ciphers Allowed' is not supported or has no Validator associated with it.`
   > > It appears that recent updates to remove the default values also removed 
any kind of validator, but I missed that detail last time around. Based on 
previous comments and support for Expression Language, it seems like using 
StandardValidators.NON_EMPTY_EL_VALIDATOR would be a good option.
   > 
   > Good catch but NON_EMPTY_EL_VALIDATOR ensures that EL is present. It's 
supported but not required Going with NON_BLANK_VALIDATOR.
   
   Thanks for the update, that sounds good, I will take another look at 
verifying the changes.


-- 
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]


Reply via email to