[SLING-5867] SlingRequestStatusHealthCheck should add timeout support

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: Health Check Support 1.0.4
• Component/s: Health Check
• Assignee: Georg Henzler
• Reporter: Konrad Windszus
• Created: 19/Jul/16 07:11
• Updated: 24/Oct/16 18:04

Description

Currently o.a.s.hc.support.impl.SlingRequestStatusHealthCheck just synchronously calls SlingRequestProcessor.processResponse.
That means in case of a non-returning response (e.g. caused by a deadlock like SLING-5847) the health check will just timeout but never actually really fail (even after a very long time).

In this case it is good to create a dedicated timeout handling within the SlingRequestStatusHealthCheck (separate from the timeout in HealthCheckExecutorImpl) because for each individual request health check configuration you might want to set different timeouts.