Skip to content

Allow IPv6 resolver for nginx #364

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

Closed
wants to merge 1 commit into from
Closed

Conversation

ramann
Copy link

@ramann ramann commented Jul 22, 2025

Check if DNS_SERVER contains colons (IPv6) and wrap in brackets for nginx

Check if DNS_SERVER contains colons (IPv6) and wrap in brackets for nginx
theseion added a commit to theseion/modsecurity-crs-docker that referenced this pull request Jul 23, 2025
- use all entries from /etc/resolv.conf
- support IPv6 addresses in /etc/resolv.conf
- make resolver options configurable
- use better variable name

Replaces coreruleset#364
@theseion
Copy link
Contributor

Thanks for this PR @ramann. It wouldn't have worked though for manually specified IPv6 addresses already in the correct format. Also, it would have broken with more than one address. I've created a PR that replaces this one and improves on the configuration of the resolver directive: #365.

@theseion theseion closed this Jul 23, 2025
theseion added a commit to theseion/modsecurity-crs-docker that referenced this pull request Jul 24, 2025
- use all entries from /etc/resolv.conf
- support IPv6 addresses in /etc/resolv.conf
- make resolver options configurable
- use better variable name

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

Successfully merging this pull request may close these issues.

2 participants