Skip to content

Remove deprecated AUTH_PASSWORD setting#54892

Merged
danhermann merged 5 commits intoelastic:masterfrom
danhermann:remove_auth_password_setting
Apr 20, 2020
Merged

Remove deprecated AUTH_PASSWORD setting#54892
danhermann merged 5 commits intoelastic:masterfrom
danhermann:remove_auth_password_setting

Conversation

@danhermann
Copy link
Copy Markdown
Contributor

@danhermann danhermann commented Apr 7, 2020

Removes the auth.password setting for monitoring HTTP exporters. Users must switch to the secure auth.secure_password setting.

Relates to #50919

Docs preview: http://elasticsearch_54892.docs-preview.app.elstc.co/diff

@danhermann danhermann added >docs General docs changes >breaking :Core/Infra/Monitoring DEPRECATED, DO NOT USE v8.0.0 labels Apr 7, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Monitoring)

@danhermann
Copy link
Copy Markdown
Contributor Author

@elasticmachine run elasticsearch-ci/docs

@lcawl
Copy link
Copy Markdown
Contributor

lcawl commented Apr 7, 2020

I think the auth.password needs to be removed from the example in https://www.elastic.co/guide/en/elasticsearch/reference/master/http-exporter.html too.

@danhermann
Copy link
Copy Markdown
Contributor Author

I think the auth.password needs to be removed from the example in https://www.elastic.co/guide/en/elasticsearch/reference/master/http-exporter.html too.

Thanks for catching that, @lcawl. I'll update that example.

@danhermann
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@danhermann
Copy link
Copy Markdown
Contributor Author

@elasticmachine run elasticsearch-ci/2

Copy link
Copy Markdown
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.

Documentation changes LGTM, thanks!

@danhermann
Copy link
Copy Markdown
Contributor Author

@elasticmachine update branch

@jakelandis jakelandis self-requested a review April 16, 2020 18:18
Copy link
Copy Markdown
Contributor

@jakelandis jakelandis left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>breaking :Core/Infra/Monitoring DEPRECATED, DO NOT USE >docs General docs changes v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants