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

Generate and store password hash for elastic user #76276

Merged
merged 13 commits into from
Aug 20, 2021

Conversation

jkakavas
Copy link
Member

For package installations (DEB,RPM), we will be generating a
random strong password for the elastic user on installation time
so that we can show it to the user.
We will subsequently hash and store this password to the
elasticsearch.keystore so that the node can pick it up on the first
run and use it to populate the relevant document for the elastic
user in the security index.
This change implements a class that can be called from the package
installation scripts and

  • Generates a strong password
  • Hashes it with the configured(default) password hashing algo
  • Stores it in the elasticsearch.keystore
  • prints it in the stdout

relates: #75310

For package installations (DEB,RPM), we will be generating a
random strong password for the elastic user on installation time
so that we can show it to the user.
We will subsequently hash and store this password to the
elasticsearch.keystore so that the node can pick it up on the first
run and use it to populate the relevant document for the elastic
user in the security index.
This change implements a class that can be called from the package
installation scripts and

- Generates a strong password
- Hashes it with the configured(default) password hashing algo
- Stores it in the elasticsearch.keystore
- prints it in the stdout
@jkakavas jkakavas added >enhancement :Security/Security Security issues without another label v8.0.0 labels Aug 10, 2021
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Aug 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

Copy link
Contributor

@albertzaharovits albertzaharovits left a comment

Choose a reason for hiding this comment

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

Only had a couple of minor things.
I don't think another round is warranted, LGTM.

Copy link
Contributor

@BigPandaToo BigPandaToo left a comment

Choose a reason for hiding this comment

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

LGTM

@jkakavas
Copy link
Member Author

@elasticmachine update branch

@jkakavas jkakavas merged commit 4c149e8 into elastic:master Aug 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :Security/Security Security issues without another label Team:Security Meta label for security team v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants