[PR #4723] [MERGED] [client] Extend Darwin network monitoring with wakeup detection (cleanup branch) #26135

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4723
Author: @pappz
Created: 10/30/2025
Status: Merged
Merged: 10/31/2025
Merged by: @lixmal

Base: mainHead: feature/detect-mac-wakeup-2


📝 Commits (10+)

  • b1ebac7 Extend Darwin network monitoring with wakeup detection
  • 885b5c6 Update client/internal/networkmonitor/monitor.go
  • bfa776c Update client/internal/networkmonitor/check_change_darwin.go
  • 158f3ac Handle better sleep period
  • 14fe7c2 Change log levels
  • 11ba253 Merge branch 'main' into feature/detect-mac-wakeup
  • c81e1ee Fix code duplication
  • 67c5c04 Fix code duplication
  • 5250292 Revert "Fix code duplication"
  • cf11c28 Fix code duplication

📊 Changes

4 files changed (+246 additions, -73 deletions)

View changed files

📝 client/internal/networkmonitor/check_change_bsd.go (+4 -73)
client/internal/networkmonitor/check_change_common.go (+92 -0)
client/internal/networkmonitor/check_change_darwin.go (+149 -0)
📝 client/internal/networkmonitor/monitor.go (+1 -0)

📄 Description

Describe your changes

Stack

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)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 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/4723 **Author:** [@pappz](https://github.com/pappz) **Created:** 10/30/2025 **Status:** ✅ Merged **Merged:** 10/31/2025 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `feature/detect-mac-wakeup-2` --- ### 📝 Commits (10+) - [`b1ebac7`](https://github.com/netbirdio/netbird/commit/b1ebac795da8568810c8248706b2e78828a1917f) Extend Darwin network monitoring with wakeup detection - [`885b5c6`](https://github.com/netbirdio/netbird/commit/885b5c68ad2d647855e4ddb112f78b46ece43281) Update client/internal/networkmonitor/monitor.go - [`bfa776c`](https://github.com/netbirdio/netbird/commit/bfa776c155575fdead60b9e311577bc21ae61dcf) Update client/internal/networkmonitor/check_change_darwin.go - [`158f3ac`](https://github.com/netbirdio/netbird/commit/158f3aceff66d0bbfc207bcd5506a605f592078a) Handle better sleep period - [`14fe7c2`](https://github.com/netbirdio/netbird/commit/14fe7c29cb4a0004aee55b09b1f729af6a548c92) Change log levels - [`11ba253`](https://github.com/netbirdio/netbird/commit/11ba253ffb66b5a8534bf925c85d6eadb3e96948) Merge branch 'main' into feature/detect-mac-wakeup - [`c81e1ee`](https://github.com/netbirdio/netbird/commit/c81e1ee8854897830bc97c3017852e7a5d486edf) Fix code duplication - [`67c5c04`](https://github.com/netbirdio/netbird/commit/67c5c04e1969e1d9f96e0268f728110d2ac299fd) Fix code duplication - [`5250292`](https://github.com/netbirdio/netbird/commit/5250292233f555aef058f785c0d307e4dcc9845e) Revert "Fix code duplication" - [`cf11c28`](https://github.com/netbirdio/netbird/commit/cf11c28ebe15e7370f05469f42b58782581406de) Fix code duplication ### 📊 Changes **4 files changed** (+246 additions, -73 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/networkmonitor/check_change_bsd.go` (+4 -73) ➕ `client/internal/networkmonitor/check_change_common.go` (+92 -0) ➕ `client/internal/networkmonitor/check_change_darwin.go` (+149 -0) 📝 `client/internal/networkmonitor/monitor.go` (+1 -0) </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### 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) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <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 07:06:56 -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#26135