mirror of
https://github.com/NginxProxyManager/nginx-proxy-manager.git
synced 2026-09-21 09:15:00 +01:00
When `nginx -t` fails, configure() is meant to move the broken config to <id>.conf.err so the failure can be inspected. renameConfigAsError() unlinked the source file before renaming it, so the rename always failed and the config was simply deleted. The deleteConfig() call after it then removed any .err file left over from an earlier failure. - unlink the destination .err file instead of the source - return the rename promise so the delete does not race it - pass delete_err_file = false so the new .err file survives - drop the stale 4th argument in the success path, which silently made delete_err_file false and left old .err files behind