[PR #1071] [MERGED] Allow netbird traffic on windows #14816

Open
opened 2026-08-05 03:06:22 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1071
Author: @bcmmbaga
Created: 8/10/2023
Status: Merged
Merged: 8/21/2023
Merged by: @bcmmbaga

Base: allow-netbird-trafficHead: allow-netbird-traffic-windows


📝 Commits (5)

  • 8e94469 Allow netbird interface traffic on Windows nodes
  • 9660880 allow wg interface traffic on windows.
  • 9bf0d81 remove windows firewall rule on reset
  • c0863ea Moved the Reset function from uspfilter.go
  • f981173 improve interface mock expectations in ACL tests

📊 Changes

5 files changed (+109 additions, -11 deletions)

View changed files

📝 client/firewall/uspfilter/allow_netbird.go (+11 -0)
📝 client/firewall/uspfilter/allow_netbird_windows.go (+85 -0)
📝 client/firewall/uspfilter/uspfilter.go (+3 -11)
📝 client/firewall/uspfilter/uspfilter_test.go (+8 -0)
📝 client/internal/acl/manager_test.go (+2 -0)

📄 Description

Describe your changes

Allow Netbird Interface traffic on Windows nodes.

  • Investigate on how to mark NetBird network as Private on Windows (#438)

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/1071 **Author:** [@bcmmbaga](https://github.com/bcmmbaga) **Created:** 8/10/2023 **Status:** ✅ Merged **Merged:** 8/21/2023 **Merged by:** [@bcmmbaga](https://github.com/bcmmbaga) **Base:** `allow-netbird-traffic` ← **Head:** `allow-netbird-traffic-windows` --- ### 📝 Commits (5) - [`8e94469`](https://github.com/netbirdio/netbird/commit/8e9446967b93ae86feb87b5810239359ee030574) Allow netbird interface traffic on Windows nodes - [`9660880`](https://github.com/netbirdio/netbird/commit/9660880ab1b8b9bea843a99bf5990d4fd6fb6e9d) allow wg interface traffic on windows. - [`9bf0d81`](https://github.com/netbirdio/netbird/commit/9bf0d81362f51f8c9e86cfc44b1e9c75cbbbe869) remove windows firewall rule on reset - [`c0863ea`](https://github.com/netbirdio/netbird/commit/c0863ea2697da7275801112a8104e44a766791c6) Moved the Reset function from uspfilter.go - [`f981173`](https://github.com/netbirdio/netbird/commit/f9811737765d0d2c56000e08298667ae4a3b42b8) improve interface mock expectations in ACL tests ### 📊 Changes **5 files changed** (+109 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `client/firewall/uspfilter/allow_netbird.go` (+11 -0) 📝 `client/firewall/uspfilter/allow_netbird_windows.go` (+85 -0) 📝 `client/firewall/uspfilter/uspfilter.go` (+3 -11) 📝 `client/firewall/uspfilter/uspfilter_test.go` (+8 -0) 📝 `client/internal/acl/manager_test.go` (+2 -0) </details> ### 📄 Description ## Describe your changes Allow Netbird Interface traffic on Windows nodes. ## Issue ticket number and link * Investigate on how to mark NetBird network as Private on Windows ([#438][i438]) [i438]: https://github.com/netbirdio/netbird/issues/438 ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] 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:06:22 -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#14816