Mechanize doesn't raise Mechanize::ResponseCodeError if error happened before getting the response about mechanize HOT 3 CLOSED

@adrianodennanni Thanks for opening this issue, this is a good conversation to have.

The exception class Mechanize::ResponseCodeError is intended to wrap up a successful response from the server that has an HTTP status code other than 200, 301, or 302.

In the case you're describing, are you saying that you received a response from the server that was a 50x status code? Or was this a networking error? I'm a little confused by the string "(500, 501, 502, 503)" in the description.

from mechanize.

I will paste here an example of this happening.

The exception message for the Net::HTTPFatalError was 503 "Service Unavailable".

A simple get, such as agent.get 'https://example.com' raised me Net::HTTPFatalError, with the following trace:

503 "Service Unavailable" (Net::HTTPFatalError)
/usr/local/lib/ruby/3.2.0/net/http/response.rb:258:in `error!'
/usr/local/lib/ruby/3.2.0/net/http/response.rb:267:in `value'
/usr/local/lib/ruby/3.2.0/net/http.rb:1293:in `connect'
/usr/local/lib/ruby/3.2.0/net/http.rb:1248:in `do_start'
/usr/local/lib/ruby/3.2.0/net/http.rb:1243:in `start'
/usr/local/bundle/gems/net-http-persistent-4.0.1/lib/net/http/persistent.rb:655:in `start'
/usr/local/bundle/gems/net-http-persistent-4.0.1/lib/net/http/persistent.rb:595:in `connection_for'
/usr/local/bundle/gems/net-http-persistent-4.0.1/lib/net/http/persistent.rb:885:in `request'
/usr/local/bundle/gems/mechanize-2.8.5/lib/mechanize/http/agent.rb:284:in `fetch'
/usr/local/bundle/gems/mechanize-2.8.5/lib/mechanize.rb:465:in `get'
/app/domain/my_example_script.rb:35:in `get_page'

In other occasions, the same agent.get line raised me a normal Mechanize::ResponseCodeError. I believe this case happened when response was completed, while the the previously mentioned exception happened during the connection phase of the fetch.

from mechanize.

@adrianodennanni Again, ResponseCodeError is only intended to be used when a response has been successfully read, and the status code is surprising.

An exception related to networking should not be rescued by Mechanize -- in this case, the exception raised by Net::HTTP is what is raised. I think this is behaving as intended.

I'm happy to continue the conversation if you like -- I'm going to convert it to a Discussion.

from mechanize.