[Enhancement]: Make it possible to set ``responseTimeout``

[AB-xdev]

Module

Core

Proposal

In

testcontainers-java/core/src/main/java/org/testcontainers/dockerclient/DockerClientProviderStrategy.java

Lines 395 to 398 in 946b6aa

	new ZerodepDockerHttpClient.Builder()
	.dockerHost(transportConfig.getDockerHost())
	.sslConfig(transportConfig.getSslConfig())
	.build();

there is currently no option to set an overall responseTimeout.

This can result in hanging threads when there is a problem with Docker's API and no response is ever returned. Although this is rare, we observed such a problem recently in xdev-software/tci#370

All other timeouts are configured in the default Apache HTTP client:

It would be nice to somehow have an option to set the overall timeout (e.g. 10 min) to prevent situations like this.

[dbunner2]

Hi all. I am happy to look at this one. My approach is to add an opt in client.response.timeout config prop (sibling to client.ping.timeout) and leave it unset by default. I'd wire it into the ZerodepDockerHttpClient.Builder inside getClientForConfig. I can keep the change scoped to only response.timeout for now unless anyone feels differently. Just let me know.

Thanks!
David