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

ClientId for DoH upstream #225

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

apearson
Copy link

Longer Description

#207

Feature

A ClientId is passed to DoH upstream

How

Use DoH with ClientId in:

Technical How

  • Reuse the upstream option send_client_info to enable sending client info (id)
  • Reuse the service option client_id_preference to choose what is chosen for the clientId
  • New upstream option client_id_type (headers, subdomain, path) to choose where the clientId goes
  • Modify the URL in the DoH resolve function
  • Send ControlD headers to 3rd party resolver if client_id_type is set to headers

Notes

  • There are no tests for the fork's code
  • Let me know if anything is needed in this PR

Thank you,

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.

1 participant