[PR #5493] [MERGED] [client] allow setting DNSLabels on client embed #28033

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/5493
Author: @Wouter0100
Created: 3/3/2026
Status: Merged
Merged: 3/17/2026
Merged by: @lixmal

Base: mainHead: feature/implement-dnslabels-embed


📝 Commits (1)

  • e5e500a feat(client): allow setting DNSLabels on client embed

📊 Changes

1 file changed (+10 additions, -1 deletions)

View changed files

📝 client/embed/embed.go (+10 -1)

📄 Description

Describe your changes

This allow you to set the DNSLabels on the embedded client in Go.

n/a

Stack

n/a

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)

There's currently no much docs on the embed client, if needed - please let me know how you'd like to have it documented.

Summary by CodeRabbit

  • New Features

    • Added support for configuring additional DNS labels for peers via a new option; labels are parsed and included in generated client configuration to be applied at creation time.
  • Bug Fixes

    • Validation added for DNS labels during client creation; invalid labels now produce a clear error and prevent an invalid configuration from being applied.

🔄 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/5493 **Author:** [@Wouter0100](https://github.com/Wouter0100) **Created:** 3/3/2026 **Status:** ✅ Merged **Merged:** 3/17/2026 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `feature/implement-dnslabels-embed` --- ### 📝 Commits (1) - [`e5e500a`](https://github.com/netbirdio/netbird/commit/e5e500ad24ad953774ce315caf65b2abe25af7ba) feat(client): allow setting DNSLabels on client embed ### 📊 Changes **1 file changed** (+10 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `client/embed/embed.go` (+10 -1) </details> ### 📄 Description ## Describe your changes This allow you to set the DNSLabels on the embedded client in Go. ## Issue ticket number and link n/a ## Stack n/a <!-- 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) There's currently no much docs on the embed client, if needed - please let me know how you'd like to have it documented. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added support for configuring additional DNS labels for peers via a new option; labels are parsed and included in generated client configuration to be applied at creation time. * **Bug Fixes** * Validation added for DNS labels during client creation; invalid labels now produce a clear error and prevent an invalid configuration from being applied. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <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:09:34 -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#28033