[PR #2254] [MERGED] Support DNS routes on iOS #14870

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2254
Author: @pascal-fischer
Created: 7/10/2024
Status: Merged
Merged: 7/15/2024
Merged by: @pascal-fischer

Base: mainHead: feature/support-dns-routes-on-ios


📝 Commits (7)

📊 Changes

9 files changed (+185 additions, -26 deletions)

View changed files

📝 client/internal/routemanager/manager.go (+11 -9)
📝 client/internal/routemanager/notifier/notifier.go (+37 -10)
📝 client/internal/routemanager/systemops/systemops.go (+12 -1)
📝 client/internal/routemanager/systemops/systemops_android.go (+1 -1)
📝 client/internal/routemanager/systemops/systemops_bsd_test.go (+1 -1)
📝 client/internal/routemanager/systemops/systemops_generic_test.go (+3 -3)
client/internal/routemanager/systemops/systemops_ios.go (+64 -0)
📝 client/ios/NetBirdSDK/client.go (+27 -1)
📝 client/ios/NetBirdSDK/routes.go (+29 -0)

📄 Description

Describe your changes

This PR adds support for exit nodes on iOS. It includes a small refactor to move the mobile notifier into the systemops and keep an in-memory map off all routes so that we can reuse the same logic for routing than on the desktop clients. It updates the route selection to show the domain names and the resolved IPs in the UI.

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/2254 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 7/10/2024 **Status:** ✅ Merged **Merged:** 7/15/2024 **Merged by:** [@pascal-fischer](https://github.com/pascal-fischer) **Base:** `main` ← **Head:** `feature/support-dns-routes-on-ios` --- ### 📝 Commits (7) - [`c373470`](https://github.com/netbirdio/netbird/commit/c37347095ad1b1769d475b1b57903b34e42bfb89) support exit nodes on iOS - [`cc95c6d`](https://github.com/netbirdio/netbird/commit/cc95c6df1f6f2c99e258022cbfda63817f02483b) fix comment - [`2b6211b`](https://github.com/netbirdio/netbird/commit/2b6211b906a338500c53f076ee22c4bb9c3dbd0e) update map for thread safety - [`2d1859c`](https://github.com/netbirdio/netbird/commit/2d1859c7979448d0fcdf72be4157e0a8bf043a50) remove wrongly commited file - [`c0f6715`](https://github.com/netbirdio/netbird/commit/c0f67155f85153a52750a2063adfb1bf782ac643) ignore linter - [`5ac5afa`](https://github.com/netbirdio/netbird/commit/5ac5afaf75f57e0ff4ea8a8a8e2e1b3c83f7eea6) ignore linter - [`f828ffa`](https://github.com/netbirdio/netbird/commit/f828ffa6c529263c7ffdde24fd157c65f9118016) remove lock from notify ### 📊 Changes **9 files changed** (+185 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/routemanager/manager.go` (+11 -9) 📝 `client/internal/routemanager/notifier/notifier.go` (+37 -10) 📝 `client/internal/routemanager/systemops/systemops.go` (+12 -1) 📝 `client/internal/routemanager/systemops/systemops_android.go` (+1 -1) 📝 `client/internal/routemanager/systemops/systemops_bsd_test.go` (+1 -1) 📝 `client/internal/routemanager/systemops/systemops_generic_test.go` (+3 -3) ➕ `client/internal/routemanager/systemops/systemops_ios.go` (+64 -0) 📝 `client/ios/NetBirdSDK/client.go` (+27 -1) 📝 `client/ios/NetBirdSDK/routes.go` (+29 -0) </details> ### 📄 Description ## Describe your changes This PR adds support for exit nodes on iOS. It includes a small refactor to move the mobile notifier into the systemops and keep an in-memory map off all routes so that we can reuse the same logic for routing than on the desktop clients. It updates the route selection to show the domain names and the resolved IPs in the UI. ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [x] Is a feature enhancement - [x] 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:30 -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#14870