exceptionfactory commented on a change in pull request #5044:
URL: https://github.com/apache/nifi/pull/5044#discussion_r632536283
##########
File path:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutSyslog.java
##########
@@ -150,6 +149,7 @@
"messages will be sent over a secure connection.")
.required(false)
.identifiesControllerService(SSLContextService.class)
+ .dependsOn(PROTOCOL, TCP_VALUE)
Review comment:
Good catch, I had intended to update both processors to make SSL Context
Service depend on TCP.
I noticed the issue with SSL Context Service and changing from TCP to UDP.
It seems like that is something that might make sense to address at the
framework level.
--
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]
