Skip to content

Commit 50214d7

Browse files
authored
Merge pull request #157 from hsbt/reduce-to-load-cgi
Use cgi/escape or cgi/util instead of the all of CGI features
2 parents 73db98b + 9b6fbd7 commit 50214d7

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

lib/net/http/persistent.rb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
require 'net/http'
22
require 'uri'
3-
require 'cgi' # for escaping
3+
require 'cgi/escape'
4+
require 'cgi/util' unless defined?(CGI::EscapeExt)
45
require 'connection_pool'
56

67
begin
@@ -835,7 +836,7 @@ def proxy= proxy
835836
@proxy_connection_id = [nil, *@proxy_args].join ':'
836837

837838
if @proxy_uri.query then
838-
@no_proxy = CGI.parse(@proxy_uri.query)['no_proxy'].join(',').downcase.split(',').map { |x| x.strip }.reject { |x| x.empty? }
839+
@no_proxy = URI.decode_www_form(@proxy_uri.query).filter_map { |k, v| v if k == 'no_proxy' }.join(',').downcase.split(',').map { |x| x.strip }.reject { |x| x.empty? }
839840
end
840841
end
841842

0 commit comments

Comments
 (0)