[PR #1513] [MERGED] getFirstListenerAvailable(): adjust logging levels and add success message #15485

Closed
opened 2026-08-05 03:07:46 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1513
Author: @nazarewk
Created: 1/31/2024
Status: Merged
Merged: 1/31/2024
Merged by: @mlsmaycon

Base: mainHead: dns-binding-messages


📝 Commits (2)

  • 142f543 getFirstListenerAvailable(): adjust logging levels and add success message
  • b5f9f00 getFirstListenerAvailable() -> getFirstDNSListenerAvailable()

📊 Changes

1 file changed (+4 additions, -3 deletions)

View changed files

📝 client/internal/dns/service_listener.go (+4 -3)

📄 Description

Describe your changes

it was worrying to see multiple warnings and no success message when lacking CAP_NET_BIND_SERVICE

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)
  • Extended the README / documentation, if necessary

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbirdio/netbird/pull/1513 **Author:** [@nazarewk](https://github.com/nazarewk) **Created:** 1/31/2024 **Status:** ✅ Merged **Merged:** 1/31/2024 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `dns-binding-messages` --- ### 📝 Commits (2) - [`142f543`](https://github.com/netbirdio/netbird/commit/142f543a7e0dd23dda1d846b0e6b2bacc9bf9ec0) getFirstListenerAvailable(): adjust logging levels and add success message - [`b5f9f00`](https://github.com/netbirdio/netbird/commit/b5f9f00814ae67f999e1ee1d722b8ea5861b3a57) getFirstListenerAvailable() -> getFirstDNSListenerAvailable() ### 📊 Changes **1 file changed** (+4 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/dns/service_listener.go` (+4 -3) </details> ### 📄 Description ## Describe your changes it was worrying to see multiple warnings and no success message when lacking CAP_NET_BIND_SERVICE ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [x] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) - [ ] Extended the README / documentation, if necessary --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 03:07:46 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#15485