Skip to content

Include https-dns-proxy with safe defaults#1691

Merged
gsanchietti merged 1 commit into
NethServer:nethsecurity-8.8from
gsanchietti:https_dns_proxy
May 26, 2026
Merged

Include https-dns-proxy with safe defaults#1691
gsanchietti merged 1 commit into
NethServer:nethsecurity-8.8from
gsanchietti:https_dns_proxy

Conversation

@gsanchietti
Copy link
Copy Markdown
Member

@gsanchietti gsanchietti commented May 26, 2026

Summary

  • include https-dns-proxy in the image
  • default dnsmasq_config_update to -
  • keep the service disabled on boot while dnsmasq integration stays disabled
  • update the DoH design documentation

Related issue

Refs: #1638

How to test

  1. Build an image from this branch.
  2. Verify /etc/config/https-dns-proxy exists and defaults dnsmasq_config_update to -.
  3. Verify /etc/init.d/https-dns-proxy enabled returns false after first boot.
  4. Set uci set https-dns-proxy.config.dnsmasq_config_update='*', commit the config, enable the service, and start it.
  5. Verify dnsmasq receives the DoH upstream entries and the proxy keeps running after manual enablement.

Dependencies

Known limitation

If dnsmasq_config_update remains -, the first-boot defaults script can disable https-dns-proxy again after an image upgrade. Right now this should not be a practical problem because configuration is supported only from the command line.

Bundle https-dns-proxy in the image and default\ndnsmasq_config_update to '-'. This keeps the package available\nout of the box without changing dnsmasq or starting the service\nuntil an administrator enables it.\n\nTreat dnsmasq_config_update='-' as the disabled state so image\nupgrades can keep the default-off behavior. This can disable the\nservice again after an upgrade if dnsmasq integration remains\ndisabled, which is acceptable for now because configuration is\nsupported only from the command line.\n\nAssisted-by: Copilot:gpt-5.4
@gsanchietti gsanchietti self-assigned this May 26, 2026
@gsanchietti gsanchietti changed the base branch from main to nethsecurity-8.8 May 26, 2026 09:08
@gsanchietti gsanchietti mentioned this pull request May 26, 2026
36 tasks
@gsanchietti gsanchietti marked this pull request as ready for review May 26, 2026 09:20
@gsanchietti gsanchietti requested a review from Tbaile May 26, 2026 09:20
@gsanchietti gsanchietti merged commit d853f7e into NethServer:nethsecurity-8.8 May 26, 2026
1 of 2 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.

2 participants