Skip to content

Commit 28bad53

Browse files
committed
Disable read timeout for Poseidon
A timeout of five seconds might be too short for some actions (i.e., synchronizing environments), since these actions are performed synchronously on Poseidon (which is good!). Hence, we need to improve the timeout handling and first byte sent in future to re-enable the timeout here. Fixes CODEOCEAN-188 Fixes CODEOCEAN-18F
1 parent e74a727 commit 28bad53

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/runner/strategy/poseidon.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ def self.headers
312312
def self.http_connection
313313
@http_connection ||= Faraday.new(ssl: {ca_file: config[:ca_file]}, headers:) do |faraday|
314314
faraday.options[:open_timeout] = 5
315-
faraday.options[:timeout] = 5
315+
faraday.options[:timeout] = nil # No timeout. Poseidon performs some actions (e.g., sync environment) synchronously.
316316

317317
faraday.adapter :net_http_persistent
318318
end
@@ -321,7 +321,7 @@ def self.http_connection
321321
def self.new_http_connection
322322
Faraday.new(ssl: {ca_file: config[:ca_file]}, headers:) do |faraday|
323323
faraday.options[:open_timeout] = 5
324-
faraday.options[:timeout] = 5
324+
faraday.options[:timeout] = nil # No timeout. Poseidon performs some actions (e.g., sync environment) synchronously
325325

326326
faraday.adapter :net_http
327327
end

0 commit comments

Comments
 (0)