Skip to content

feat: include stopped containers for creation/renewal#1211

Closed
ftsartek wants to merge 1 commit into
nginx-proxy:mainfrom
ftsartek:feat/stopped-containers
Closed

feat: include stopped containers for creation/renewal#1211
ftsartek wants to merge 1 commit into
nginx-proxy:mainfrom
ftsartek:feat/stopped-containers

Conversation

@ftsartek

Copy link
Copy Markdown

The intent here is to start a discussion on the viability of enabling certificate renewal for stopped containers; useful when you might want NGINX proxy to serve an 'unavailable' page or similar with a valid certificate.

Seemingly a simple change, though might need some test cases. Would you be open to including this or something similar?

@ftsartek ftsartek marked this pull request as draft September 17, 2025 01:42
@ftsartek ftsartek marked this pull request as ready for review September 25, 2025 02:25
Enables toggling the -include-stopped flag in docker-gen with the `INCLUDE_STOPPED` environment variable to enable certificate creation/renewal (and avoiding certificate/symlink removal) for stopped containers.
@buchdag buchdag force-pushed the feat/stopped-containers branch from cc11a1d to 7758a19 Compare September 28, 2025 07:29
@buchdag

buchdag commented Oct 1, 2025

Copy link
Copy Markdown
Member

@ftsartek this will be indirectly made possible by an upstream feature on docker-gen

See nginx-proxy/docker-gen#623 and nginx-proxy/docker-gen#700, once the later is merged (and a docker-gen version released then used in this project, etc) you'll just have to correctly set the DOCKER_CONTAINER_FILTERS environment variable on your acme-companion container for this to work.

@ftsartek

ftsartek commented Oct 1, 2025

Copy link
Copy Markdown
Author

Ah, that's awesome. Glad to hear it, so this is effectively redundant. Thank you!

@ftsartek ftsartek closed this Oct 1, 2025
@ftsartek ftsartek deleted the feat/stopped-containers branch March 30, 2026 20:37
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.

2 participants