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

feature: Allow unencrypted DOH #1276

Closed

Conversation

krombel
Copy link
Contributor

@krombel krombel commented Dec 19, 2019

This is a first step to implement #1009.
This adds the config option

tls:
  allow_clear_doh: true

If allow_clear_doh is true (defaults to false), AdGuardHome allows serving DoH queries from bind_host:bind_port - so the normal http interface of AdGuardHome.

  • Note
    If you want to pass the remote ip to AdGuardHome, you can setup nginx as follows:
        location /dns-query {
              proxy_set_header Host $http_host;
              proxy_set_header X-Real-IP $remote_addr;
              proxy_redirect off;
              proxy_buffering off;
              proxy_pass http://127.0.0.1:3000;
        }

@ameshkov ameshkov requested a review from szolin December 19, 2019 10:34
@codecov-io
Copy link

codecov-io commented Dec 19, 2019

Codecov Report

Merging #1276 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1276   +/-   ##
=======================================
  Coverage   35.94%   35.94%           
=======================================
  Files          44       44           
  Lines        6805     6805           
=======================================
  Hits         2446     2446           
  Misses       4000     4000           
  Partials      359      359
Impacted Files Coverage Δ
home/config.go 49.12% <ø> (ø) ⬆️
home/control.go 14.43% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00fabb0...c4aa013. Read the comment docs.

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