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

Get rid of Server.internalProxy #4870

Open
3 tasks done
ainar-g opened this issue Aug 29, 2022 · 0 comments
Open
3 tasks done

Get rid of Server.internalProxy #4870

ainar-g opened this issue Aug 29, 2022 · 0 comments

Comments

@ainar-g
Copy link
Contributor

ainar-g commented Aug 29, 2022

Prerequisites

  • I have checked the Wiki and Discussions and found no answer

  • I have searched other issues and found no duplicates

  • I want to request a feature or enhancement and not ask a question

Description

What problem are you trying to solve?

The internal DNS client, dnsforward.Server.internalProxy, is used for a few tasks, such as resolving IP addresses during an autoupdate or resolving hostnames for runtime clients. There are two issues with this:

  1. Not all upstream and similar settings are applied to it. See, for example, Honor the user-defined UpstreamMode for the internal DNS proxy #4403.

  2. These queries aren't logged into the query log.

Proposed solution

Remove it and use the same DNS client that we use for forwarding. Mark these queries as “internal” or something like that in the Query Log.

Alternatives considered

Configure the internal proxy the same way as the main one. This increases code duplication and complicates data flow.

@ameshkov, what do you think?

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

No branches or pull requests

1 participant