WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@Siomachkin
Copy link
Contributor

This PR adds support for a global resolvers directive in the Caddyfile to configure DNS resolvers for ACME DNS challenges at the global level, addressing issue #7192.

Changes

  • Added resolvers global option that accepts one or more DNS resolver addresses
  • Global resolvers are stored in the TLS app configuration
  • Resolvers are automatically applied to ACME issuers that use DNS challenges
  • Local resolvers directive in tls blocks can override global settings

Assistance Disclosure

I consulted Claude to understand the project architecture.

@mholt
Copy link
Member

mholt commented Oct 9, 2025

Thanks for the PR. I suppose there are many places we would need to apply the resolver config -- not just the TLS app. We might need to also apply this to, for example, reverse proxy, etc.

If this option is truly intended to be only for the TLS app, then maybe it should be renamed to something like tls_resolver or something.

@Gunni
Copy link

Gunni commented Nov 19, 2025

Still draft? I'd like this feature, but I mirror the above comment about calling it tls_resolvers or similar.

@Siomachkin Siomachkin marked this pull request as ready for review November 21, 2025 18:27
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.

3 participants