Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-756

Extend default health check for database connection

    XMLWordPrintable

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

        Attachments

          Activity

            People

              Unassigned Unassigned
              chrisD Christoph Damm
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Task DoD