Jennifer88huang commented on a change in pull request #5069: [doc] Improve
Pulsar Security Kerberos
URL: https://github.com/apache/pulsar/pull/5069#discussion_r319383635
##########
File path: site2/docs/security-kerberos.md
##########
@@ -346,14 +342,14 @@ superUserRoles=client/{clientIp}@EXAMPLE.COM
## Regarding authentication between ZooKeeper and Broker
-Pulsar Broker acts as a Kerberos client when authenticating with Zookeeper.
According to [ZooKeeper
document](https://cwiki.apache.org/confluence/display/ZOOKEEPER/Client-Server+mutual+authentication),
you need these settings in `conf/zookeeper.conf`:
+Pulsar Broker acts as a Kerberos client when you are authenticating with
Zookeeper. According to [ZooKeeper
document](https://cwiki.apache.org/confluence/display/ZOOKEEPER/Client-Server+mutual+authentication),
you need these settings in `conf/zookeeper.conf`:
```
authProvider.1=org.apache.zookeeper.server.auth.SASLAuthenticationProvider
requireClientAuthScheme=sasl
```
-And add a section of `Client` configurations in the file `pulsar_jaas.conf`,
which is used by Pulsar Broker:
+And enter the following commands to add a section of `Client` configurations
in the file `pulsar_jaas.conf`, which Pulsar Broker uses:
Review comment:
Delete "And" at the beginning.
----------------------------------------------------------------
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
With regards,
Apache Git Services
