Skip to content

Conversation

@kpmueller
Copy link

  • this PR will set a connection timeout, in addition to the read timeout that is already done. (by breaking out socket.connect explicitly)

I didn't have another great way of doing it other than a nested try-with-resources, but I think it's a pretty standard way of doing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant