-
-
Notifications
You must be signed in to change notification settings - Fork 629
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
Register listener on the dbus interface to detect Avahi deamon restarts #970
Conversation
@adriancable I'm currently unable to test this on a live machine. Could you verify, that this works as expected? |
Pull Request Test Coverage Report for Build 3358922347
💛 - Coveralls |
Hi @Supereg - I'm on vacation until the end of the month and don't have access to a Linux machine to test. I'll happily have a look at the start of October if you can't find anyone else to test before that. Sorry! |
Well, I’m for a half a year in South Korea 😄 so let’s see, maybe I can start up one of my machines at home :) otherwise it will be October 🙃 |
1cd69df
to
3f69ec8
Compare
Have verified functionality (have restarted the daemon twice in the below log output). The re advertisement is also pretty fast:
Calling
|
…ts (#970) * Register listener on the listener to detect Avahi deamon restarts * Add comment about systemd-resolved feature state.
♻️ Current situation
As of now we don't detect restarts of Avahi Deamon and therefore remain without an advertising in this situation as explained in #923.
💡 Proposed solution
This PR fixes this by subscribing on the dbus
StateChanged
listener to detect avahi deamon restarts.⚙️ Release Notes
➕ Additional Information
Testing
--
Reviewer Nudging
--