-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
https-dns-proxy init'ed before network is available and fails to find bootstrap server(s) #19531
Comments
Do you also have a WAN interface or just WAN6? |
Both. But only IPV6 bootstrap servers defined. |
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca>
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 796a3dd)
@Ohfalderal If you could test the init script from the PRs above, I'd appreciate it. It should cause https-dns-proxy to be restarted on WAN6 changes and should alleviate the problem. |
@stangri
Replaced init.d/https-dns-proxy.init and rebooted -
Due to #19366 it is not possible to determine if the DoH servers are correctly initialised. @stangri
N.B. - you may have noticed that the syslogs are from different APs. '-2' is 21.02 and '-0' is 22.03 |
If you're willing to disable logging to test the boot up functionality, please try the init script from this commit. The Please limit 1 (one) problem per issue. |
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt/packages#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca>
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt/packages#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca>
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca>
@stangri - Was this comment intended for #19366? |
@Ohfalderal yes, thank you, I'll move the comment. On a related note, I'm testing this: stangri/source.openwrt.melmac.net@7402fdc |
* update to upstream version 2022-08-12 * add ca_certs_file option for CA certs file for curl * add procd_add_interface_trigger for wan6 (hopefully fixes openwrt#19531) Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 796a3dd)
@Ohfalderal do you have a different experience now that the |
@stangri it would appear that a race condition still exists -
|
Can you try adjusting the value of |
@stangri ...done (restart @ 03:21 CEST so tomorrow an update) But why not implement the initialisation on a hotplug trigger? |
There is a hotplug script and PROCD service trigger. |
@stangri increasing the startup delay provokes cURL errors -
But what is more interesting is when the logging level is raised from 2 to 4 the cURL errors do not occur (perhaps the build of cURL for the Archer Cn platform has logging enabled #19366?) -
|
@Ohfalderal this PR: #22122 in combination with a previous one to stop service if resolution is not working on boot should address the issue. |
I added a 'service https-dns-proxy restart' to rc.local .
…On Sun, 17 Sept 2023, 03:59 Stan Grishin, ***@***.***> wrote:
@Ohfalderal <https://github.com/Ohfalderal> this PR: #22122
<#22122> in combination with a
previous one to stop service if resolution is not working on boot should
address the issue.
—
Reply to this email directly, view it on GitHub
<#19531 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3JNOAKRE3EZ4GXI5ZZIXCLX2ZKPHANCNFSM6AAAAAAQ5MIY4I>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I'm closing this then. |
Maintainer: @stangri
Environment: mips_24kc, TP-Link Archer C7 v5, OpenWrt 21.02.3 r16554-1d4dea6d4f
Description: process(es) spawned (rc.d/S80https-dns-proxy) and init'ed before network connectivity is established and never retries to resolve the DoH server(s).
Wed Oct 5 03:20:18 2022 user.notice https-dns-proxy: Starting service ✓✓✓✓✓
Wed Oct 5 03:20:34 2022 daemon.notice netifd: Interface 'wan6' is now up
https-dns-proxy log -
N.B. 1664932820 = Wednesday, October 5, 2022 3:20:20
The text was updated successfully, but these errors were encountered: