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

[ruby/net-http] No longer neccessary to call String#freeze on string literals.

Browse Source 
See #144

https://github.com/ruby/net-http/commit/5a986c13d3
This commit is contained in:
0x1eef 2023-09-15 03:35:34 -03:00 committed by git
parent 465c222ef1
commit e77c766b7a
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/net/http.rb
View File

@ -1798,7 +1798,7 @@ module Net #:nodoc:
def proxy_uri # :nodoc:
return if @proxy_uri == false
@proxy_uri ||= URI::HTTP.new(
"http".freeze, nil, address, port, nil, nil, nil, nil, nil
"http", nil, address, port, nil, nil, nil, nil, nil
).find_proxy || false
@proxy_uri || nil
end

8
lib/net/http/generic_request.rb
View File

@ -23,7 +23,7 @@ class Net::HTTPGenericRequest
raise ArgumentError, "no host component for URI" unless (hostname && hostname.length > 0)
@uri = uri_or_path.dup
host = @uri.hostname.dup
host << ":".freeze << @uri.port.to_s if @uri.port != @uri.default_port
host << ":" << @uri.port.to_s if @uri.port != @uri.default_port
@path = uri_or_path.request_uri
raise ArgumentError, "no HTTP request path given" unless @path
else
@ -212,15 +212,15 @@ class Net::HTTPGenericRequest
return unless @uri
if ssl
scheme = 'https'.freeze
scheme = 'https'
klass = URI::HTTPS
else
scheme = 'http'.freeze
scheme = 'http'
klass = URI::HTTP
end
if host = self['host']
host.sub!(/:.*/m, ''.freeze)
host.sub!(/:.*/m, '')
elsif host = @uri.host
else
host = addr