-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
fish: Use /bin/hostname on darwin. Fixes #11012. #11691
Conversation
By analyzing the blame information on this pull request, we identified @samuelrivas, @bjornfor and @ocharles to be potential reviewers |
I don't have any real thoughts on this - it looks like it makes |
fish: Use /bin/hostname on darwin. Fixes #11012.
It comes from the cc @pikajude |
Ok, cool! Thanks! |
prometheus 2.40.x is out of support since 12/2022 therefore the update to 2.41.0 between those two releases security critical patches have been added both to prometheus codebase and upstream dependecies, see: * 2.40.6 / 2022-12-09 prometheus/prometheus#11690 [SECURITY] Security upgrade from go and upstream dependencies that include security fixes to the net/http and os packages. NixOS#11691 * 2.40.4 / 2022-11-29 GHSA-7rg2-cxvp-9p7p GHSA-4v48-4q5m-8vx4 [SECURITY] Fix basic authentication bypass vulnerability (CVE-2022-46146) GHSA-4v48-4q5m-8vx4
prometheus 2.40.x is out of support since 12/2022 therefore the update to 2.41.0 between those two releases security critical patches have been added both to prometheus codebase and upstream dependecies, see: * 2.40.6 / 2022-12-09 prometheus/prometheus#11690 [SECURITY] Security upgrade from go and upstream dependencies that include security fixes to the net/http and os packages. #11691 * 2.40.4 / 2022-11-29 GHSA-7rg2-cxvp-9p7p GHSA-4v48-4q5m-8vx4 [SECURITY] Fix basic authentication bypass vulnerability (CVE-2022-46146) GHSA-4v48-4q5m-8vx4 (cherry picked from commit bfac961)
nettools
are unavailable for Darwin and I don't know of the alternatives. So falling back to usinghostname
from the environment (as it was prior to #10841).If there's a nix package with
hostname
for Darwin, please let me know, I'll fix this PR to use it instead.