Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve certutil --pass documentation about empty password #40137

Conversation

diegocsandrim
Copy link
Contributor

Improve the documentation of parameter --pass of elasticsearch-certutil. Closes #35285

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@javanna javanna added the >docs General docs changes label Mar 18, 2019
Co-Authored-By: diegocsandrim <diegocsandrim@users.noreply.github.com>
Copy link
Contributor

@bizybot bizybot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thank you.

@tvernum
Copy link
Contributor

tvernum commented Mar 27, 2019

LGTM.
@elasticmachine test this please

@tvernum tvernum merged commit dabc4c3 into elastic:master Mar 27, 2019
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Mar 28, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>

Backport of: elastic#40137
tvernum added a commit that referenced this pull request Mar 28, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Backport of: #40137

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Apr 8, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>

Backport of: elastic#40137
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Apr 8, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>

Backport of: elastic#40137
tvernum added a commit that referenced this pull request Apr 8, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Backport of: #40137

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>
tvernum added a commit that referenced this pull request Apr 8, 2019
Improve the documentation of parameter --pass of elasticsearch-certutil

Backport of: #40137

Co-Authored-By: Diego Cardozo Sandrim <diegocsandrim@users.noreply.github.com>
Co-Authored-By: Vigneash Sundar <vikene@users.noreply.github.com>
@jakelandis jakelandis added v6.8.2 and removed v6.8.1 labels Jun 17, 2019
@jpountz
Copy link
Contributor

jpountz commented Jul 5, 2019

@tvernum I'm assuming there is nothing left to backport and removed the backport pending label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :Security/Security Security issues without another label v6.8.2 v7.3.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation of elasticsearch-certutil is wrong for parameter --pass
7 participants