Skip to content
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.

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

NPM passes through some requests even though 444 option is set #3704

Open
ericswpark opened this issue Apr 16, 2024 · 2 comments
Open

NPM passes through some requests even though 444 option is set #3704

ericswpark opened this issue Apr 16, 2024 · 2 comments
Labels

Comments

@ericswpark
Copy link

Checklist

  • Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
    • Yes
  • Are you sure you're not using someone else's docker image?
    • Yes
  • Have you searched for similar issues (both open and closed)?
    • Yes

Describe the bug

When running NPM with the default page option set to 444 (No Response), some requests still make it to proxied hosts, causing errors because the wrong HTTP_HOST is set (the server IP instead of the proxy hostname).

Nginx Proxy Manager Version

v2.11.1

To Reproduce
Steps to reproduce the behavior:

  1. Set up a proxy host
  2. Set default server behavior to 444
  3. Send requests to the server IP

Expected behavior

None of the requests should make it through to the proxied host

Additional context

This issue appears to be sporadic as I'm unable to reproduce it on-demand. However, I get errors from the proxied app behind NPM, telling me that requests are coming through with the HTTP_HOST header set to the server's IP address instead of the correct hostname I set while setting up the proxy host in NPM.

@ericswpark ericswpark added the bug label Apr 16, 2024
@h4de5
Copy link

h4de5 commented May 2, 2024

are those requests maybe related to static assets like .jpg, .css, .js or .woff files?

if so, there is a location rule in a include/assets.conf that takes precedence here.

@ericswpark
Copy link
Author

@h4de5 nope, root (/) with no files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants