davebarnes97 commented on a change in pull request #5139:
URL: https://github.com/apache/geode/pull/5139#discussion_r428315433



##########
File path: 
geode-core/src/main/java/org/apache/geode/distributed/ConfigurationProperties.java
##########
@@ -2032,6 +2032,23 @@
    * Default: empty. All security components use basic (username/password) 
authentication
    */
   String SECURITY_AUTH_TOKEN_ENABLED_COMPONENTS = SECURITY_PREFIX + 
"auth-token-enabled-components";
+
+  /**
+   * The static String definition of the <i>"security-cn-auth-enabled"</i> 
property
+   * </p>
+   *
+   * <U>Description</U> This parameter only works if SSL with two-way 
handshake is enabled on
+   * all components and security manager is enabled. This property will 
determine if common name
+   * from client certificate will be used for authentication and authorization.

Review comment:
       Present tense would be clearer: 
   This property allows the common name from the client certificate to be used 
for authentication and authorization.




----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to