[PR #193] [MERGED] Support new Management service protocol (NetworkMap) #12273

Closed
opened 2026-08-05 01:32:46 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/193
Author: @braginini
Created: 1/16/2022
Status: Merged
Merged: 1/18/2022
Merged by: @braginini

Base: mainHead: client-support-new-protocol


📝 Commits (10+)

  • 124fa60 feature: support new management service protocol
  • 38fb540 chore: add more logging to track networkmap serial
  • 5c68161 refactor: organize peer update code in engine
  • 94be4e6 refactor: organize peer update code in engine
  • b6b2848 chore: fix lint issues
  • ad42ac2 Merge remote-tracking branch 'origin/main' into client-support-new-protocol
  • 726b24f Merge remote-tracking branch 'origin/main' into client-support-new-protocol
  • c0e9ada refactor: extract Signal client interface
  • 5086fde test: add signal client mock
  • b4a912f refactor: introduce Management Service client interface

📊 Changes

14 files changed (+1102 additions, -651 deletions)

View changed files

📝 client/cmd/login.go (+2 -2)
📝 client/cmd/up.go (+2 -2)
📝 client/internal/engine.go (+108 -60)
📝 client/internal/engine_test.go (+241 -13)
📝 client/internal/peer/conn.go (+4 -0)
📝 management/client/client.go (+6 -244)
📝 management/client/client_test.go (+1 -1)
management/client/grpc.go (+253 -0)
management/client/mock.go (+49 -0)
📝 signal/client/client.go (+10 -327)
📝 signal/client/client_test.go (+2 -2)
signal/client/grpc.go (+336 -0)
signal/client/mock.go (+72 -0)
util/common.go (+16 -0)

📄 Description

No description provided


🔄 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/193 **Author:** [@braginini](https://github.com/braginini) **Created:** 1/16/2022 **Status:** ✅ Merged **Merged:** 1/18/2022 **Merged by:** [@braginini](https://github.com/braginini) **Base:** `main` ← **Head:** `client-support-new-protocol` --- ### 📝 Commits (10+) - [`124fa60`](https://github.com/netbirdio/netbird/commit/124fa602c8afeba10a9fbd5a8721cd8ac9c5ed88) feature: support new management service protocol - [`38fb540`](https://github.com/netbirdio/netbird/commit/38fb54063b8cbfc2bafd9d76b478dc0c4f5856e9) chore: add more logging to track networkmap serial - [`5c68161`](https://github.com/netbirdio/netbird/commit/5c68161dcdc22d5bc694b63e4edecfa723dd5b16) refactor: organize peer update code in engine - [`94be4e6`](https://github.com/netbirdio/netbird/commit/94be4e65400f8ee9291b410c35c3e0f4d4621106) refactor: organize peer update code in engine - [`b6b2848`](https://github.com/netbirdio/netbird/commit/b6b2848d120e7f63e35eb0a387e5ab1fdb9049dd) chore: fix lint issues - [`ad42ac2`](https://github.com/netbirdio/netbird/commit/ad42ac28695124a9d94112517607c7ec380d7dd1) Merge remote-tracking branch 'origin/main' into client-support-new-protocol - [`726b24f`](https://github.com/netbirdio/netbird/commit/726b24f8af1fa7737b5e8fe562f8a8f6a63d50e1) Merge remote-tracking branch 'origin/main' into client-support-new-protocol - [`c0e9ada`](https://github.com/netbirdio/netbird/commit/c0e9adaf7ffa13056e0ee0162835e752d07f2d81) refactor: extract Signal client interface - [`5086fde`](https://github.com/netbirdio/netbird/commit/5086fde516e30f08d3aa3353427c8bb22d8237e3) test: add signal client mock - [`b4a912f`](https://github.com/netbirdio/netbird/commit/b4a912f6eeacf3c9af7e8f71d60e644fff98c618) refactor: introduce Management Service client interface ### 📊 Changes **14 files changed** (+1102 additions, -651 deletions) <details> <summary>View changed files</summary> 📝 `client/cmd/login.go` (+2 -2) 📝 `client/cmd/up.go` (+2 -2) 📝 `client/internal/engine.go` (+108 -60) 📝 `client/internal/engine_test.go` (+241 -13) 📝 `client/internal/peer/conn.go` (+4 -0) 📝 `management/client/client.go` (+6 -244) 📝 `management/client/client_test.go` (+1 -1) ➕ `management/client/grpc.go` (+253 -0) ➕ `management/client/mock.go` (+49 -0) 📝 `signal/client/client.go` (+10 -327) 📝 `signal/client/client_test.go` (+2 -2) ➕ `signal/client/grpc.go` (+336 -0) ➕ `signal/client/mock.go` (+72 -0) ➕ `util/common.go` (+16 -0) </details> ### 📄 Description _No description provided_ --- <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 01:32:46 -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#12273