jdeppe-pivotal commented on a change in pull request #6844:
URL: https://github.com/apache/geode/pull/6844#discussion_r705556458
##########
File path:
geode-apis-compatible-with-redis/src/main/java/org/apache/geode/redis/internal/netty/NettyRedisServer.java
##########
@@ -86,18 +85,20 @@
private final Channel serverChannel;
private final int serverPort;
private final DistributedMember member;
+ private final SecurityManager securityManager;
public NettyRedisServer(Supplier<DistributionConfig> configSupplier,
RegionProvider regionProvider, PubSub pubsub, Supplier<Boolean>
allowUnsupportedSupplier,
Runnable shutdownInvoker, int port, String requestedAddress, RedisStats
redisStats,
- DistributedMember member) {
+ DistributedMember member, SecurityManager securityManager) {
Review comment:
Done
##########
File path:
geode-core/src/main/java/org/apache/geode/distributed/internal/AbstractDistributionConfig.java
##########
@@ -1317,7 +1317,7 @@ static Class _getAttributeType(String attName) {
"Specifies the address on which the Redis API for Geode is listening.
If set to the empty string or this property is not specified, localhost is
requested from the operating system.");
m.put(REDIS_ENABLED,
"When the default value of false, the Redis API for Geode is not
available. Set to true to enable the Redis API for Geode.");
- m.put(REDIS_PASSWORD,
+ m.put(REDIS_USERNAME,
"Specifies the password that the server uses when a client attempts to
authenticate. The default is none and no authentication will be required.");
Review comment:
Done
##########
File path:
geode-core/src/main/java/org/apache/geode/distributed/internal/DistributionConfig.java
##########
@@ -3515,24 +3515,24 @@ static InetAddress _getDefaultMcastAddress() {
boolean DEFAULT_REDIS_ENABLED = false;
/**
- * Returns the value of the {@link ConfigurationProperties#REDIS_PASSWORD}
property
+ * Returns the value of the {@link ConfigurationProperties#REDIS_USERNAME}
property
* <p>
* Returns the value of the
- * {@link ConfigurationProperties#REDIS_PASSWORD} property
+ * {@link ConfigurationProperties#REDIS_USERNAME} property
*
* @return the authentication password for GeodeRedisServer
Review comment:
Done
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
