[PR #3675] [CLOSED] [client] Feature/lazy connection backward #18191

Open
opened 2026-08-05 04:08:13 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3675
Author: @pappz
Created: 4/14/2025
Status: Closed

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


📝 Commits (10+)

  • 34d86c5 [management] Sync account peers on network router group changes (#3573)
  • fceb3ca [client] fix route handling for local peer state (#3586)
  • a4f04f5 [management] fix extend call and move config to types (#3575)
  • 49d5cc2 Add backward compatibility handling
  • 101cce2 [client] Ensure status recorder is always initialized (#3588)
  • 50f5cc4 [management] Fix extended config when nil (#3593)
  • 6124e3b [client] Disable systemd-resolved default route explicitly on match domains only (#3584)
  • 29a6e5b [client] Stop flow grpc receiver properly (#3596)
  • ed56470 [client] Prevent calling the onDisconnected callback in incorrect state (#3582)
  • 21464ac [client] Fix close WireGuard watcher (#3598)

📊 Changes

195 files changed (+7541 additions, -3088 deletions)

View changed files

.git-branches.toml (+27 -0)
📝 .github/pull_request_template.md (+4 -0)
.github/workflows/git-town.yml (+21 -0)
📝 .github/workflows/golang-test-freebsd.yml (+8 -2)
📝 .github/workflows/golangci-lint.yml (+1 -1)
📝 .github/workflows/test-infrastructure-files.yml (+1 -0)
📝 README.md (+10 -10)
📝 client/cmd/debug.go (+34 -0)
client/cmd/debug_unix.go (+39 -0)
client/cmd/debug_windows.go (+126 -0)
📝 client/cmd/login.go (+21 -9)
📝 client/cmd/service_controller.go (+1 -0)
📝 client/cmd/testutil_test.go (+6 -4)
📝 client/cmd/up.go (+10 -1)
📝 client/firewall/iptables/router_linux.go (+67 -6)
📝 client/firewall/iptables/router_linux_test.go (+3 -1)
📝 client/firewall/nftables/acl_linux.go (+9 -6)
📝 client/firewall/nftables/router_linux.go (+119 -28)
📝 client/firewall/nftables/router_linux_test.go (+3 -3)
📝 client/firewall/uspfilter/conntrack/common_test.go (+1 -2)

...and 80 more files

📄 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/3675 **Author:** [@pappz](https://github.com/pappz) **Created:** 4/14/2025 **Status:** ❌ Closed **Base:** `feature/lazy-connection` ← **Head:** `feature/lazy-connection-backward` --- ### 📝 Commits (10+) - [`34d86c5`](https://github.com/netbirdio/netbird/commit/34d86c5ab84e941dd4ad8a53ded5edb0f05b0ea3) [management] Sync account peers on network router group changes (#3573) - [`fceb3ca`](https://github.com/netbirdio/netbird/commit/fceb3ca392ae620d3fff6440e75686a3fcb34b61) [client] fix route handling for local peer state (#3586) - [`a4f04f5`](https://github.com/netbirdio/netbird/commit/a4f04f557073c241a3b297d6f59be5ce6112ec73) [management] fix extend call and move config to types (#3575) - [`49d5cc2`](https://github.com/netbirdio/netbird/commit/49d5cc2bbd0fffabeb8f6b120587f88c70608a20) Add backward compatibility handling - [`101cce2`](https://github.com/netbirdio/netbird/commit/101cce27f25feb2d3ac43f180638c202f0874434) [client] Ensure status recorder is always initialized (#3588) - [`50f5cc4`](https://github.com/netbirdio/netbird/commit/50f5cc48cdf006acf211294e95a497708dacfbca) [management] Fix extended config when nil (#3593) - [`6124e3b`](https://github.com/netbirdio/netbird/commit/6124e3b93789b37b877e2132eac55b117da28fc0) [client] Disable systemd-resolved default route explicitly on match domains only (#3584) - [`29a6e5b`](https://github.com/netbirdio/netbird/commit/29a6e5be71fbd2ef4dac32550666cbb8e1dd9f17) [client] Stop flow grpc receiver properly (#3596) - [`ed56470`](https://github.com/netbirdio/netbird/commit/ed5647028a2d676cdc4f5a7adcd603f8d4b6f29a) [client] Prevent calling the onDisconnected callback in incorrect state (#3582) - [`21464ac`](https://github.com/netbirdio/netbird/commit/21464ac77013bdb47ea6933cbd0474c53bef96ad) [client] Fix close WireGuard watcher (#3598) ### 📊 Changes **195 files changed** (+7541 additions, -3088 deletions) <details> <summary>View changed files</summary> ➕ `.git-branches.toml` (+27 -0) 📝 `.github/pull_request_template.md` (+4 -0) ➕ `.github/workflows/git-town.yml` (+21 -0) 📝 `.github/workflows/golang-test-freebsd.yml` (+8 -2) 📝 `.github/workflows/golangci-lint.yml` (+1 -1) 📝 `.github/workflows/test-infrastructure-files.yml` (+1 -0) 📝 `README.md` (+10 -10) 📝 `client/cmd/debug.go` (+34 -0) ➕ `client/cmd/debug_unix.go` (+39 -0) ➕ `client/cmd/debug_windows.go` (+126 -0) 📝 `client/cmd/login.go` (+21 -9) 📝 `client/cmd/service_controller.go` (+1 -0) 📝 `client/cmd/testutil_test.go` (+6 -4) 📝 `client/cmd/up.go` (+10 -1) 📝 `client/firewall/iptables/router_linux.go` (+67 -6) 📝 `client/firewall/iptables/router_linux_test.go` (+3 -1) 📝 `client/firewall/nftables/acl_linux.go` (+9 -6) 📝 `client/firewall/nftables/router_linux.go` (+119 -28) 📝 `client/firewall/nftables/router_linux_test.go` (+3 -3) 📝 `client/firewall/uspfilter/conntrack/common_test.go` (+1 -2) _...and 80 more files_ </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 04:08:13 -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#18191