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

The default context should have GETDNS_RESOLUTION_STUB #516

Open
Philip-NLnetLabs opened this issue Apr 20, 2022 · 0 comments
Open

The default context should have GETDNS_RESOLUTION_STUB #516

Philip-NLnetLabs opened this issue Apr 20, 2022 · 0 comments

Comments

@Philip-NLnetLabs
Copy link

Current the spec (Section 8·3, Contexts for Basic Resolution) requires contexts to default to GETDNS_RESOLUTION_RECURSING.

This is confusing because it means that by default, getdns will ignore the system's local recursive resolvers. This means that, for example, local zones configured on recursive resolvers will be unavailable, queries go out over the internet by default. And some networks block access to the root DNS servers.

Possible solutions: update the documentation that with the introduction of getdns_context_create, the need for GETDNS_RESOLUTION_STUB is discussed immediately. And change the examples to include GETDNS_RESOLUTION_STUB as well.

Longer term, either an incompatible change to the API specification. Or, to deprecate getdns_context_create in favor of a new function (getdns_context_create2) that has a different default.

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

No branches or pull requests

1 participant