[PR #460] [MERGED] Add SetupKey auto-groups property #12648

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/460
Author: @braginini
Created: 9/9/2022
Status: Merged
Merged: 9/11/2022
Merged by: @braginini

Base: mainHead: feature/setupkey-autotagging


📝 Commits (10+)

  • 9bfb210 Add Groups to SetupKey API requests and BAO
  • 207fcfc Save SetupKey.AutoGroups in the store
  • 6eb99ed Refactor: Move SetupKey related AM code to setupkey file
  • ab958c1 Add SetupKey patch operations
  • 4bf8bb8 Unify Update Setup Key method
  • 2474505 Save setup key
  • 5344d43 Add UpdatedAt property ot SetupKey
  • 0ef866d Fix Codacy issues
  • b7983b2 Fix lint issues
  • d5a1a5d Add SetupKey new props test validation

📊 Changes

11 files changed (+765 additions, -257 deletions)

View changed files

📝 management/server/account.go (+6 -101)
📝 management/server/http/api/openapi.yml (+17 -0)
📝 management/server/http/api/types.gen.go (+9 -0)
📝 management/server/http/handler.go (+4 -5)
📝 management/server/http/routes.go (+5 -0)
📝 management/server/http/routes_test.go (+6 -3)
📝 management/server/http/setupkeys.go (+107 -84)
management/server/http/setupkeys_test.go (+222 -0)
📝 management/server/mock_server/account_mock.go (+37 -31)
📝 management/server/setupkey.go (+191 -22)
📝 management/server/setupkey_test.go (+161 -11)

📄 Description

Extend Management API to add support for setup keys auto-tagging feature that allows automatic groups assignment to peers that used the key.
This PR just adds the property to the setup key.

Works as the first feature MVP for this request https://github.com/netbirdio/netbird/issues/458#issue-1366523452


🔄 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/460 **Author:** [@braginini](https://github.com/braginini) **Created:** 9/9/2022 **Status:** ✅ Merged **Merged:** 9/11/2022 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `feature/setupkey-autotagging` --- ### 📝 Commits (10+) - [`9bfb210`](https://github.com/netbirdio/netbird/commit/9bfb2105504a474ecdc8621bbd572060886e706c) Add Groups to SetupKey API requests and BAO - [`207fcfc`](https://github.com/netbirdio/netbird/commit/207fcfc6c13fc8969f607f903e504e223ea2371a) Save SetupKey.AutoGroups in the store - [`6eb99ed`](https://github.com/netbirdio/netbird/commit/6eb99ed96faa58bc74b18c5f62b5f96204da8250) Refactor: Move SetupKey related AM code to setupkey file - [`ab958c1`](https://github.com/netbirdio/netbird/commit/ab958c114abf99f779ffd78f160fb9cf6c5e420a) Add SetupKey patch operations - [`4bf8bb8`](https://github.com/netbirdio/netbird/commit/4bf8bb87d8a2f8038cf7340aa4f404e2b060b3ae) Unify Update Setup Key method - [`2474505`](https://github.com/netbirdio/netbird/commit/24745056697d7967d872be4e4bfef40e2e14319d) Save setup key - [`5344d43`](https://github.com/netbirdio/netbird/commit/5344d434df03671df29097a8e8440f71d917808f) Add UpdatedAt property ot SetupKey - [`0ef866d`](https://github.com/netbirdio/netbird/commit/0ef866d8afdc43e644734d60f0624109b6a9cdf1) Fix Codacy issues - [`b7983b2`](https://github.com/netbirdio/netbird/commit/b7983b221b3520296f24146ab496cff1093b1dc6) Fix lint issues - [`d5a1a5d`](https://github.com/netbirdio/netbird/commit/d5a1a5d7d20e6cb02d1161383636617e3ba68473) Add SetupKey new props test validation ### 📊 Changes **11 files changed** (+765 additions, -257 deletions) <details> <summary>View changed files</summary> 📝 `management/server/account.go` (+6 -101) 📝 `management/server/http/api/openapi.yml` (+17 -0) 📝 `management/server/http/api/types.gen.go` (+9 -0) 📝 `management/server/http/handler.go` (+4 -5) 📝 `management/server/http/routes.go` (+5 -0) 📝 `management/server/http/routes_test.go` (+6 -3) 📝 `management/server/http/setupkeys.go` (+107 -84) ➕ `management/server/http/setupkeys_test.go` (+222 -0) 📝 `management/server/mock_server/account_mock.go` (+37 -31) 📝 `management/server/setupkey.go` (+191 -22) 📝 `management/server/setupkey_test.go` (+161 -11) </details> ### 📄 Description Extend Management API to add support for setup keys auto-tagging feature that allows automatic groups assignment to peers that used the key. This PR just adds the property to the setup key. Works as the first feature MVP for this request https://github.com/netbirdio/netbird/issues/458#issue-1366523452 --- <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:21 -04:00
saavagebueno changed title from [PR #460] Add SetupKey auto-groups property to [PR #460] [MERGED] Add SetupKey auto-groups property 2026-08-05 03:06:37 -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#12648