turcsanyip commented on a change in pull request #4733:
URL: https://github.com/apache/nifi/pull/4733#discussion_r551455310
##########
File path:
nifi-nar-bundles/nifi-grpc-bundle/nifi-grpc-processors/src/main/java/org/apache/nifi/processors/grpc/ListenGRPC.java
##########
@@ -85,18 +89,20 @@
.build();
public static final PropertyDescriptor PROP_USE_SECURE = new
PropertyDescriptor.Builder()
.name("Use TLS")
- .displayName("Use TLS")
- .description("Whether or not to use TLS to send the contents of
the gRPC messages.")
+ .displayName("Use SSL/TLS")
+ .description("Whether or not to use SSL/TLS to receive the
contents of the gRPC messages.")
Review comment:
Done.
My original intent was to keep some kind of consistency with the name of the
'SSL Context Service' property.
But it is better to go ahead and use TLS where possible.
----------------------------------------------------------------
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]