New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Possible multiple X-Forwarded-For headers #2373

Merged

jc21 merged 1 commit into NginxProxyManager:develop from lakkeri:develop

Nov 8, 2022

Contributor

lakkeri commented Nov 5, 2022

NMP behind another reverse proxy can multiply X-Forwarded-For headers. $proxy_add_x_forwarded_for equals to $remote_addr if this header not present in client request https://nginx.org/en/docs/http/ngx_http_proxy_module.html#var_proxy_add_x_forwarded_for


          Possible multiple X-Forwarded-For headers

052cb8f

NMP behind another reverse proxy can multiply X-Forwarded-For headers. $proxy_add_x_forwarded_for equals to $remote_addr if this header not present in client request 
https://nginx.org/en/docs/http/ngx_http_proxy_module.html#var_proxy_add_x_forwarded_for

nginxproxymanagerci bot commented Nov 5, 2022

This is an automated message from CI:

Docker Image for build 1 is available on DockerHub as jc21/nginx-proxy-manager:github-pr-2373

Note: ensure you backup your NPM instance before testing this PR image! Especially if this PR contains database changes.

jc21 merged commit b62b6b5 into NginxProxyManager:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet