exceptionfactory commented on a change in pull request #4990:
URL: https://github.com/apache/nifi/pull/4990#discussion_r611696557
##########
File path:
nifi-nar-bundles/nifi-redis-bundle/nifi-redis-extensions/src/main/java/org/apache/nifi/redis/state/RedisStateProvider.java
##########
@@ -106,6 +122,16 @@ public final void initialize(final
StateProviderInitializationContext context) t
+ " is configured in clustered mode, and this
service requires a non-clustered Redis")
.build());
}
+ final boolean enableTls =
validationContext.getProperty(ENABLE_TLS).asBoolean();
+ if (enableTls && sslContext == null) {
+ results.add(new ValidationResult.Builder()
+ .subject(ENABLE_TLS.getDisplayName())
+ .valid(false)
+ .explanation(ENABLE_TLS.getDisplayName()
+ + " is set to 'true', but Apache NiFi is not
secured. This state provider can only use a TLS-enabled connection " +
+ "if a keystore and truststore is provided in
nifi.properties.")
Review comment:
Minor grammatical adjustment:
```suggestion
"if a keystore and truststore are provided in
nifi.properties.")
```
--
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]
