Details
-
Improvement
-
Resolution: Not an issue
-
Neutral
-
None
-
None
-
None
-
None
Description
The current status/healthcheck endpoints only checks for
MgnlContext.hasInstance() && !this.moduleManager.getStatus().needsUpdateOrInstall() ? Response.ok("{}").build() : Response.status(Status.SERVICE_UNAVAILABLE).build();
However my assumption is that checking for the context itself does not include a check whether the JCR/Database is reachable at all, e.g. by doing a query against the configuration workspace (which should always be there).
Of course we can add this via a custom java endpoint ourself and override the above implementation, but wouldn't it be good to add it as default?
The extended HC module could also be used, however for DX only.
Checklists
Acceptance criteria