Skip to content

Example Test for DNS is not helpful #401

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

Open
lcrilly opened this issue Apr 14, 2025 · 2 comments
Open

Example Test for DNS is not helpful #401

lcrilly opened this issue Apr 14, 2025 · 2 comments
Assignees
Labels
product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com)

Comments

@lcrilly
Copy link

lcrilly commented Apr 14, 2025

Describe the bug

On this page, https://docs.nginx.com/nginx/admin-guide/load-balancer/udp-health-check/#example-test-for-dns
The send and expect parameters are confusing and misleading. They cannot (and should not) be used to create a working DNS health check.

To reproduce

Steps to reproduce the behavior:

  1. Configure NGINX Plus with the example match configuration.
  2. Watch all health checks fail
  3. Notice that NGINX Plus is requesting a DNS lookup for *stlu.msl.edu which is bad for the servers in the upstream group and bad for any subsequent DNS server that will handle this request. And it's bad for Massachusetts School of Law.

Expected behavior

  1. Docs should explain that DNS health checks require configuration to be made on the DNS server.
  2. Docs should provide a means for creating the hex string to send a DNS lookup
  3. Docs should explain what the expect value should be based on 1 & 2
@mjang
Copy link
Contributor

mjang commented Apr 14, 2025

I think https://docs.nginx.com/nginx/admin-guide/load-balancer/udp-health-check/ is most relevant. That page suggests that send and receive can be used to create a working DNS health check

which we now know is wrong.

@lcrilly
Copy link
Author

lcrilly commented Apr 15, 2025

Apologies for not adding the URI. You're right @mjang , I am reffering to https://docs.nginx.com/nginx/admin-guide/load-balancer/udp-health-check/#example-test-for-dns (will edit the issue).

@y82 y82 self-assigned this Apr 16, 2025
@y82 y82 added the product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com) label Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product/nginx-plus NGINX Plus (and NGINX Open Source in docs.ngninx.com)
Projects
None yet
Development

No branches or pull requests

3 participants