Skip to content

Commit

Permalink
clarification on mTLS verification with curl (#925)
Browse files Browse the repository at this point in the history
* clarification on mTLS verification with curl

from some confusion by user on slack

* update ptal
  • Loading branch information
ldemailly authored Feb 3, 2018
1 parent 80aeade commit 5b4b284
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions _docs/tasks/security/mutual-tls.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,8 @@ There are several steps:
```
```bash
...
* error fetching CN from cert:The requested data were not available.
...
< HTTP/1.1 200 OK
< content-type: text/html; charset=utf-8
< content-length: 1867
Expand All @@ -111,11 +113,14 @@ Note that Istio uses [Kubernetes service account](https://kubernetes.io/docs/tas
as service identity, which offers stronger security than service name
(refer [here]({{home}}/docs/concepts/security/mutual-tls.html#identity) for more information).
Thus the certificates used in Istio do not have service name, which is the information that curl needs to verify
server identity. As a result, we use curl option '-k' to prevent the curl client from verifying service identity
in server's (i.e., productpage) certificate.
server identity. As a result, we use curl option '-k' to prevent the curl client from aborting when failing to
find and verify the server name (i.e., productpage.ns.svc.cluster.local) in the certificate provided by the server.

Please check secure naming [here]({{home}}/docs/concepts/security/mutual-tls.html#workflow) for more information
about how the client verifies the server's identity in Istio.

What we are demonstrating and verifying above is that the server accepts the connection from the client. Try not giving the client `--key` and `--cert` and observe you are not allowed to connect and you do not get an HTTP 200.

## Further reading

* Learn more about the design principles behind Istio's automatic mTLS authentication
Expand Down

0 comments on commit 5b4b284

Please sign in to comment.