Skip to content

Fix Docker runtime healthcheck by including wget in final image#136

Merged
cbulock merged 2 commits into
mainfrom
copilot/fix-docker-healthcheck-wget-missing
May 24, 2026
Merged

Fix Docker runtime healthcheck by including wget in final image#136
cbulock merged 2 commits into
mainfrom
copilot/fix-docker-healthcheck-wget-missing

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 22, 2026

Docker marked iptv-proxy unhealthy even when the app was running because the healthcheck script invoked wget, which was not present in the runtime image. This PR resolves the dependency mismatch without changing healthcheck behavior.

  • Runtime image dependency alignment

    • Updated the final-stage package install in Dockerfile to include wget alongside ffmpeg.
    • Keeps the existing /usr/src/app/healthcheck.sh contract intact (wget --spider http://localhost:34400/health).
  • Scope

    • No changes to application logic, routes, or healthcheck script behavior.
    • Fix is limited to container runtime packaging.
RUN apt-get update && \
    apt-get install -y --no-install-recommends ffmpeg wget && \
    rm -rf /var/lib/apt/lists/*

Copilot AI changed the title [WIP] Fix Docker healthcheck failure due to missing wget Fix Docker runtime healthcheck by including wget in final image May 22, 2026
Copilot AI requested a review from cbulock May 22, 2026 17:50
Copy link
Copy Markdown
Owner

@cbulock cbulock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No code-level issues found.

The change directly fixes the observed runtime failure: the container healthcheck calls wget, and the final image now installs wget alongside ffmpeg while preserving the existing healthcheck script and endpoint behavior.

One validation caveat before merging: the CI run for the current head SHA is currently action_required, so the Docker build/CI has not completed yet. I’d let CI run before merging.

@cbulock cbulock marked this pull request as ready for review May 22, 2026 20:19
Copilot AI review requested due to automatic review settings May 22, 2026 20:19
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a runtime/container packaging issue where the Docker healthcheck script depends on wget, but wget was not installed in the final production image—causing the container to be marked unhealthy even when the app was running.

Changes:

  • Install wget alongside ffmpeg in the final Docker image stage to satisfy /usr/src/app/healthcheck.sh.
  • Keep the existing healthcheck script and health endpoint contract unchanged.

@cbulock cbulock merged commit 6fe2912 into main May 24, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker healthcheck fails because wget is missing from image

3 participants