Skip to content

docs(deployment): add restart policy to nginx service in SSL docker-compose example#216

Open
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-nginx-restart-policy-20260625
Open

docs(deployment): add restart policy to nginx service in SSL docker-compose example#216
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-nginx-restart-policy-20260625

Conversation

@andrerfneves

Copy link
Copy Markdown
Owner

Summary

Adds a restart: unless-stopped policy to the nginx reverse proxy service in the SSL/TLS production docker-compose example in DEPLOYMENT_DOCKER.md.

Why

The nginx service was missing a restart policy, meaning Docker would not automatically restart it if the container crashes or the host reboots. This is inconsistent with:

  • The lightning-decoder service in the same example (which already has restart: unless-stopped)
  • The simple docker-compose example (which also has restart: unless-stopped)
  • Production best practices for critical infrastructure components

Changes

  • DEPLOYMENT_DOCKER.md: Added restart: unless-stopped to the nginx service definition in the SSL/TLS production docker-compose example

Test Plan

  • No behavior change — documentation-only fix
  • YAML syntax is valid
  • Consistent with the restart policy used by the lightning-decoder service in the same example

…ompose example

The nginx reverse proxy service in the production SSL/TLS docker-compose
example was missing a restart policy. Without it, Docker will not
automatically restart the container if it crashes or the host reboots.

Added `restart: unless-stopped` for consistency with the
lightning-decoder service in the same example and the best practice
pattern used throughout the document.
@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lightning-decoder Ready Ready Preview, Comment Jun 25, 2026 12:13pm

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.

1 participant