[PR #3350] [MERGED] [client] fix privacy warning on macOS #17195

Open
opened 2026-08-05 04:07:02 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3350
Author: @hurricanehrndz
Created: 2/19/2025
Status: Merged
Merged: 2/22/2025
Merged by: @lixmal

Base: mainHead: fix_macos_privacy_warning


📝 Commits (4)

  • 17a35cb fix: macos privacy warning
  • 9767865 Remove unused import
  • 66061d7 add comment to onTrayExit
  • 4b781b5 Merge branch 'main' into fix_macos_privacy_warning

📊 Changes

4 files changed (+13 additions, -10 deletions)

View changed files

📝 client/system/info.go (+0 -6)
📝 client/ui/client_ui.go (+3 -2)
client/ui/desktop/desktop.go (+8 -0)
📝 client/ui/event/event.go (+2 -2)

📄 Description

Describe your changes

On macOS, detect cloud and detect platform reach out to broadcast address, thus triggering the network privacy warning. This patch ensures that init is only called by programs running as root, i.e. the daemon.

This will ensure user programs like netbird-ui do not access broadcast address triggering privacy warnings.

static_info is unused by netbird-ui.

fixes #3326

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/3350 **Author:** [@hurricanehrndz](https://github.com/hurricanehrndz) **Created:** 2/19/2025 **Status:** ✅ Merged **Merged:** 2/22/2025 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `fix_macos_privacy_warning` --- ### 📝 Commits (4) - [`17a35cb`](https://github.com/netbirdio/netbird/commit/17a35cb4ac0072f803201c28fc664ce428767136) fix: macos privacy warning - [`9767865`](https://github.com/netbirdio/netbird/commit/9767865766e3fe6b679c70c54561f21959f8e089) Remove unused import - [`66061d7`](https://github.com/netbirdio/netbird/commit/66061d7fc033f133fe062dfd014d2c9f7dd61233) add comment to onTrayExit - [`4b781b5`](https://github.com/netbirdio/netbird/commit/4b781b52f164f998cb73b61b6b1ab0bb3f6f3970) Merge branch 'main' into fix_macos_privacy_warning ### 📊 Changes **4 files changed** (+13 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `client/system/info.go` (+0 -6) 📝 `client/ui/client_ui.go` (+3 -2) ➕ `client/ui/desktop/desktop.go` (+8 -0) 📝 `client/ui/event/event.go` (+2 -2) </details> ### 📄 Description ## Describe your changes On macOS, detect cloud and detect platform reach out to broadcast address, thus triggering the network privacy warning. This patch ensures that init is only called by programs running as root, i.e. the daemon. This will ensure user programs like netbird-ui do not access broadcast address triggering privacy warnings. static_info is unused by netbird-ui. ## Issue ticket number and link fixes #3326 ### 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 04:07:02 -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#17195