Skip to content

Fix nginx reload after certificate renewal#5720

Open
DoyoDia wants to merge 1 commit into
NginxProxyManager:developfrom
DoyoDia:fix/reload-nginx-after-certificate-renewal
Open

Fix nginx reload after certificate renewal#5720
DoyoDia wants to merge 1 commit into
NginxProxyManager:developfrom
DoyoDia:fix/reload-nginx-after-certificate-renewal

Conversation

@DoyoDia

@DoyoDia DoyoDia commented Jul 19, 2026

Copy link
Copy Markdown

Why

A successful Let's Encrypt renewal updates the certificate files on disk, but nginx continues serving the certificate already loaded in memory until it is reloaded.

The shared certificate renewal path did not reload nginx after Certbot completed. This affects both HTTP and DNS challenge renewals, including Cloudflare.

This change calls the existing nginx reload function immediately after a successful renewal, allowing the renewed certificate to be served without restarting the container or manually reloading nginx.

This restores behavior that was lost when auto-renewal was changed to use the shared renewal function in #3392.

Fixes #3803

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • API changes
  • Performance improvement
  • Test addition or update

AI Usage

  • AI was used to write this
  • AI was used to review this

@nginxproxymanagerci

Copy link
Copy Markdown

Docker Image for build 1 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-5720

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

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.

SSL certificates expired

1 participant