Skip to content

Add Quad9 Unsecured w/ ECS support provider - #22

Merged
stangri merged 2 commits into
mossdef-org:mainfrom
Snuupy:patch-1
Aug 12, 2026
Merged

Add Quad9 Unsecured w/ ECS support provider#22
stangri merged 2 commits into
mossdef-org:mainfrom
Snuupy:patch-1

Conversation

@Snuupy

@Snuupy Snuupy commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: Snuupy <Snuupy@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the missing Quad9 “Unsecured w/ ECS support” endpoint option to the Quad9 provider definition so users can select it in https-dns-proxy’s configuration.

Changes:

  • Add new dns12 provider variant labeled “Unsecured with ECS Support”.
  • Extend the Quad9 options list to expose the new variant in the UI/config.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread root/usr/share/https-dns-proxy/providers/net.quad9.json
@stangri

stangri commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Thank you for this @Snuupy , can you please address the comment from copilot, so that I could merge this? I'm not confident how quickly this will make it into the OpenWrt luci repo -- there have been many reports that Quad9 has been flaky lately and we were actually considering dropping it from the list. I believe @egc112 has been more involved with the community lately, so I would rely on their opinion on this.

@egc112

egc112 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Updated regex to include 'dns12' as a valid option.

Signed-off-by: Snuupy <Snuupy@users.noreply.github.com>
@Snuupy

Snuupy commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

I don't know anything about the curl/quad9 issues, all I know is that dns.sb has been routing my NA packets to EU recently so I've had to switch DNS providers or hardcode the specific geolocation for dns.sb, so I've resolved to use quad9 for the time being.

However, the malware/adblocking lists sometimes block tracking links which leads to people always complaining that they can't click affiliate links/shopping etc. so I needed to use unfiltered. I only realized I wanted to use unfiltered (.10) after trying to use unfiltered + ECS (.12) and submitting this PR.

Regardless, updated the PR with the regex as well :)

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@stangri

stangri commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

@Snuupy thank you for this PR and quick response to copilot. I've been meaning to create a PR for a major change in the principal package and the luci package will get updated at the same time.

I have limited time to work on OSS until mid-September, so it may not happen until then, but I'll make sure the package in mossdef-org repo is updated.

@stangri
stangri merged commit c8d4734 into mossdef-org:main Aug 12, 2026
4 checks passed
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.

4 participants