[PR #2639] [MERGED] [client] Remove usage of custom dialer for localhost #15421

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2639
Author: @lixmal
Created: 9/24/2024
Status: Merged
Merged: 9/24/2024
Merged by: @lixmal

Base: mainHead: use-std-dialer-for-local


📝 Commits (2)

  • cb6d604 Downgrade error log level for network monitor warnings
  • 9269909 Do not use custom dialer for localhost

📊 Changes

2 files changed (+7 additions, -8 deletions)

View changed files

📝 client/internal/networkmonitor/monitor_bsd.go (+5 -5)
📝 client/internal/wgproxy/proxy_userspace.go (+2 -3)

📄 Description

Describe your changes

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/2639 **Author:** [@lixmal](https://github.com/lixmal) **Created:** 9/24/2024 **Status:** ✅ Merged **Merged:** 9/24/2024 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `use-std-dialer-for-local` --- ### 📝 Commits (2) - [`cb6d604`](https://github.com/netbirdio/netbird/commit/cb6d604b4ade95cbe559aefa4fefe94d8ad4ed0f) Downgrade error log level for network monitor warnings - [`9269909`](https://github.com/netbirdio/netbird/commit/92699090d8f0f713fd5f1cb7cef42be268a3cc6f) Do not use custom dialer for localhost ### 📊 Changes **2 files changed** (+7 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/networkmonitor/monitor_bsd.go` (+5 -5) 📝 `client/internal/wgproxy/proxy_userspace.go` (+2 -3) </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### Checklist - [x] 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 --- <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:37 -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#15421