From 9a2c5ae18c5a4bce20d1a8c572d1261b191701dc Mon Sep 17 00:00:00 2001 From: Stan Grishin Date: Sun, 3 Sep 2023 20:33:17 +0000 Subject: [PATCH] https-dns-proxy: fix dns resolution not working on boot * fix dns resolution not working on boot * add hotplug-online script * reorganizes files/ and Makefile to reflect files destinations Signed-off-by: Stan Grishin --- net/https-dns-proxy/Makefile | 15 ++++---- .../config/https-dns-proxy} | 0 .../etc/hotplug.d/online/30-https-dns-proxy | 2 ++ .../init.d/https-dns-proxy} | 36 ++++++++++++------- .../50-https-dns-proxy-migrate-options.sh} | 0 .../files/https-dns-proxy.hotplug.iface | 6 ---- 6 files changed, 34 insertions(+), 25 deletions(-) rename net/https-dns-proxy/files/{https-dns-proxy.config => etc/config/https-dns-proxy} (100%) create mode 100644 net/https-dns-proxy/files/etc/hotplug.d/online/30-https-dns-proxy rename net/https-dns-proxy/files/{https-dns-proxy.init => etc/init.d/https-dns-proxy} (92%) rename net/https-dns-proxy/files/{https-dns-proxy.defaults => etc/uci-defaults/50-https-dns-proxy-migrate-options.sh} (100%) delete mode 100644 net/https-dns-proxy/files/https-dns-proxy.hotplug.iface diff --git a/net/https-dns-proxy/Makefile b/net/https-dns-proxy/Makefile index ddc43a8920..3319cbfe50 100644 --- a/net/https-dns-proxy/Makefile +++ b/net/https-dns-proxy/Makefile @@ -2,7 +2,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=https-dns-proxy PKG_VERSION:=2023-05-25 -PKG_RELEASE:=3 +PKG_RELEASE:=4 PKG_SOURCE_PROTO:=git PKG_SOURCE_URL:=https://github.com/aarond10/https_dns_proxy/ @@ -39,15 +39,16 @@ endef define Package/https-dns-proxy/install $(INSTALL_DIR) $(1)/usr/sbin + $(INSTALL_BIN) $(PKG_BUILD_DIR)/https_dns_proxy $(1)/usr/sbin/https-dns-proxy $(INSTALL_DIR) $(1)/etc/init.d + $(INSTALL_BIN) ./files/etc/init.d/https-dns-proxy $(1)/etc/init.d/https-dns-proxy + $(SED) "s|^\(readonly PKG_VERSION\).*|\1='$(PKG_VERSION)-$(PKG_RELEASE)'|" $(1)/etc/init.d/https-dns-proxy $(INSTALL_DIR) $(1)/etc/config - $(INSTALL_DIR) $(1)/etc/hotplug.d/iface + $(INSTALL_CONF) ./files/etc/config/https-dns-proxy $(1)/etc/config/https-dns-proxy + $(INSTALL_DIR) $(1)/etc/hotplug.d/online + $(INSTALL_DATA) ./files/etc/hotplug.d/online/30-https-dns-proxy $(1)/etc/hotplug.d/online/30-https-dns-proxy $(INSTALL_DIR) $(1)/etc/uci-defaults/ - $(INSTALL_BIN) $(PKG_BUILD_DIR)/https_dns_proxy $(1)/usr/sbin/https-dns-proxy - $(INSTALL_BIN) ./files/https-dns-proxy.init $(1)/etc/init.d/https-dns-proxy - $(SED) "s|^\(readonly PKG_VERSION\).*|\1='$(PKG_VERSION)-$(PKG_RELEASE)'|" $(1)/etc/init.d/https-dns-proxy - $(INSTALL_CONF) ./files/https-dns-proxy.config $(1)/etc/config/https-dns-proxy - $(INSTALL_BIN) ./files/https-dns-proxy.defaults $(1)/etc/uci-defaults/50-https-dns-proxy-migrate-options.sh + $(INSTALL_BIN) ./files/etc/uci-defaults/50-https-dns-proxy-migrate-options.sh $(1)/etc/uci-defaults/50-https-dns-proxy-migrate-options.sh endef $(eval $(call BuildPackage,https-dns-proxy)) diff --git a/net/https-dns-proxy/files/https-dns-proxy.config b/net/https-dns-proxy/files/etc/config/https-dns-proxy similarity index 100% rename from net/https-dns-proxy/files/https-dns-proxy.config rename to net/https-dns-proxy/files/etc/config/https-dns-proxy diff --git a/net/https-dns-proxy/files/etc/hotplug.d/online/30-https-dns-proxy b/net/https-dns-proxy/files/etc/hotplug.d/online/30-https-dns-proxy new file mode 100644 index 0000000000..0ff3ca28fb --- /dev/null +++ b/net/https-dns-proxy/files/etc/hotplug.d/online/30-https-dns-proxy @@ -0,0 +1,2 @@ +#!/bin/sh +/etc/init.d/https-dns-proxy start 'on_hotplug' diff --git a/net/https-dns-proxy/files/https-dns-proxy.init b/net/https-dns-proxy/files/etc/init.d/https-dns-proxy similarity index 92% rename from net/https-dns-proxy/files/https-dns-proxy.init rename to net/https-dns-proxy/files/etc/init.d/https-dns-proxy index e403686a62..6529bf3e32 100755 --- a/net/https-dns-proxy/files/https-dns-proxy.init +++ b/net/https-dns-proxy/files/etc/init.d/https-dns-proxy @@ -1,9 +1,9 @@ #!/bin/sh /etc/rc.common -# Copyright 2019-2022 Stan Grishin (stangri@melmac.ca) +# Copyright 2019-2023 Stan Grishin (stangri@melmac.ca) # shellcheck disable=SC1091,SC3043,SC3060 # shellcheck disable=SC2034 -START=95 +START=90 # shellcheck disable=SC2034 USE_PROCD=1 @@ -71,7 +71,7 @@ dnsmasq_restart() { [ -x /etc/init.d/dnsmasq ] || return 1; /etc/init.d/dnsmasq version() { echo "$PKG_VERSION"; } -xappend() { param="$param $1"; } +xappend() { PROG_param="$PROG_param $1"; } append_bool() { local section="$1" @@ -127,13 +127,18 @@ append_bootstrap() { [ "$ipv6_resolvers_only" -eq 0 ] && xappend '-4' } +resolver_health_check() { resolveip -t 3 one.one.one.one >/dev/null 2>&1; } + boot() { ubus -t 30 wait_for network.interface 2>/dev/null rc_procd start_service 'on_boot' + resolver_health_check || rc_procd stop_service 'on_boot' } start_instance() { - local cfg="$1" param listen_addr listen_port ipv6_resolvers_only p url iface + local cfg="$1" param="$2" + local PROG_param + local listen_addr listen_port ipv6_resolvers_only p url iface config_get url "$cfg" 'resolver_url' config_get_bool ipv6_resolvers_only "$cfg" 'use_ipv6_resolvers_only' '0' @@ -153,7 +158,7 @@ start_instance() { procd_open_instance # shellcheck disable=SC2086 - procd_set_param command $PROG $param + procd_set_param command $PROG $PROG_param procd_set_param stderr 1 procd_set_param stdout 1 procd_set_param respawn @@ -161,7 +166,6 @@ start_instance() { json_add_object mdns procd_add_mdns_service "$packageName" 'udp' "$port" "DNS over HTTPS proxy" json_close_object - json_add_string url "$url" if [ "$force_dns" -ne 0 ]; then json_add_array firewall for iface in $procd_fw_src_interfaces; do @@ -218,12 +222,13 @@ start_instance() { } start_service() { + local param="$1" local canaryDomains canary_domains_icloud canary_domains_mozilla local dnsmasq_config_update force_dns force_dns_port local procd_fw_src_interfaces local port=5053 - output "Starting $serviceName instances " + output "Starting $serviceName instances ${param:+$param }" config_load "$packageName" config_get_bool canary_domains_icloud 'config' 'canary_domains_icloud' '1' config_get_bool canary_domains_mozilla 'config' 'canary_domains_mozilla' '1' @@ -239,7 +244,7 @@ start_service() { fi dhcp_backup 'create' config_load "$packageName" - config_foreach start_instance "$packageName" + config_foreach start_instance "$packageName" "$param" output "\\n" if [ -n "$(uci_changes dhcp)" ]; then output "Updating dnsmasq config " @@ -248,6 +253,10 @@ start_service() { else output_failn fi + param='dnsmasq_restart' + fi + if [ "$param" = 'on_hotplug' ] || [ "$param" = 'on_boot' ] || \ + [ "$param" = 'dnsmasq_restart' ] ; then output "Restarting dnsmasq " if dnsmasq_restart; then output_okn @@ -258,10 +267,11 @@ start_service() { } stop_service() { + local param="$1" local canaryDomains canary_domains_icloud canary_domains_mozilla local dnsmasq_config_update local s=0 - output "Stopping $serviceName " + output "Stopping $serviceName ${param:+$param }" config_load "$packageName" config_get dnsmasq_config_update 'config' 'dnsmasq_config_update' '*' config_get_bool canary_domains_icloud 'config' 'canary_domains_icloud' '1' @@ -278,7 +288,7 @@ stop_service() { dnsmasq_restart || s=1 fi # shellcheck disable=SC2015 - [ "$s" -eq 0 ] && output_okn || output_failn + [ "$s" = '0' ] && output_okn || output_failn } # shellcheck disable=SC1091 @@ -319,14 +329,16 @@ dnsmasq_doh_server() { ::) address='::1';; esac uci_add_list_if_new 'dhcp' "$cfg" 'server' "${address}#${port}" + uci_add_list_if_new 'dhcp' "$cfg" 'doh_server' "${address}#${port}" ;; remove) - eval "$(ubus call service list "{ 'verbose': true, 'name': '$packageName' }" | jsonfilter -F '# ' -e 'TUPLES=@[*].instances[*].command[4,6]')" - for i in $TUPLES; do + for i in $(uci -q get "dhcp.$cfg.doh_server"); do uci_remove_list 'dhcp' "$cfg" 'server' "$i" + uci_remove_list 'dhcp' "$cfg" 'doh_server' "$i" done for i in $canaryDomains; do uci_remove_list 'dhcp' "$cfg" 'server' "/${i}/" + uci_remove_list 'dhcp' "$cfg" 'doh_server' "/${i}/" done ;; esac diff --git a/net/https-dns-proxy/files/https-dns-proxy.defaults b/net/https-dns-proxy/files/etc/uci-defaults/50-https-dns-proxy-migrate-options.sh similarity index 100% rename from net/https-dns-proxy/files/https-dns-proxy.defaults rename to net/https-dns-proxy/files/etc/uci-defaults/50-https-dns-proxy-migrate-options.sh diff --git a/net/https-dns-proxy/files/https-dns-proxy.hotplug.iface b/net/https-dns-proxy/files/https-dns-proxy.hotplug.iface deleted file mode 100644 index c25b9e26d0..0000000000 --- a/net/https-dns-proxy/files/https-dns-proxy.hotplug.iface +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/sh - -if [ "$ACTION" = 'ifup' ] && [ "$INTERFACE" = 'wan' ] && /etc/init.d/https-dns-proxy enabled; then - logger -t "https-dns-proxy" "Restarting https-dns-proxy due to $ACTION of $INTERFACE" - /etc/init.d/https-dns-proxy restart -fi -- 2.30.2