Skip to content

Conversation

@spinscale
Copy link
Contributor

The keystore configuration options have not been mentioned yet, but have so in hipchat/jira/pagerduty actions.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

A couple comments.


In order to not store the account SMTP password as plaintext in the
elasticsearch yaml file, you should use the keystore instead (you can still
store the password in the elasticsearch configuration file, if you need to)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should have this node about putting the password in elasticsearch.yml. This should be going away; we shouldn't be promoting it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed that mention

--------------------------------------------------

In order to not store the account SMTP password as plaintext in the
elasticsearch yaml file, you should use the keystore instead (you can still
Copy link
Member

Choose a reason for hiding this comment

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

We should crosslink to the keystore docs ("secure settings").

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@rjernst rjernst removed the review label Oct 10, 2018
@colings86 colings86 added v6.6.0 and removed v6.5.0 labels Oct 25, 2018
Copy link
Member

@danielmitterdorfer danielmitterdorfer left a comment

Choose a reason for hiding this comment

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

LGTM

@spinscale spinscale merged commit 97259f0 into elastic:master Dec 6, 2018
spinscale added a commit that referenced this pull request Dec 6, 2018
Always refer to secure settings when setting up email account passwords
spinscale added a commit that referenced this pull request Dec 6, 2018
Always refer to secure settings when setting up email account passwords
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants