[PR #2209] [MERGED] Add context to throughout the project and update logging #18929

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/2209
Author: @pascal-fischer
Created: 6/27/2024
Status: Merged
Merged: 7/3/2024
Merged by: @mlsmaycon

Base: mainHead: feature/add-context-throughout-management


📝 Commits (9)

  • 4234cc2 Context creation and logging in management (#2144)
  • 687a1a9 Propagate context through all methods in management (#2145)
  • fb597a4 propagate context to management
  • 39d67d2 Merge branch 'refs/heads/main' into feature/add-context-throughout-management
  • f5b3292 update after merge
  • f7f0fb5 fix tests
  • 251fa50 fix tests
  • 354132a go mod tidy
  • ea26c41 Update management-integrations

📊 Changes

127 files changed (+2936 additions, -2642 deletions)

View changed files

📝 client/cmd/testutil_test.go (+4 -4)
📝 client/internal/engine_test.go (+5 -5)
📝 client/server/server_test.go (+4 -4)
📝 formatter/hook.go (+72 -0)
📝 formatter/set.go (+10 -1)
📝 go.mod (+1 -2)
📝 go.sum (+2 -5)
📝 management/client/client_test.go (+4 -4)
📝 management/cmd/management.go (+100 -60)
📝 management/cmd/migration_up.go (+10 -4)
📝 management/server/account.go (+265 -257)
📝 management/server/account_test.go (+101 -100)
📝 management/server/activity/sqlite/sqlite.go (+12 -11)
📝 management/server/activity/sqlite/sqlite_test.go (+6 -5)
📝 management/server/activity/store.go (+10 -7)
management/server/context/keys.go (+8 -0)
📝 management/server/dns.go (+17 -16)
📝 management/server/dns_test.go (+20 -19)
📝 management/server/ephemeral.go (+25 -18)
📝 management/server/ephemeral_test.go (+14 -13)

...and 80 more files

📄 Description

Describe your changes

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/2209 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 6/27/2024 **Status:** ✅ Merged **Merged:** 7/3/2024 **Merged by:** [@mlsmaycon](https://github.com/mlsmaycon) **Base:** `main` ← **Head:** `feature/add-context-throughout-management` --- ### 📝 Commits (9) - [`4234cc2`](https://github.com/netbirdio/netbird/commit/4234cc26c44a5719d696446327eaa78e518d84fd) Context creation and logging in management (#2144) - [`687a1a9`](https://github.com/netbirdio/netbird/commit/687a1a9910802b2d4416fcfe3e64333b2f448efc) Propagate context through all methods in management (#2145) - [`fb597a4`](https://github.com/netbirdio/netbird/commit/fb597a40bd18f8c7f825483c073f034a6bcfb083) propagate context to management - [`39d67d2`](https://github.com/netbirdio/netbird/commit/39d67d2c5808048a7b772052023cb0a4e9080fb4) Merge branch 'refs/heads/main' into feature/add-context-throughout-management - [`f5b3292`](https://github.com/netbirdio/netbird/commit/f5b32928b7a78e343fbf1c1bf810d78bfd5b0583) update after merge - [`f7f0fb5`](https://github.com/netbirdio/netbird/commit/f7f0fb5dd8af43fab428b39738d568cb4efdafea) fix tests - [`251fa50`](https://github.com/netbirdio/netbird/commit/251fa50541687e4cf3b4a5a3a4f982d3cb560db5) fix tests - [`354132a`](https://github.com/netbirdio/netbird/commit/354132a7c0e13a12245e2dd9e967c48d0a919d67) go mod tidy - [`ea26c41`](https://github.com/netbirdio/netbird/commit/ea26c41893cd1dd1c936fda1a55fdf501910bdf1) Update management-integrations ### 📊 Changes **127 files changed** (+2936 additions, -2642 deletions) <details> <summary>View changed files</summary> 📝 `client/cmd/testutil_test.go` (+4 -4) 📝 `client/internal/engine_test.go` (+5 -5) 📝 `client/server/server_test.go` (+4 -4) 📝 `formatter/hook.go` (+72 -0) 📝 `formatter/set.go` (+10 -1) 📝 `go.mod` (+1 -2) 📝 `go.sum` (+2 -5) 📝 `management/client/client_test.go` (+4 -4) 📝 `management/cmd/management.go` (+100 -60) 📝 `management/cmd/migration_up.go` (+10 -4) 📝 `management/server/account.go` (+265 -257) 📝 `management/server/account_test.go` (+101 -100) 📝 `management/server/activity/sqlite/sqlite.go` (+12 -11) 📝 `management/server/activity/sqlite/sqlite_test.go` (+6 -5) 📝 `management/server/activity/store.go` (+10 -7) ➕ `management/server/context/keys.go` (+8 -0) 📝 `management/server/dns.go` (+17 -16) 📝 `management/server/dns_test.go` (+20 -19) 📝 `management/server/ephemeral.go` (+25 -18) 📝 `management/server/ephemeral_test.go` (+14 -13) _...and 80 more files_ </details> ### 📄 Description ## Describe your changes ## Issue ticket number and link ### 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 --- <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:09:05 -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#18929