Jennifer88huang commented on a change in pull request #5038: [doc] Improve
Pulsar security-tls-authentication
URL: https://github.com/apache/pulsar/pull/5038#discussion_r317531180
##########
File path: site2/docs/security-tls-authentication.md
##########
@@ -26,27 +27,27 @@ $ openssl pkcs8 -topk8 -inform PEM -outform PEM \
-in admin.key.pem -out admin.key-pk8.pem -nocrypt
```
-Generate the certificate request. When asked for a **common name**, enter the
**role token** which you want this key pair to authenticate a client as.
+Next, generate the certificate request. When you are asked for a **common
name**, enter the **role token** that you want this key pair to authenticate a
client as.
```bash
$ openssl req -config openssl.cnf \
-key admin.key.pem -new -sha256 -out admin.csr.pem
```
> Note
-> If there is no openssl.cnf, please read [Certificate
authority](http://pulsar.apache.org/docs/en/security-tls-transport/#certificate-authority)
to get the openssl.cnf.
+> If openssl.cnf is not given, please read [Certificate
authority](http://pulsar.apache.org/docs/en/security-tls-transport/#certificate-authority)
to get the openssl.cnf.
Review comment:
1. "given"--> "specified"
2. do not use "please" in technical writing.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
