[PR #4692] [MERGED] [client, management] Move client-imported GPL code to separate package #26050

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4692
Author: @lixmal
Created: 10/23/2025
Status: Merged
Merged: 11/5/2025
Merged by: @lixmal

Base: ssh-rewriteHead: move-licensed-code


📝 Commits (3)

  • 4fd6437 Move client-imported GPL code to separate package
  • b5d75ad Go fmt everything
  • 9378b6b Merge branch 'ssh-rewrite' into move-licensed-code

📊 Changes

59 files changed (+397 additions, -368 deletions)

View changed files

📝 .github/workflows/check-license-dependencies.yml (+8 -7)
📝 client/internal/profilemanager/config_test.go (+4 -4)
📝 client/ssh/proxy/proxy_test.go (+1 -1)
📝 client/ssh/server/jwt_test.go (+1 -1)
📝 client/ssh/server/server.go (+4 -4)
📝 management/server/account.go (+11 -9)
📝 management/server/account/manager.go (+6 -5)
📝 management/server/account_test.go (+26 -26)
📝 management/server/auth/manager.go (+9 -8)
📝 management/server/auth/manager_mock.go (+8 -7)
📝 management/server/auth/manager_test.go (+6 -6)
📝 management/server/context/auth.go (+8 -30)
📝 management/server/http/handlers/accounts/accounts_handler_test.go (+2 -1)
📝 management/server/http/handlers/dns/dns_settings_handler.go (+1 -1)
📝 management/server/http/handlers/dns/dns_settings_handler_test.go (+3 -2)
📝 management/server/http/handlers/dns/nameservers_handler_test.go (+2 -1)
📝 management/server/http/handlers/events/events_handler_test.go (+3 -2)
📝 management/server/http/handlers/groups/groups_handler.go (+1 -1)
📝 management/server/http/handlers/groups/groups_handler_test.go (+7 -6)
📝 management/server/http/handlers/networks/handler.go (+3 -3)

...and 39 more files

📄 Description

Describe your changes

Stack

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)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__


🔄 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/4692 **Author:** [@lixmal](https://github.com/lixmal) **Created:** 10/23/2025 **Status:** ✅ Merged **Merged:** 11/5/2025 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `ssh-rewrite` ← **Head:** `move-licensed-code` --- ### 📝 Commits (3) - [`4fd6437`](https://github.com/netbirdio/netbird/commit/4fd64379da5a66556b7f2a6df197835e27220f9c) Move client-imported GPL code to separate package - [`b5d75ad`](https://github.com/netbirdio/netbird/commit/b5d75ad9c46761c9309a51773ccd9cf469a432dc) Go fmt everything - [`9378b6b`](https://github.com/netbirdio/netbird/commit/9378b6b0a372acbaba2ace2d5f7c11ab82a2260e) Merge branch 'ssh-rewrite' into move-licensed-code ### 📊 Changes **59 files changed** (+397 additions, -368 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/check-license-dependencies.yml` (+8 -7) 📝 `client/internal/profilemanager/config_test.go` (+4 -4) 📝 `client/ssh/proxy/proxy_test.go` (+1 -1) 📝 `client/ssh/server/jwt_test.go` (+1 -1) 📝 `client/ssh/server/server.go` (+4 -4) 📝 `management/server/account.go` (+11 -9) 📝 `management/server/account/manager.go` (+6 -5) 📝 `management/server/account_test.go` (+26 -26) 📝 `management/server/auth/manager.go` (+9 -8) 📝 `management/server/auth/manager_mock.go` (+8 -7) 📝 `management/server/auth/manager_test.go` (+6 -6) 📝 `management/server/context/auth.go` (+8 -30) 📝 `management/server/http/handlers/accounts/accounts_handler_test.go` (+2 -1) 📝 `management/server/http/handlers/dns/dns_settings_handler.go` (+1 -1) 📝 `management/server/http/handlers/dns/dns_settings_handler_test.go` (+3 -2) 📝 `management/server/http/handlers/dns/nameservers_handler_test.go` (+2 -1) 📝 `management/server/http/handlers/events/events_handler_test.go` (+3 -2) 📝 `management/server/http/handlers/groups/groups_handler.go` (+1 -1) 📝 `management/server/http/handlers/groups/groups_handler_test.go` (+7 -6) 📝 `management/server/http/handlers/networks/handler.go` (+3 -3) _...and 39 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [ ] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [x] 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) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ --- <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:06:49 -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#26050