-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
perl: verify_SSL=>1
by default in HTTP::Tiny
#187480
Conversation
4078f27
to
5b70927
Compare
@GrahamcOfBorg build perl |
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.
💯
Some thoughts: It's likely that this will break stuff, like tests that connect to TLS servers with self signed certificates. I'll keep an eye on Hydra for this when it reaches staging-next. But, breakage from modules/tools that connect to (invalid) HTTPS resources on the internet will be harder to catch since our tests are network isolated. But breaking insecure HTTPS usage would be a good thing, imho. |
Description of changes
Perl's core
HTTP::Tiny
module does not verify TLS/SSL connections by default. This PR applies a proposed patch from Debian by @jmdh to fix that.Upstream has decided against changing the default due to backwards compatibility concerns.
So this is an opinionated patch which would break for users of
HTTP::Tiny
that rely on the insecure default.Example:
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes