Reviewed 1 of 1 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @mangalaman93, @martinmr, and @sleto-it)

README.md, line 100 at r1 (raw file):

Creating a Secure Client using TLS

Add a link to the docs about TLS setup. Is adding a trust manager only needed for self-signed certs?

after fixing Daniel's comment.

Reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @mangalaman93 and @sleto-it)

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @danielmai, @martinmr, and @sleto-it)

README.md, line 100 at r1 (raw file):

Done.

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @danielmai, @mangalaman93, @martinmr, and @sleto-it)

README.md, line 108 at r3 (raw file):

```java
SslContextBuilder builder = GrpcSslContexts.forClient();
builder.trustManager(new File("<path to ca.crt>"));

You might want to mention. The CA.crt is the CA that signed the "server" certificate. If the server certificate is self-signed, then the server's certificate goes in here.

Reviewable status: 0 of 1 files reviewed, 2 unresolved discussions (waiting on @danielmai, @martinmr, @parasssh, and @sleto-it)

README.md, line 108 at r3 (raw file):

The way instructions are provided in the https://docs.dgraph.io/deploy/#tls-configuration, we generally use self signed certificate. We don't have a way to get the certificate signed from an external CA yet.

LGTM