Skip to content
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

DHCP server broken on v0.105.0/snap 1687 #2667

Closed
marxjohnson opened this issue Feb 11, 2021 · 4 comments
Closed

DHCP server broken on v0.105.0/snap 1687 #2667

marxjohnson opened this issue Feb 11, 2021 · 4 comments
Assignees
Labels
bug P3: Medium snap Snap-related issues
Milestone

Comments

@marxjohnson
Copy link

Issue Details

  • Version of AdGuard Home server:
    • v0.105.0
  • How did you install AdGuard Home:
    • snap
  • How did you setup DNS configuration:
    • System
  • If it's a router or IoT, please write device model:
  • Operating system and version:
    • Ubuntu 18.04.5

Expected Behavior

DHCP server continues to work after the snap updated.

Actual Behavior

DHCP server does not respond to requests. Disabling and re-enabling the DCHP server through the web interface results in the error:
Error: control/dhcp/set_config | enabling dhcp: checking static ip: open /etc/network/interfaces: permission denied | 500

Additional Information

Previous issues with the DHCP server were resolved by running sudo snap connect adguard-home:network-control. This now results in the message error: snap "adguard-home" has no plug named "network-control".

Running sudo snap revert adguard-home reverted to v0.104.3, which works as before.

Other information about the snap:

$ snap info adguard-home
name:      adguard-home
summary:   Network-wide ads & trackers blocking DNS server
publisher: AdGuard (ameshkov✓)
store-url: https://snapcraft.io/adguard-home
  thus preventing your devices from connecting to those servers. It's based
  on software we use for our public AdGuard DNS servers -- both share a lot
  of common code.
  
  IMPORTANT:
  1. Once it's installed, open http://127.0.0.1:3000/ in your browser to do the initial setup and
  learn how to configure your devices to use AdGuard Home. You don't need to Launch anything
commands:
  - adguard-home.adguard-home-web
services:
  adguard-home: simple, enabled, active
snap-id:      UXZIkJfJT2SPCGejjnSjOBqJ71yHk8bw
store-url: https://snapcraft.io/adguard-home
  you set it up, it'll cover ALL your home devices, and you don't need any
  client-side software for that.
  
  It operates as a DNS server that re-routes tracking domains to a "black hole,"
  thus preventing your devices from connecting to those servers. It's based
  on software we use for our public AdGuard DNS servers -- both share a lot
  of common code.
  
  IMPORTANT:
  1. Once it's installed, open http://127.0.0.1:3000/ in your browser to do the initial setup and
  learn how to configure your devices to use AdGuard Home. You don't need to Launch anything
  explicitly, AdGuard Home service is already started after the installation.
  2. If you're running it on Ubuntu, you may need to select a different "Listen interface" for the
  DNS server. Listening to all interfaces is not possible because Ubuntu's systemd-resolved service
  uses 127.0.0.1:53 already.
commands:
  - adguard-home.adguard-home-web
services:
  adguard-home: simple, enabled, active
snap-id:      UXZIkJfJT2SPCGejjnSjOBqJ71yHk8bw
tracking:     latest/stable
refresh-date: yesterday at 22:38 UTC
channels:
  latest/stable:    v0.105.0        2021-02-10 (1687) 10MB -
  latest/candidate: ↑                                      
  latest/beta:      v0.105.0-beta.5 2021-02-10 (1682) 10MB -
  latest/edge:      v0.105.0        2021-02-10 (1687) 10MB -
installed:          v0.105.0                   (1687) 10MB -
$ snap connections adguard-home 
Interface        Plug                          Slot           Notes
desktop          adguard-home:desktop          :desktop       -
network-bind     adguard-home:network-bind     :network-bind  -
network-observe  adguard-home:network-observe  -              -
@ainar-g ainar-g added bug P3: Medium snap Snap-related issues labels Feb 11, 2021
@ainar-g ainar-g self-assigned this Feb 11, 2021
@ainar-g ainar-g added this to the v0.105.1 milestone Feb 11, 2021
@ainar-g
Copy link
Contributor

ainar-g commented Feb 11, 2021

Hi, and thank you for the report! Does sudo snap connect adguard-home:network-observe fix the issue? We've been advised by the people working in Snap Store to not use network-control, as it may provide too much access to the host machine, but network-observe should work for most cases.

@marxjohnson
Copy link
Author

I tried as you suggested, but to no avail:

mark@strongmad:~$ sudo snap refresh adguard-home 
adguard-home v0.105.0 from AdGuard (ameshkov✓) refreshed
mark@strongmad:~$ sudo snap connect adguard-home:network-observe
mark@strongmad:~$ snap connections adguard-home 
Interface        Plug                          Slot              Notes
desktop          adguard-home:desktop          :desktop          -
network-bind     adguard-home:network-bind     :network-bind     -
network-observe  adguard-home:network-observe  :network-observe  manual

After this, the error occurred as before.

adguard pushed a commit that referenced this issue Feb 12, 2021
Merge in DNS/adguard-home from 2667-eperm-dhcp to master

Updates #2667.

Squashed commit of the following:

commit 7fad607
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Fri Feb 12 16:27:59 2021 +0300

    dhcpd: assume static ip on eperm
@ainar-g
Copy link
Contributor

ainar-g commented Feb 12, 2021

We've pushed some fixes in snapshot 10f03b7. It should already be on the edge channel in the Snap store. Can you please check if our solution fixes the issue for you?

@marxjohnson
Copy link
Author

I can confirm that snapshot fixes the issue.

@ainar-g ainar-g closed this as completed Feb 13, 2021
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from 2667-eperm-dhcp to master

Updates AdguardTeam#2667.

Squashed commit of the following:

commit 7fad607
Author: Ainar Garipov <A.Garipov@AdGuard.COM>
Date:   Fri Feb 12 16:27:59 2021 +0300

    dhcpd: assume static ip on eperm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug P3: Medium snap Snap-related issues
Projects
None yet
Development

No branches or pull requests

2 participants