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.

Sign up for GitHub

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

Jump to bottom

[feature request]Hide elasticsearch username and password from log #1980

Closed
wei-hai opened this issue Nov 1, 2016 · 0 comments · Fixed by #2186
Closed

[feature request]Hide elasticsearch username and password from log #1980

wei-hai opened this issue Nov 1, 2016 · 0 comments · Fixed by #2186
Assignees
@sparrc
Labels
bug unexpected problem or unintended behavior
Milestone
1.2.0

Comments

@wei-hai
Copy link

wei-hai commented Nov 1, 2016

Feature Request

Proposal:

Hide elasticsearch username and password from log

Current behavior:

given servers = ["https://admin:password@localhost:9200"]
The log includes username and password in elasticsearch URL like:
2016/11/01 13:53:09 ERROR in input [elasticsearch]: Errors encountered: [Get https://admin:password@localhost:9200/_nodes/_local/stats: remote error: handshake failure]

Desired behavior:

Hide username and password in the log file
@sparrc sparrc added the bug unexpected problem or unintended behavior label Nov 2, 2016
@sparrc sparrc added this to the 1.2.0 milestone Nov 2, 2016
sparrc added a commit that referenced this issue Dec 20, 2016
@sparrc
Mask username/password from error messages
7ea9e47 
closes #1980
sparrc added a commit that referenced this issue Dec 20, 2016
@sparrc
Mask username/password from error messages
a9f03a7 
closes #1980
@sparrc sparrc mentioned this issue Dec 20, 2016
Merged
3 tasks
njwhite pushed a commit to njwhite/telegraf that referenced this issue Jan 31, 2017
@sparrc
Mask username/password from error messages
90829b7 
closes influxdata#1980
maxunt pushed a commit that referenced this issue Jun 26, 2018
@sparrc
Mask username/password from error messages
8311ff9 
closes #1980
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sparrc sparrc

Labels
bug unexpected problem or unintended behavior
Projects
None yet
Milestone
1.2.0
Development

Successfully merging a pull request may close this issue.

Mask username/password from error messages influxdata/telegraf
3 participants
@wei-hai @sparrc @121watts