Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upLink to article outlining CA creation #2401
Conversation
This comment has been minimized.
This comment has been minimized.
elasticsearch-release
commented
Aug 28, 2016
|
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run. |
1 similar comment
This comment has been minimized.
This comment has been minimized.
elasticsearch-release
commented
Aug 28, 2016
|
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run. |
ruflin
added
docs
review
labels
Aug 30, 2016
This comment has been minimized.
This comment has been minimized.
|
@dedemorton Can you have a look at this? |
ruflin
reviewed
Aug 30, 2016
| @@ -17,7 +17,7 @@ To use TLS mutual authentication: | |||
|
|
|||
| . Create a certificate authority (CA) and use it to sign the certificates that you plan to use for | |||
| {beatname_uc} and Logstash. Creating a correct SSL/TLS infrastructure is outside the scope of this | |||
| document. There are many online resources available that describe how to create certificates. | |||
| document. There are many online resources available that describe how to create certificates including https://www.elastic.co/guide/en/shield/2.3/certificate-authority.html[this section of the documentation for Shield]. | |||
This comment has been minimized.
This comment has been minimized.
ruflin
Aug 30, 2016
Collaborator
We should use here if possible an internal reference instead of a link to make sure it always links to the same version. @dedemorton Is that possible across projects?
This comment has been minimized.
This comment has been minimized.
dedemorton
Aug 30, 2016
Contributor
@ruflin Sure, we can use variables defined in the index.asciidoc files just like we do for other guides. If you want to merge this, I can add the variables.
This comment has been minimized.
This comment has been minimized.
|
LGTM |
mpchadwick commentedAug 28, 2016
No description provided.