Celluloid Timeout Support

[aronchick]

I know that current Celluloid Timeouts are currently not supported, but is there an interim workaround?

Here's the code I'd like to run:

  def fetch(url, options = {} )
    puts "Request -> #{url}"
    begin
      options = options.merge({ socket_class: Celluloid::IO::TCPSocket,
                                timeout_class: HTTP::Timeout::Global,
                                timeout_options: {
                                  connect_timeout: 1,
                                  read_timeout: 1,
                                  write_timeout: 1
                                  }
                              })
      HTTP.get(url, options)
    rescue HTTP::TimeoutError => e
      [do more stuff]
    rescue Exception => e
      puts e
    end
  end

Its goal is to test a server as being live and healthy.

[zanker]

Set options = options.merge(socket_class: Celluloid::IO::TCPSocket, timeout_class: HTTP::Timeout::Null) should work, as it bypasses the timeout code.

[aronchick]

Interesting! Can we call that with different values?

On Tue, May 12, 2015 at 12:04 PM, Zachary Anker notifications@github.com
wrote:

Set options = options.merge(socket_class: Celluloid::IO::TCPSocket,
timeout_class: HTTP::Timeout::Null) should work, as it bypasses the
timeout code.

—
Reply to this email directly or view it on GitHub
#225 (comment).

[zanker]

Which part with different values? Not quite sure what you mean.

[aronchick]

Sorry, I was hoping to have the ability to set a timeout at 1s for some
requests, and 20s for other requests. Is that variable?

On Tue, May 12, 2015 at 2:35 PM, Zachary Anker notifications@github.com
wrote:

Which part with different values? Not quite sure what you mean.

—
Reply to this email directly or view it on GitHub
#225 (comment).

[zanker]

If you aren't using Celluloid IO, you can change it to whatever you want, but we don't support timeouts when using Celluloid IO right now.

[aronchick]

I guess that's my issue - I'm using Celluloid IO - is there an alternative
for Timeouts?

On Tue, May 12, 2015 at 2:40 PM, Zachary Anker notifications@github.com
wrote:

If you aren't using Celluloid IO, you can change it to whatever you want,
but we don't support timeouts when using Celluloid IO right now.

—
Reply to this email directly or view it on GitHub
#225 (comment).

[zanker]

I'm not familiar enough with Celluloid IO to offer a suggestion, sorry.

[tarcieri]

Closing this. Please continue discussion in #272