Skip to content

[DOCS] Add configurable password hashing docs #32849

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

Merged
merged 9 commits into from
Aug 21, 2018

Conversation

jkakavas
Copy link
Member

Adds documentation about the newly introduced configuration option
for setting the password hashing algorithm to be used for the users
cache and for storing credentials for the native and file realm.

Adds documentation about the newly introduced configuration option
for setting the password hashing algorithm to be used for the users
cache and for storing credentials for the native and file realm.
@jkakavas jkakavas added >docs General docs changes v7.0.0 :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) v6.4.0 v6.5.0 labels Aug 14, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@lcawl
Copy link
Contributor

lcawl commented Aug 16, 2018

I think it makes sense for the hash algorithm lists appear in the security settings page instead of the Stack Overview. I've made that change in 74048ff

@lcawl
Copy link
Contributor

lcawl commented Aug 16, 2018

The new password-hashing.asciidoc file isn't included in either the Stack Overview or the Elasticsearch Reference at this point. I actually recommend removing it. I put the gist of that information in a note in the configuring-native-realm.asciidoc and configuring-file-realm.asciidoc pages instead.

If you think the concept of password hashing should be covered in the Stack Overview after all, let me know and I can add that page to the table of contents, but as it is, I think the password-hashing.asciidoc file should just be deleted.

@jkakavas
Copy link
Member Author

I think the password-hashing.asciidoc file should just be deleted.

Agreed. I think everything is covered in security-hash-settings.asciidoc and your additions to the configuring-X-realm.asciidoc.

Thank you @lcawl

Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jaymode jaymode left a comment

Choose a reason for hiding this comment

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

LGTM

==== User cache and password hash algorithms

Certain realms store user credentials in memory. To limit exposure
to credential theft and mitigate credential compromise, cached user credentials
Copy link
Member

Choose a reason for hiding this comment

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

maybe To limit exposure to credential theft and mitigate credential compromise, the cache only stored a hashed version of the user credentials

Copy link
Member Author

@jkakavas jkakavas Aug 17, 2018

Choose a reason for hiding this comment

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

👍 Thanks, I like that better too, made relevant changes to configuring-{native,file}-realm.asciidoc

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.

LGTM

@jkakavas jkakavas merged commit 65d4f27 into elastic:master Aug 21, 2018
jkakavas added a commit that referenced this pull request Aug 21, 2018
* [DOCS] Add configurable password hashing docs

Adds documentation about the newly introduced configuration option
for setting the password hashing algorithm to be used for the users
cache and for storing credentials for the native and file realm.
jkakavas added a commit that referenced this pull request Aug 21, 2018
Adds documentation about the newly introduced configuration option
for setting the password hashing algorithm to be used for the users
cache and for storing credentials for the native and file realm.
@jkakavas jkakavas deleted the pwd-hashing-docs branch September 14, 2018 06:49
@colings86 colings86 removed the v7.0.0 label Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) v6.4.0 v6.5.0 v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants