-
Notifications
You must be signed in to change notification settings - Fork 2k
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
AdGuardHome installed wrong Init script #2757
Comments
Merge in DNS/adguard-home from 2757-openwrt to master Updates #2757. Squashed commit of the following: commit 8e94e6a Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 17:02:24 2021 +0300 util: convert only once commit f1c74f4 Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 16:22:51 2021 +0300 util: log changes, imp docs commit 0a4558d Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 15:53:26 2021 +0300 util: imp os detection
We've tried to improve the OpenWRT detection as of snapshot 91403d0. Could you please check if our solution fixes the issue for you? |
I unfortunately cannot, at the moment, im running AdGuard Home in my main router and i would love not to reinstall it from scratch. |
Well, thanks for opening an issue anyway. Our testers checked the solution on our test machines, including the OpenWRT one, before it was merged to |
I just ran into this bug using a custom build from Divested, which is effectively running OpenWRT HEAD with a few patches. Looking at the AdguardHome code, it's checking for "OpenWrt" in However
Edit: Actually
|
Merge in DNS/adguard-home from 2757-openwrt to master Updates AdguardTeam#2757. Squashed commit of the following: commit 8e94e6a Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 17:02:24 2021 +0300 util: convert only once commit f1c74f4 Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 16:22:51 2021 +0300 util: log changes, imp docs commit 0a4558d Author: Eugene Burkov <[email protected]> Date: Mon Mar 1 15:53:26 2021 +0300 util: imp os detection
Expected Behavior
After install, i would exepct AdGuard to run and enable the correct init script
Actual Behavior
Init script is installed in /etc/init.d but it's not the correct one for OpenWRT and so it doesn't "enable" it.
Additional Information
Problem might be related to how OpenWRT is detected.
AdGuardHome/internal/util/helpers.go
Lines 68 to 79 in 0d67aa2
There is no guarantee that /etc/os-release file will be present in the host system, you should rather look for /etc/openwrt_release.
The text was updated successfully, but these errors were encountered: