Jennifer88huang commented on a change in pull request #5038: [doc] Improve
Pulsar security-tls-authentication
URL: https://github.com/apache/pulsar/pull/5038#discussion_r317555790
##########
File path: site2/docs/security-tls-authentication.md
##########
@@ -44,10 +44,10 @@ $ openssl ca -config openssl.cnf -extensions usr_cert \
-in admin.csr.pem -out admin.cert.pem
```
-This will give you a cert, `admin.cert.pem`, and a key, `admin.key-pk8.pem`.
With `ca.cert.pem`, clients can used this cert and this key to authenticate
themselves to brokers and proxies as the role token ``admin``.
+This command gives you a cert, `admin.cert.pem`, and a key,
`admin.key-pk8.pem`. With `ca.cert.pem`, clients can used this cert and this
key to authenticate themselves to brokers and proxies as the role token
``admin``.
Review comment:
Use another word for "gives".
Reason: Don't attribute human qualities to software or hardware.
For guidelines, see https://developers.google.com/style/anthropomorphism
----------------------------------------------------------------
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
