[PR #4439] [MERGED] [client] Debug information for connection #25491

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/4439
Author: @pappz
Created: 9/3/2025
Status: Merged
Merged: 9/8/2025
Merged by: @pappz

Base: mainHead: debug-slow-conn


📝 Commits (8)

  • 1d83fcc Improve logging for management service connection and system info gathering
  • fd13247 Explicit print out the first success handshake time
  • f86a7f7 Fix logging library
  • b2548a4 Add verbose sys info gathering information
  • 23977c6 Add debug ticker
  • 59a1c87 Revert "Add debug ticker"
  • 068371c Print out in a human-readable format
  • 5861b20 Merge branch 'main' into debug-slow-conn

📊 Changes

5 files changed (+22 additions, -6 deletions)

View changed files

📝 client/iface/configurer/usp.go (+7 -0)
📝 client/internal/engine.go (+1 -2)
📝 client/internal/peer/wg_watcher.go (+10 -3)
📝 client/system/info.go (+4 -0)
📝 client/system/info_windows.go (+0 -1)

📄 Description

Describe your changes

Improve logging

  • Print the exact time when the first WireGuard handshake occurs
  • Print the steps for gathering system 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)

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/4439 **Author:** [@pappz](https://github.com/pappz) **Created:** 9/3/2025 **Status:** ✅ Merged **Merged:** 9/8/2025 **Merged by:** [@pappz](https://github.com/pappz) **Base:** `main` ← **Head:** `debug-slow-conn` --- ### 📝 Commits (8) - [`1d83fcc`](https://github.com/netbirdio/netbird/commit/1d83fccd9c004c52abdd911bb53bb886c87ae1af) Improve logging for management service connection and system info gathering - [`fd13247`](https://github.com/netbirdio/netbird/commit/fd13247d66b917f9ab5c339a13769599065e9806) Explicit print out the first success handshake time - [`f86a7f7`](https://github.com/netbirdio/netbird/commit/f86a7f745e9c7446a7860607fc79bb874f6b4ee3) Fix logging library - [`b2548a4`](https://github.com/netbirdio/netbird/commit/b2548a4037761efccbe7865c813d20bc0c410aba) Add verbose sys info gathering information - [`23977c6`](https://github.com/netbirdio/netbird/commit/23977c64098b0aa13620fe4516f9d67a8bf123f4) Add debug ticker - [`59a1c87`](https://github.com/netbirdio/netbird/commit/59a1c87e097210ea5229ae72a014b00ec3ca51b8) Revert "Add debug ticker" - [`068371c`](https://github.com/netbirdio/netbird/commit/068371c5a45416a67ef47043cb1ecde65fa0fc96) Print out in a human-readable format - [`5861b20`](https://github.com/netbirdio/netbird/commit/5861b20dd0bdfd1c32ec7cf6ecdb746c0a262187) Merge branch 'main' into debug-slow-conn ### 📊 Changes **5 files changed** (+22 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `client/iface/configurer/usp.go` (+7 -0) 📝 `client/internal/engine.go` (+1 -2) 📝 `client/internal/peer/wg_watcher.go` (+10 -3) 📝 `client/system/info.go` (+4 -0) 📝 `client/system/info_windows.go` (+0 -1) </details> ### 📄 Description ## Describe your changes Improve logging - Print the exact time when the first WireGuard handshake occurs - Print the steps for gathering system information ## 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:04 -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#25491