1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

remote_fetcher.rb: fix leaked FDs

Browse Source 
* lib/rubygems/remote_fetcher.rb (request): close connections.

* lib/rubygems/request.rb (close): ditto.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@46944 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
nobu 2014-07-25 07:55:42 +00:00
parent 895c1ce477
commit a818740553
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/rubygems/remote_fetcher.rb
View File

@ -337,6 +337,8 @@ class Gem::RemoteFetcher
request.fetch do |req| request.fetch do |req|
yield req if block_given? yield req if block_given?
end end
ensure
request.close if request
end end
def https?(uri) def https?(uri)

6
lib/rubygems/request.rb
View File

@ -28,6 +28,12 @@ class Gem::Request
@env_no_proxy = get_no_proxy_from_env @env_no_proxy = get_no_proxy_from_env
end end
def close
@connections.each_value do |conn|
conn.finish
end
end
def add_rubygems_trusted_certs(store) def add_rubygems_trusted_certs(store)
pattern = File.expand_path("./ssl_certs/*.pem", File.dirname(__FILE__)) pattern = File.expand_path("./ssl_certs/*.pem", File.dirname(__FILE__))
Dir.glob(pattern).each do |ssl_cert_file| Dir.glob(pattern).each do |ssl_cert_file|