diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-09-13 04:57:25 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-09-13 04:57:25 +0000 |
commit | 80d8216415c3e42aab8237ef0b5696b488ecf387 (patch) | |
tree | 69abdec16e017f367f8c5506d6411e35459ccc39 /lib/uri | |
parent | 0c252b43adb5a4aae03464783947c234652925c0 (diff) | |
download | ruby-80d8216415c3e42aab8237ef0b5696b488ecf387.tar.gz |
* lib/uri/generic.rb (URI::Generic.find_proxy): return nil if
http_proxy environment variable is empty string.
[ruby-core:57140] [Bug #8898]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@42930 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'lib/uri')
-rw-r--r-- | lib/uri/generic.rb | 32 |
1 files changed, 15 insertions, 17 deletions
diff --git a/lib/uri/generic.rb b/lib/uri/generic.rb index 387682b5b5..b1195fdf0b 100644 --- a/lib/uri/generic.rb +++ b/lib/uri/generic.rb @@ -1647,31 +1647,29 @@ module URI proxy_uri = ENV[name] || ENV[name.upcase] end - if proxy_uri && self.hostname + if proxy_uri.nil? || proxy_uri.empty? + return nil + end + + if self.hostname require 'socket' begin addr = IPSocket.getaddress(self.hostname) - proxy_uri = nil if /\A127\.|\A::1\z/ =~ addr + return nil if /\A127\.|\A::1\z/ =~ addr rescue SocketError end end - if proxy_uri - proxy_uri = URI.parse(proxy_uri) - name = 'no_proxy' - if no_proxy = ENV[name] || ENV[name.upcase] - no_proxy.scan(/([^:,]*)(?::(\d+))?/) {|host, port| - if /(\A|\.)#{Regexp.quote host}\z/i =~ self.host && - (!port || self.port == port.to_i) - proxy_uri = nil - break - end - } - end - proxy_uri - else - nil + name = 'no_proxy' + if no_proxy = ENV[name] || ENV[name.upcase] + no_proxy.scan(/([^:,]*)(?::(\d+))?/) {|host, port| + if /(\A|\.)#{Regexp.quote host}\z/i =~ self.host && + (!port || self.port == port.to_i) + return nil + end + } end + URI.parse(proxy_uri) end end end |