[PR #3718] [MERGED] [client] Feature/lazy connection backward #23511

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3718
Author: @pappz
Created: 4/23/2025
Status: Merged
Merged: 4/25/2025
Merged by: @pappz

Base: feature/lazy-connectionHead: feature/lazy-connection-backward


📝 Commits (8)

📊 Changes

10 files changed (+412 additions, -271 deletions)

View changed files

📝 client/internal/conn_mgr.go (+28 -25)
📝 client/internal/engine.go (+14 -4)
client/internal/lazyconn/env.go (+26 -0)
📝 client/internal/lazyconn/manager/manager.go (+15 -13)
client/internal/lazyconn/support.go (+51 -0)
client/internal/lazyconn/support_test.go (+31 -0)
📝 client/internal/peer/conn.go (+6 -0)
📝 management/proto/management.pb.go (+235 -225)
📝 management/proto/management.proto (+1 -0)
📝 management/server/grpcserver.go (+5 -4)

📄 Description

Describe your changes

Check the minimum version number of the remote peer, and if the remote peer does not support the lazy connection, then always use a permanent connection.

Keep in mind just because the user updated the agent, the management server will not push a new network map with the new version information.

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)
  • 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/3718 **Author:** [@pappz](https://github.com/pappz) **Created:** 4/23/2025 **Status:** ✅ Merged **Merged:** 4/25/2025 **Merged by:** [@pappz](https://github.com/pappz) **Base:** `feature/lazy-connection` ← **Head:** `feature/lazy-connection-backward` --- ### 📝 Commits (8) - [`49d5cc2`](https://github.com/netbirdio/netbird/commit/49d5cc2bbd0fffabeb8f6b120587f88c70608a20) Add backward compatibility handling - [`782a346`](https://github.com/netbirdio/netbird/commit/782a346c2fda698169c7a75443700729182c0755) Handle version changes - [`715262c`](https://github.com/netbirdio/netbird/commit/715262c21e05e7904e8f615ba2c6154ba13ab2fe) Move envs - [`a0b15e6`](https://github.com/netbirdio/netbird/commit/a0b15e6f161e97ad0b80159718928518c6c6158c) Print out version info - [`dda290c`](https://github.com/netbirdio/netbird/commit/dda290cc1f3d49143872329f8ebb22a65a77d8a0) Add todo - [`240afd1`](https://github.com/netbirdio/netbird/commit/240afd15ccaf773da61fb60717d7fe506a00387f) Fix grammar - [`bb6254d`](https://github.com/netbirdio/netbird/commit/bb6254d22bbc2e965eb65b228a5f24908f253192) Merge branch 'feature/lazy-connection' into feature/lazy-connection-backward - [`23348a1`](https://github.com/netbirdio/netbird/commit/23348a19fba986027bf74325c286a708752ee71a) Use strconv for env var ### 📊 Changes **10 files changed** (+412 additions, -271 deletions) <details> <summary>View changed files</summary> 📝 `client/internal/conn_mgr.go` (+28 -25) 📝 `client/internal/engine.go` (+14 -4) ➕ `client/internal/lazyconn/env.go` (+26 -0) 📝 `client/internal/lazyconn/manager/manager.go` (+15 -13) ➕ `client/internal/lazyconn/support.go` (+51 -0) ➕ `client/internal/lazyconn/support_test.go` (+31 -0) 📝 `client/internal/peer/conn.go` (+6 -0) 📝 `management/proto/management.pb.go` (+235 -225) 📝 `management/proto/management.proto` (+1 -0) 📝 `management/server/grpcserver.go` (+5 -4) </details> ### 📄 Description ## Describe your changes Check the minimum version number of the remote peer, and if the remote peer does not support the lazy connection, then always use a permanent connection. Keep in mind just because the user updated the agent, the management server will not push a new network map with the new version information. ## Issue ticket number and link ## Stack <!-- 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) - [ ] 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 06:07:18 -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#23511