Fix Docker Healthcheck

[benbz]

Checklist

• Tests written for new code (and old code if feasible).
• New or updated public/exported symbols have accurate TSDoc documentation.
• Linter and other CI checks pass.
• I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

Fixes up the healthcheck added in #29351.

The wget in container is provided by busybox and only provides the following options:

$ docker run -it --rm --entrypoint sh vectorim/element-web:v1.11.95              
/ $ wget
BusyBox v1.37.0 (2025-01-17 18:12:01 UTC) multi-call binary.

Usage: wget [-cqS] [--spider] [-O FILE] [-o LOGFILE] [--header STR]
	[--post-data STR | --post-file FILE] [-Y on/off]
	[-P DIR] [-U AGENT] [-T SEC] URL...

Retrieve files via HTTP or FTP

	--spider	Only check URL existence: $? is 0 if exists
	--header STR	Add STR (of form 'header: value') to headers
	--post-data STR	Send STR using POST method
	--post-file FILE	Send FILE using POST method
	-c		Continue retrieval of aborted transfer
	-q		Quiet
	-P DIR		Save to DIR (default .)
	-S    		Show server response
	-T SEC		Network read timeout is SEC seconds
	-O FILE		Save to FILE ('-' for stdout)
	-o LOGFILE	Log messages to FILE
	-U STR		Use STR for User-Agent header
	-Y on/off	Use proxy
/ $ 

The test didn't catch this as State.Running is true even if the healthcheck is failing, e.g:

...
        "State": {
            "Status": "running",
            "Running": true,
            "Paused": false,
            "Restarting": false,
            "OOMKilled": false,
            "Dead": false,
            "Pid": 747376,
            "ExitCode": 0,
            "Error": "",
            "StartedAt": "2025-03-11T15:35:11.270785924Z",
            "FinishedAt": "0001-01-01T00:00:00Z",
            "Health": {
                "Status": "unhealthy",
                "FailingStreak": 35,
                "Log": [
                    {
                        "Start": "2025-03-11T15:50:18.41101394Z",
                        "End": "2025-03-11T15:50:18.465344516Z",
                        "ExitCode": 1,
                        "Output": "wget: unrecognized option: retry-connrefused\nBusyBox v1.37.0 (2025-01-17 18:12:01 UTC) multi-call binary.\n\nUsage: wget [-cqS] [--spider] [-O FILE] [-o LOGFILE] [--header STR]\n\t[--post-data STR | --post-file FILE] [-Y on/off]\n\t[-P DIR] [-U AGENT] [-T SEC] URL...\n\nRetrieve files via HTTP or FTP\n\n\t--spider\tOnly check URL existence: $? is 0 if exists\n\t--header STR\tAdd STR (of form 'header: value') to headers\n\t--post-data STR\tSend STR using POST method\n\t--post-file FILE\tSend FILE using POST method\n\t-c\t\tContinue retrieval of aborted transfer\n\t-q\t\tQuiet\n\t-P DIR\t\tSave to DIR (default .)\n\t-S    \t\tShow server response\n\t-T SEC\t\tNetwork read timeout is SEC seconds\n\t-O FILE\t\tSave to FILE ('-' for stdout)\n\t-o LOGFILE\tLog messages to FILE\n\t-U STR\t\tUse STR for User-Agent header\n\t-Y on/off\tUse proxy\n"
                    },
...

Fixes #29472

[litetex]

I would appreciate if this could be hotfixed in a new release because currently loadbalancers may refuse to route traffic to the container as the healthcheck is failing...

Totally didn't happen to me and I'm totally not going to disable the healthcheck for now :D

[Domoel]

A hotfix new release would be appreciated indeed! :)

[Aterfax]

Temporary solution if using compose:

    healthcheck:
      test: ["CMD", "sh", "-c", "wget -q -O- http://localhost:80/config.json"]
      start_period: 5s
      interval: 30s
      retries: 3
      timeout: 5s

You may need to amend the port.

Surprised no hotfix is out? (yet?)