Open LePresidente opened 2 years ago
Ok this is done can you redo the build since 0.1.11 was just released, my testing now shows the memory usage back to normal levels of 100mb-200mb for my 7 Vhosts
This will also require a change in the main project for the config files as well https://github.com/NginxProxyManager/nginx-proxy-manager/pull/1875
Before:
After:
Can't pull the above due to permissions but i have built a docker image my side, With the following two PR's if you want to test
https://github.com/NginxProxyManager/docker-nginx-full/pull/8 https://github.com/NginxProxyManager/nginx-proxy-manager/pull/1875
https://hub.docker.com/r/lepresidente/nginxproxymanager:latest
I have been using this for the past month with no issues
Hello, what is the status on this PR?
My understanding is that the feature request was merged-and-reverted due to a memory leak, but the memory leak has since been fixed based on op's description in this PR.
fixed merge conflicts
@jc21 Any update on Readding this and going forward with this change?
Is there any additional testing (or otherwise) I can possibly help with in order to see this PR merged by chance?
Docker Image for build 223 is available on DockerHub as:
nginxproxymanager/nginx-full:github-pr-8
nginxproxymanager/nginx-full:github-pr-8-certbot
nginxproxymanager/nginx-full:github-pr-8-certbot-node
nginxproxymanager/nginx-full:github-pr-8-acmesh
nginxproxymanager/nginx-full:github-pr-8-acmesh-golang
Up
Re add the crowdsec nginx bouncer
Requires upstream change first https://github.com/crowdsecurity/cs-openresty-bouncer/pull/28, will bump the version once its been released and remove the ONHOLD from the title.