-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
fix(inputs.http_response): fix the use of hostnames + interface #15596
Conversation
00d77be
to
fa18413
Compare
Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip. 👍 This pull request doesn't change the Telegraf binary size 📦 Click here to get additional PR build artifactsArtifact URLs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Mic92 - can you please file an issue with some background as to why this change needs to occur and justify the TODO as well please?
// TODO: Resolving the address to bind is better done at gather time rather than setup time. | ||
// Then we could ask the system resolver to resolve the address with the scope of the interface | ||
// and it would also gives us the correct address for the interface to use. | ||
// Unfortunally the portable libc interfaces does not allow us to do that. | ||
// macOS, systemd-resolved on Linux and Windows have different ways to do that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really do not like TODOs in code like this. This also seems outside the scope of the issue at hand?
Hello! I am closing this issue due to inactivity. I hope you were able to resolve your problem, if not please try posting this question in our Community Slack or Community Forums or provide additional details in this issue and reqeust that it be re-opened. Thank you! |
Hi, Would be happy to circle around to this PR once you file an issue and remove the TODO. Thanks! |
2 weeks is a bit short for a timeout, i.e. if you go on holiday. |
Which is why we are perfectly fine re-opening this PR if you came back around to it or if you communicated you are out ;) |
I don't have permissions to re-open a PR and all previous reviews are missing in the new pull request. |
Summary
Checklist
Related issues
resolves