joshcangit / dnsproxy-resolvconf

Run Adguard's dnsproxy in a systemd service.
Apache License 2.0
9 stars 2 forks source link

fix setup #1

Closed malikshi closed 2 years ago

malikshi commented 2 years ago
malikshi commented 2 years ago

do we need WatchdogSec ?

joshcangit commented 2 years ago

do we need WatchdogSec ?

Oh, I took that from the systemd-resolved.service file without thinking much. I think that line might not be needed. I wonder what else from that service file is not needed. If so, should I wait for you to edit the service file?

joshcangit commented 2 years ago

Thanks for fixing the Makefile.

I was considering to remove xh

3c3
< CMDS=$(type -P {aria2c,xh,curl,wget})
---
> CMDS=$(type -P {aria2c,curl,wget})
6d5
<     *xh*) xh HEAD $1 --no-check-status -h;;
20,21d18
<     *xh*)
<     xh -Fd $URL -o "$TMP_DIR/$FILE" Accept-Encoding:'zstd, gzip, deflate';;

For me, using xh was pretty fast but that was on my Ubuntu PC. It also worked on Termux but strangely wouldn't work on Termux:Tasker. Maybe xh might have given you problems but I doubt wget would have been problematic.

malikshi commented 2 years ago

For me, using xh was pretty fast but that was on my Ubuntu PC. It also worked on Termux but strangely wouldn't work on Termux:Tasker. Maybe xh might have given you problems but I doubt wget would have been problematic.

i didn't really use xh. i test the script with RPI4B and debian VPS's

do we need WatchdogSec ?

Oh, I took that from the systemd-resolved.service file without thinking much. I think that line might not be needed. I wonder what else from that service file is not needed. If so, should I wait for you to edit the service file?

is it good now?

malikshi commented 2 years ago

add priority dns, use resolv head instead of resolv tail.