[SLING-9175] TeleporterHttpClient should set HttpURLConnection timeouts

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: JUnit Tests Teleporter 1.0.18
• Fix Version/s: JUnit Tests Teleporter 1.0.20
• Component/s: Testing
• Assignee: Bertrand Delacretaz
• Reporter: Bertrand Delacretaz
• Created: 05/Mar/20 15:35
• Updated: 23/Mar/20 11:51
• Resolved: 06/Mar/20 15:21

Description

The teleporter client uses HttpURLConnection in several places but doesn't set timeouts on those connections.

It should set both read + connect timeouts to avoid using the default values which as per https://stackoverflow.com/questions/45199702/httpurlconnection-timeout-defaults and https://bluxte.net/musings/2008/08/25/dont-forget-set-javaneturl-default-timeouts/ (haven't found official docs for that) is infinite.