[ 
https://issues.apache.org/jira/browse/SLING-5799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15352985#comment-15352985
 ] 

Georg Henzler commented on SLING-5799:
--------------------------------------

The requested paths are already logged when activating debug (hcDebug=true when 
using the health check servlet). If debug is not enabled, there is just no log 
output at all ATM. For that case, there should be at least one info line as 
overview of what was done (it is just good UX practice to give feedback on user 
actions, see use case \[1]).  Logging all paths on info is too much IMHO 
because potentially many can be configured.

\[1] 
https://cwiki.apache.org/confluence/display/SLING/Health+Checks+Executor+Design#HealthChecksExecutorDesign-B)Humanuser,webconsole

> Request Status HC does not log any INFO logs
> --------------------------------------------
>
>                 Key: SLING-5799
>                 URL: https://issues.apache.org/jira/browse/SLING-5799
>             Project: Sling
>          Issue Type: Improvement
>          Components: Health Check
>    Affects Versions: Health Check Support 1.0.4
>            Reporter: Georg Henzler
>
> For the case that all requests answer with response codes as expected, the 
> SlingRequestStatusHealthCheck (org.apache.sling.hc.support) is currently not 
> giving feedback in the logs (only the bare status itself). To improve this 
> the following line should be changed from logging debug to logging info: 
> https://github.com/apache/sling/blob/trunk/bundles/extensions/healthcheck/support/src/main/java/org/apache/sling/hc/support/impl/SlingRequestStatusHealthCheck.java#L144



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to