[PR #3291] [MERGED] [management] Support Extra DNS Labels for Peer Addressing #19965

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3291
Author: @hakansa
Created: 2/6/2025
Status: Merged
Merged: 2/20/2025
Merged by: @hakansa

Base: feature/dns-labelsHead: feat/extra-dns-labels-mgmt


📝 Commits (10+)

  • d0f24b6 Add ExtraDNSLabels to PeerLogin and Peer structures
  • 07631dd Clone ExtraDNSLabels in Peer Copy method
  • 9356b30 Update go.mod
  • e93a2af Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt
  • 259c72e Update go.mod
  • 5470e05 Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt
  • 30272f8 Add allowExtraDNSLabels parameter to CreateSetupKey and related functions
  • 90c0d27 Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt
  • 229b433 Add AllowExtraDNSLabels field to Peer and update related logic
  • e0fa038 Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt

📊 Changes

19 files changed (+313 additions, -72 deletions)

View changed files

📝 management/domain/validate.go (+25 -0)
📝 management/domain/validate_test.go (+109 -0)
📝 management/server/account.go (+1 -1)
📝 management/server/account_test.go (+3 -3)
📝 management/server/grpcserver.go (+1 -0)
📝 management/server/http/api/openapi.yml (+16 -0)
📝 management/server/http/api/types.gen.go (+18 -0)
📝 management/server/http/handlers/peers/peers_handler.go (+10 -0)
📝 management/server/http/handlers/setup_keys/setupkeys_handler.go (+22 -15)
📝 management/server/http/handlers/setup_keys/setupkeys_handler_test.go (+3 -2)
📝 management/server/management_proto_test.go (+1 -1)
📝 management/server/mock_server/account_mock.go (+3 -2)
📝 management/server/peer.go (+26 -0)
📝 management/server/peer/peer.go (+7 -0)
📝 management/server/peer_test.go (+3 -3)
📝 management/server/setupkey.go (+2 -2)
📝 management/server/setupkey_test.go (+12 -12)
📝 management/server/types/account.go (+16 -1)
📝 management/server/types/setupkey.go (+35 -30)

📄 Description

Describe your changes

Support Extra DNS Labels for Peer Addressing

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/3291 **Author:** [@hakansa](https://github.com/hakansa) **Created:** 2/6/2025 **Status:** ✅ Merged **Merged:** 2/20/2025 **Merged by:** [@hakansa](https://github.com/hakansa) **Base:** `feature/dns-labels` ← **Head:** `feat/extra-dns-labels-mgmt` --- ### 📝 Commits (10+) - [`d0f24b6`](https://github.com/netbirdio/netbird/commit/d0f24b6a1e3b228257fab80496f2cf8776486a31) Add ExtraDNSLabels to PeerLogin and Peer structures - [`07631dd`](https://github.com/netbirdio/netbird/commit/07631dd49626f46eecb2cf66ad88cdf9863f2b5c) Clone ExtraDNSLabels in Peer Copy method - [`9356b30`](https://github.com/netbirdio/netbird/commit/9356b30476063b90147a947919eb61243b8f230d) Update go.mod - [`e93a2af`](https://github.com/netbirdio/netbird/commit/e93a2aff1790b53e47766dfae6bf1c3980b0dc55) Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt - [`259c72e`](https://github.com/netbirdio/netbird/commit/259c72eb37410b50916729dc1766888018ea4386) Update go.mod - [`5470e05`](https://github.com/netbirdio/netbird/commit/5470e052870b0256ce2f23136e597c877f585049) Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt - [`30272f8`](https://github.com/netbirdio/netbird/commit/30272f85feb59600fe7e463c4c17bde06e802567) Add allowExtraDNSLabels parameter to CreateSetupKey and related functions - [`90c0d27`](https://github.com/netbirdio/netbird/commit/90c0d270b3c44e40337cd051b76f78e0e80662a2) Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt - [`229b433`](https://github.com/netbirdio/netbird/commit/229b43345f4e051dc75a6133641511aa41e4f67c) Add AllowExtraDNSLabels field to Peer and update related logic - [`e0fa038`](https://github.com/netbirdio/netbird/commit/e0fa0388f3d91c2db659bfee3ebab81e5a2da301) Merge branch 'feature/dns-labels' into feat/extra-dns-labels-mgmt ### 📊 Changes **19 files changed** (+313 additions, -72 deletions) <details> <summary>View changed files</summary> 📝 `management/domain/validate.go` (+25 -0) 📝 `management/domain/validate_test.go` (+109 -0) 📝 `management/server/account.go` (+1 -1) 📝 `management/server/account_test.go` (+3 -3) 📝 `management/server/grpcserver.go` (+1 -0) 📝 `management/server/http/api/openapi.yml` (+16 -0) 📝 `management/server/http/api/types.gen.go` (+18 -0) 📝 `management/server/http/handlers/peers/peers_handler.go` (+10 -0) 📝 `management/server/http/handlers/setup_keys/setupkeys_handler.go` (+22 -15) 📝 `management/server/http/handlers/setup_keys/setupkeys_handler_test.go` (+3 -2) 📝 `management/server/management_proto_test.go` (+1 -1) 📝 `management/server/mock_server/account_mock.go` (+3 -2) 📝 `management/server/peer.go` (+26 -0) 📝 `management/server/peer/peer.go` (+7 -0) 📝 `management/server/peer_test.go` (+3 -3) 📝 `management/server/setupkey.go` (+2 -2) 📝 `management/server/setupkey_test.go` (+12 -12) 📝 `management/server/types/account.go` (+16 -1) 📝 `management/server/types/setupkey.go` (+35 -30) </details> ### 📄 Description ## Describe your changes Support Extra DNS Labels for Peer Addressing ## Issue ticket number and link ### 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 05:06:40 -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#19965