[PR #543] [MERGED] Feature/dns protocol #12724

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/543
Author: @mlsmaycon
Created: 11/5/2022
Status: Merged
Merged: 11/7/2022
Merged by: @mlsmaycon

Base: mainHead: feature/dns-protocol


📝 Commits (10+)

  • 191479d Add protocol update
  • 8f167af Add dns update to protocol
  • 9e3d445 get label when updating/creating peer
  • 69eb73d Merge branch 'main' into feature/dns-protocol
  • 604e3b0 Update network map to include dns update
  • adfdd7b Send DNS update to client
  • 4c693fe Add retry with hostname
  • acdf49d update nameserver api with new fields
  • 17b7fc8 pass dns domain to manager builder
  • c1ef36d Fix tests

📊 Changes

29 files changed (+1316 additions, -242 deletions)

View changed files

📝 client/cmd/testutil.go (+1 -1)
client/internal/dns/mockServer.go (+35 -0)
📝 client/internal/dns/server.go (+23 -16)
📝 client/internal/dns/server_test.go (+18 -18)
📝 client/internal/engine.go (+55 -2)
📝 client/internal/engine_test.go (+182 -3)
📝 dns/dns.go (+31 -3)
📝 management/client/client_test.go (+1 -1)
management/cmd/defaults.go (+18 -0)
📝 management/cmd/management.go (+7 -1)
📝 management/cmd/root.go (+8 -33)
📝 management/proto/management.pb.go (+550 -103)
📝 management/proto/management.proto (+39 -0)
📝 management/server/account.go (+29 -2)
📝 management/server/account_test.go (+1 -1)
management/server/dns.go (+152 -0)
📝 management/server/grpcserver.go (+15 -15)
📝 management/server/http/api/openapi.yml (+5 -1)
📝 management/server/http/api/types.gen.go (+5 -0)
📝 management/server/http/nameservers.go (+14 -0)

...and 9 more files

📄 Description

Added DNS update protocol message

Added sync to clients

Update nameserver API with new fields

Added default NS groups

Added new dns-name flag for the management service append to peer DNS label


🔄 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/543 **Author:** [@mlsmaycon](https://github.com/mlsmaycon) **Created:** 11/5/2022 **Status:** ✅ Merged **Merged:** 11/7/2022 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `feature/dns-protocol` --- ### 📝 Commits (10+) - [`191479d`](https://github.com/netbirdio/netbird/commit/191479dd931bbf1149791def956603c3166c94fe) Add protocol update - [`8f167af`](https://github.com/netbirdio/netbird/commit/8f167af2c3e6fb75c86bbb10c33c08d68dad7469) Add dns update to protocol - [`9e3d445`](https://github.com/netbirdio/netbird/commit/9e3d445966a8a16cbe0b970d1bbe72d3c11775a8) get label when updating/creating peer - [`69eb73d`](https://github.com/netbirdio/netbird/commit/69eb73d8b92fa6c26fef538d4797e4d55c4b18e8) Merge branch 'main' into feature/dns-protocol - [`604e3b0`](https://github.com/netbirdio/netbird/commit/604e3b09d1b8d58b6a09393b87eeb2bbaa893b50) Update network map to include dns update - [`adfdd7b`](https://github.com/netbirdio/netbird/commit/adfdd7bb45e2edd1235ed7546aa1a941c38b3674) Send DNS update to client - [`4c693fe`](https://github.com/netbirdio/netbird/commit/4c693fee5cdf4fa47d905ac8c24980ea6633f66a) Add retry with hostname - [`acdf49d`](https://github.com/netbirdio/netbird/commit/acdf49dd24f8094fb340498e2b96c33d4917a193) update nameserver api with new fields - [`17b7fc8`](https://github.com/netbirdio/netbird/commit/17b7fc8874d5004573db2918c0ea98ba5d7d9c9d) pass dns domain to manager builder - [`c1ef36d`](https://github.com/netbirdio/netbird/commit/c1ef36dc1d7eae9461fee77dca8b7f6e982a8804) Fix tests ### 📊 Changes **29 files changed** (+1316 additions, -242 deletions) <details> <summary>View changed files</summary> 📝 `client/cmd/testutil.go` (+1 -1) ➕ `client/internal/dns/mockServer.go` (+35 -0) 📝 `client/internal/dns/server.go` (+23 -16) 📝 `client/internal/dns/server_test.go` (+18 -18) 📝 `client/internal/engine.go` (+55 -2) 📝 `client/internal/engine_test.go` (+182 -3) 📝 `dns/dns.go` (+31 -3) 📝 `management/client/client_test.go` (+1 -1) ➕ `management/cmd/defaults.go` (+18 -0) 📝 `management/cmd/management.go` (+7 -1) 📝 `management/cmd/root.go` (+8 -33) 📝 `management/proto/management.pb.go` (+550 -103) 📝 `management/proto/management.proto` (+39 -0) 📝 `management/server/account.go` (+29 -2) 📝 `management/server/account_test.go` (+1 -1) ➕ `management/server/dns.go` (+152 -0) 📝 `management/server/grpcserver.go` (+15 -15) 📝 `management/server/http/api/openapi.yml` (+5 -1) 📝 `management/server/http/api/types.gen.go` (+5 -0) 📝 `management/server/http/nameservers.go` (+14 -0) _...and 9 more files_ </details> ### 📄 Description Added DNS update protocol message Added sync to clients Update nameserver API with new fields Added default NS groups Added new dns-name flag for the management service append to peer DNS label --- <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 02:06:33 -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#12724