[GitHub] [pulsar] momo-jun commented on a diff in pull request #15172: [DOC]Add saslJaasServerRoleTokenSignerSecret parameter in security-kerberos.md

[GitBox <gi...@apache.org>]

momo-jun commented on code in PR #15172:
URL: https://github.com/apache/pulsar/pull/15172#discussion_r850449751


##########
site2/docs/reference-configuration.md:
##########
@@ -598,6 +598,7 @@ You can set the log level and configuration in the  [log4j2.yaml](https://github
 | tokenAudience | The token audience stands for this broker. The field `tokenAudienceClaim` of a valid token need contains this parameter.| |
 |saslJaasClientAllowedIds|This is a regexp, which limits the range of possible ids which can connect to the Broker using SASL. By default, it is set to `SaslConstants.JAAS_CLIENT_ALLOWED_IDS_DEFAULT`, which is ".*pulsar.*", so only clients whose id contains 'pulsar' are allowed to connect.|N/A|
 |saslJaasServerSectionName|Service Principal, for login context name. By default, it is set to `SaslConstants.JAAS_DEFAULT_BROKER_SECTION_NAME`, which is "Broker".|N/A|
+|saslJaasServerRoleTokenSignerSecret|Secret for SaslRoleTokenSigner. By default, it is set to `SaslConstants.JAAS_DEFAULT_ROLE_TOKEN_SIGNER_SECRET`, which is "PulsarSecret".|N/A|

Review Comment:
   Should the default value be put in the last column?



-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org