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
Hide passwords from advancedsettings log #10790
Conversation
jenkins build this please |
I can find one more place that could benefit from this. See httpproxypassword in http://kodi.wiki/view/advancedsettings.xml#network Confimed working in linux. |
a809e45
to
ad733d9
Compare
I missed that. thanks. Updated. |
jenkins build this please |
@Montellese ping |
jenkins build this please |
weird.. android build couldn't git the commit.. |
jenkins build this please |
Hide passwords when we dump advancedsettings in the log.
Motivation and Context
Right now we dump passwords from advancedsettings in our log
How Has This Been Tested?
I tested it with my advancedsettings file
Types of change
Checklist: