This project is closed and read-only.
Backport #7831 ยป net.http.user_handled_content_encoding.patch
| lib/net/http/generic_request.rb (working copy) | ||
|---|---|---|
|
raise ArgumentError, "HTTP request path is empty" if path.empty?
|
||
|
@path = path
|
||
|
@decode_content = false
|
||
|
if @response_has_body and Net::HTTP::HAVE_ZLIB then
|
||
|
if !initheader ||
|
||
|
!initheader.keys.any? { |k|
|
||
|
%w[accept-encoding range].include? k.downcase
|
||
|
} then
|
||
|
@decode_content = true
|
||
|
initheader = initheader ? initheader.dup : {}
|
||
|
initheader["accept-encoding"] =
|
||
|
"gzip;q=1.0,deflate;q=0.6,identity;q=0.3"
|
||
| ... | ... | |
|
attr_reader :path
|
||
|
attr_reader :uri
|
||
|
# Automatically set to false if the user sets the Accept-Encoding header.
|
||
|
# This indicates they wish to handle Content-encoding in responses
|
||
|
# themselves.
|
||
|
attr_reader :decode_content
|
||
|
def inspect
|
||
|
"\#<#{self.class} #{@method}>"
|
||
|
end
|
||
|
##
|
||
|
# Don't automatically decode response content-encoding if the user indicates
|
||
|
# they want to handle it.
|
||
|
def []=(key, val) # :nodoc:
|
||
|
@decode_content = false if key.downcase == 'accept-encoding'
|
||
|
super key, val
|
||
|
end
|
||
|
def request_body_permitted?
|
||
|
@request_has_body
|
||
|
end
|
||
| ... | ... | |
|
if IO.select([sock.io], nil, nil, sock.continue_timeout)
|
||
|
res = Net::HTTPResponse.read_new(sock)
|
||
|
unless res.kind_of?(Net::HTTPContinue)
|
||
|
res.decode_content = @decode_content
|
||
|
throw :response, res
|
||
|
end
|
||
|
end
|
||
| lib/net/http/response.rb (working copy) | ||
|---|---|---|
|
@body = nil
|
||
|
@read = false
|
||
|
@uri = nil
|
||
|
@decode_content = false
|
||
|
end
|
||
|
# The HTTP version supported by the server.
|
||
| ... | ... | |
|
# if a URI was used to create the request.
|
||
|
attr_reader :uri
|
||
|
# Set to true automatically when the request did not contain an
|
||
|
# Accept-Encoding header from the user.
|
||
|
attr_accessor :decode_content
|
||
|
def inspect
|
||
|
"#<#{self.class} #{@code} #{@message} readbody=#{@read}>"
|
||
|
end
|
||
| ... | ... | |
|
def inflater # :nodoc:
|
||
|
return yield @socket unless Net::HTTP::HAVE_ZLIB
|
||
|
return yield @socket unless @decode_content
|
||
|
return yield @socket if self['content-range']
|
||
|
case self['content-encoding']
|
||
| lib/net/http.rb (working copy) | ||
|---|---|---|
|
req.exec @socket, @curr_http_version, edit_path(req.path)
|
||
|
begin
|
||
|
res = HTTPResponse.read_new(@socket)
|
||
|
res.decode_content = req.decode_content
|
||
|
end while res.kind_of?(HTTPContinue)
|
||
|
res.uri = req.uri
|
||
| test/net/http/test_http.rb (working copy) | ||
|---|---|---|
|
def test_request
|
||
|
start {|http|
|
||
|
_test_request__GET http
|
||
|
_test_request__accept_encoding http
|
||
|
_test_request__file http
|
||
|
# _test_request__range http # WEBrick does not support Range: header.
|
||
|
_test_request__HEAD http
|
||
| ... | ... | |
|
end
|
||
|
assert_equal $test_net_http_data.size, res.body.size
|
||
|
assert_equal $test_net_http_data, res.body
|
||
|
assert res.decode_content, 'Bug #7831'
|
||
|
}
|
||
|
end
|
||
|
def _test_request__accept_encoding(http)
|
||
|
req = Net::HTTP::Get.new('/', 'accept-encoding' => 'deflate')
|
||
|
http.request(req) {|res|
|
||
|
assert_kind_of Net::HTTPResponse, res
|
||
|
assert_kind_of String, res.body
|
||
|
unless self.is_a?(TestNetHTTP_v1_2_chunked)
|
||
|
assert_not_nil res['content-length']
|
||
|
assert_equal $test_net_http_data.size, res['content-length'].to_i
|
||
|
end
|
||
|
assert_equal $test_net_http_data.size, res.body.size
|
||
|
assert_equal $test_net_http_data, res.body
|
||
|
refute res.decode_content, 'Bug #7831'
|
||
|
}
|
||
|
end
|
||
| test/net/http/test_http_request.rb (working copy) | ||
|---|---|---|
|
assert_equal expected, req.to_hash
|
||
|
end
|
||
|
def test_initialize_accept_encoding
|
||
|
req1 = Net::HTTP::Get.new '/'
|
||
|
assert req1.decode_content, 'Bug #7831 - automatically decode content'
|
||
|
req2 = Net::HTTP::Get.new '/', 'accept-encoding' => 'identity'
|
||
|
refute req2.decode_content,
|
||
|
'Bug #7381 - do not decode content if the user overrides'
|
||
|
end
|
||
|
def test_header_set
|
||
|
req = Net::HTTP::Get.new '/'
|
||
|
assert req.decode_content, 'Bug #7831 - automatically decode content'
|
||
|
req['accept-encoding'] = 'identity'
|
||
|
refute req.decode_content,
|
||
|
'Bug #7831 - do not decode content if the user overrides'
|
||
|
end
|
||
|
end
|
||
| test/net/http/test_httpresponse.rb (working copy) | ||
|---|---|---|
|
EOS
|
||
|
res = Net::HTTPResponse.read_new(io)
|
||
|
res.decode_content = true
|
||
|
body = nil
|
||
| ... | ... | |
|
EOS
|
||
|
res = Net::HTTPResponse.read_new(io)
|
||
|
res.decode_content = true
|
||
|
body = nil
|
||
| ... | ... | |
|
end
|
||
|
end
|
||
|
def test_read_body_content_encoding_deflate_disabled
|
||
|
io = dummy_io(<<EOS)
|
||
|
HTTP/1.1 200 OK
|
||
|
Connection: close
|
||
|
Content-Encoding: deflate
|
||
|
Content-Length: 13
|
||
|
x\x9C\xCBH\xCD\xC9\xC9\a\x00\x06,\x02\x15
|
||
|
EOS
|
||
|
res = Net::HTTPResponse.read_new(io)
|
||
|
res.decode_content = false # user set accept-encoding in request
|
||
|
body = nil
|
||
|
res.reading_body io, true do
|
||
|
body = res.read_body
|
||
|
end
|
||
|
assert_equal 'deflate', res['content-encoding'], 'Bug #7831'
|
||
|
assert_equal "x\x9C\xCBH\xCD\xC9\xC9\a\x00\x06,\x02\x15", body, 'Bug #7381'
|
||
|
end
|
||
|
def test_read_body_content_encoding_deflate_no_length
|
||
|
io = dummy_io(<<EOS)
|
||
|
HTTP/1.1 200 OK
|
||
| ... | ... | |
|
EOS
|
||
|
res = Net::HTTPResponse.read_new(io)
|
||
|
res.decode_content = true
|
||
|
body = nil
|
||