diff options
author | Michael Mair-Keimberger <m.mairkeimberger@gmail.com> | 2018-05-17 10:21:13 +0200 |
---|---|---|
committer | Aaron Bauman <bman@gentoo.org> | 2018-05-18 08:32:58 -0400 |
commit | 2ee33026ec8924ade78e58987141042d34805a83 (patch) | |
tree | ad14b8faaafba15993d797915eb160da1238a4bd /app-admin/puppet | |
parent | app-emacs/moccur-edit: use HTTPS (diff) | |
download | gentoo-2ee33026ec8924ade78e58987141042d34805a83.tar.gz gentoo-2ee33026ec8924ade78e58987141042d34805a83.tar.bz2 gentoo-2ee33026ec8924ade78e58987141042d34805a83.zip |
app-admin/puppet: remove unused patch
Closes: https://github.com/gentoo/gentoo/pull/8445
Diffstat (limited to 'app-admin/puppet')
-rw-r--r-- | app-admin/puppet/files/puppet-4-webrick-ruby.patch | 53 |
1 files changed, 0 insertions, 53 deletions
diff --git a/app-admin/puppet/files/puppet-4-webrick-ruby.patch b/app-admin/puppet/files/puppet-4-webrick-ruby.patch deleted file mode 100644 index ce4b055644f6..000000000000 --- a/app-admin/puppet/files/puppet-4-webrick-ruby.patch +++ /dev/null @@ -1,53 +0,0 @@ -From a358d5ec01c6f87b39e5bfbab7a35582624fdbf7 Mon Sep 17 00:00:00 2001 -From: Josh Cooper <josh@puppet.com> -Date: Fri, 12 Jan 2018 15:30:38 -0800 -Subject: [PATCH] (PUP-8297) Don't accept already connected socket - -Webrick's start method yields an already connected socket, so it is not -necessary to call accept on the socket. In ruby versions prior to 2.3.6 and -2.4.3, the accept call was a noop. - -In 2.3.6 and 2.4.3, it is an error, because ruby those versions use a -non-blocking accept to handle clients that make an TCP connection, but -are slow to negotiate the TLS handshake[1,2]. On these versions, puppet -master would report: - - SSL_accept returned=1 errno=0 state=unknown state: unexpected record - -Remove the call to Socket#accept. - -[1] https://bugs.ruby-lang.org/issues/14005 -[2] https://github.com/ruby/ruby/commit/2e728d51e70ed3756ad760c687a08b8487b0112f ---- - lib/puppet/network/http/webrick.rb | 1 - - spec/unit/network/http/webrick_spec.rb | 5 +++++ - 2 files changed, 5 insertions(+), 1 deletion(-) - -diff --git a/lib/puppet/network/http/webrick.rb b/lib/puppet/network/http/webrick.rb -index 9ebf1763be5..3ac2ff40650 100644 ---- a/lib/puppet/network/http/webrick.rb -+++ b/lib/puppet/network/http/webrick.rb -@@ -29,7 +29,6 @@ def listen(address, port) - if ! IO.select([sock],nil,nil,timeout) - raise "Client did not send data within %.1f seconds of connecting" % timeout - end -- sock.accept - @server.run(sock) - end - end -diff --git a/spec/unit/network/http/webrick_spec.rb b/spec/unit/network/http/webrick_spec.rb -index a90fc3f3378..843b0d54c8a 100644 ---- a/spec/unit/network/http/webrick_spec.rb -+++ b/spec/unit/network/http/webrick_spec.rb -@@ -92,6 +92,11 @@ - expect(server).to be_listening - end - -+ it "is passed an already connected socket" do -+ socket.expects(:accept).never -+ server.listen(address, port) -+ end -+ - describe "when the REST protocol is requested" do - it "should register the REST handler at /" do - # We don't care about the options here. |