Net::HTTPResponse has bad nil handling in read_body and stream_check

[BrianHawley]

I was getting a weird error from Net::HTTPResponse, and noticed that a couple methods had some bad nil checking.

In read_body, it sets @body = nil and then later on calls string methods on it like @body.force_encoding. Right after you set @read = true you should add this code: return if @body.nil?.

Then, the weird error was from stream_check:

undefined method `closed?' for nil:NilClass (NoMethodError)

    raise IOError, 'attempt to read body out of block' if @socket.closed?

Since you set @socket = nil in reading_body, the stream_check method should be:

def stream_check
  raise IOError, 'attempt to read body out of block' if @socket.nil? || @socket.closed?
end

[kalashnikovisme]

Having this issue too. Waiting for PR to be merged or another solution.

Using

gem 'net-http', git: 'https://github.com/BrianHawley/net-http', branch: 'fixes_70'

[BrianHawley]

Here it is in patch form, for those who prefer that:

# frozen_string_literal: true

require "net/http"

begin
  response = Net::HTTPResponse.new("", "", "")
  response.body
rescue IOError
  # We're good
rescue
  # Bad exception
  Net::HTTPResponse.class_exec do
    private

    def stream_check
      raise IOError, "attempt to read body out of block" if @socket.nil? || @socket.closed?
    end
  end
end