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

Remove double password and username in the Redis input config opt… #6662

Merged
merged 3 commits into from
Mar 27, 2018

Conversation

ph
Copy link
Contributor

@ph ph commented Mar 26, 2018

…ions

  • Remove a double password in the yml file.
  • Remove mention of username in the configuration, nothing in the input
    code uses thats.
  • Remove space

Fixes: #6573

…ions

- Remove a double password in the yml file.
- Remove mention of username in the configuration, nothing in the input
code uses thats.
- Remove space

Fixes: elastic#6573
@ph ph added docs Filebeat Filebeat labels Mar 26, 2018
@kvch
Copy link
Contributor

kvch commented Mar 26, 2018

Could you please add a changelog entry?

@ph
Copy link
Contributor Author

ph commented Mar 26, 2018

@kvch good catch, in the changelog I only mentioned the removal of the username option, everything else will be for the clarification of the documentation.

@kvch
Copy link
Contributor

kvch commented Mar 26, 2018

jenkins test it

@ruflin ruflin merged commit 1f8d072 into elastic:master Mar 27, 2018
@ruflin ruflin added the review label Mar 27, 2018
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.

3 participants