You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2019/05/27 09:40:15 UTC

[GitHub] [pulsar] jiazhai commented on a change in pull request #4385: Update pulsar kerberos document

jiazhai commented on a change in pull request #4385: Update pulsar kerberos document
URL: https://github.com/apache/pulsar/pull/4385#discussion_r287720986
 
 

 ##########
 File path: site2/docs/security-kerberos.md
 ##########
 @@ -107,6 +107,7 @@ It is also a choice to have 2 separate JAAS configuration files: the file for br
  - Set `authenticationProviders` to choose `AuthenticationProviderSasl`;
  - Set `saslJaasClientAllowedIds` regex for principal that is allowed to connect to broker. 
  - Set `saslJaasBrokerSectionName` that corresponding to the section in JAAS configuration file for broker.
+ - Set `superUserRoles` that corresponding to the name registered in kdc.
 
 Review comment:
   This need set when `authorizationEnabled` is set to true.
   How about adding it at around line 270, which is related to 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:
users@infra.apache.org


With regards,
Apache Git Services