[PR #1567] [MERGED] Add account usage logic #15563

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/1567
Author: @lixmal
Created: 2/12/2024
Status: Merged
Merged: 2/22/2024
Merged by: @lixmal

Base: mainHead: feature/account-usage


📝 Commits (10+)

📊 Changes

21 files changed (+576 additions, -30 deletions)

View changed files

📝 go.mod (+2 -2)
📝 go.sum (+4 -4)
📝 management/cmd/management.go (+4 -3)
📝 management/server/account.go (+23 -2)
📝 management/server/activity/codes.go (+11 -4)
📝 management/server/activity/event.go (+7 -1)
📝 management/server/event.go (+1 -2)
📝 management/server/file_store.go (+39 -0)
📝 management/server/file_store_test.go (+30 -0)
📝 management/server/http/handler.go (+11 -3)
📝 management/server/http/middleware/access_control.go (+6 -1)
📝 management/server/http/middleware/auth_middleware.go (+6 -0)
📝 management/server/http/middleware/auth_middleware_test.go (+35 -2)
management/server/http/middleware/bypass/bypass.go (+39 -0)
management/server/http/middleware/bypass/bypass_test.go (+103 -0)
📝 management/server/mock_server/account_mock.go (+13 -3)
📝 management/server/sqlite_store.go (+49 -2)
📝 management/server/sqlite_store_test.go (+27 -0)
📝 management/server/status/error.go (+3 -1)
📝 management/server/store.go (+2 -0)

...and 1 more files

📄 Description

Describe your changes

  • Updated management-integrations
  • Implemented bypass middleware logic
  • Added logic to calculate active users

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/1567 **Author:** [@lixmal](https://github.com/lixmal) **Created:** 2/12/2024 **Status:** ✅ Merged **Merged:** 2/22/2024 **Merged by:** [@lixmal](https://github.com/lixmal) **Base:** `main` ← **Head:** `feature/account-usage` --- ### 📝 Commits (10+) - [`b8c81be`](https://github.com/netbirdio/netbird/commit/b8c81beb324cb4ba87038760c41331d9b18521ef) Add account usage endpoint - [`e713638`](https://github.com/netbirdio/netbird/commit/e7136381e3f61fe1e9a4469aab62f3fdb2aaa867) Fix test - [`3bb3f10`](https://github.com/netbirdio/netbird/commit/3bb3f1052d47086f531b78b4276701447cfd3f89) Make event methods accept an interface - [`f019182`](https://github.com/netbirdio/netbird/commit/f01918246c67bc88966eef73398b4d8bc1e8707b) Allow to add activity codes - [`9f6f048`](https://github.com/netbirdio/netbird/commit/9f6f048f4c82bcbe32e83bac30b7864d5e066d8c) Remove unused mock - [`063a93e`](https://github.com/netbirdio/netbird/commit/063a93e84500038e38a129359d7554941e0d2caf) Use t.Cleanup instead of defer - [`e41d74c`](https://github.com/netbirdio/netbird/commit/e41d74c8614d12aae7504d3ce771cad8c2f6586f) Close sqlite db - [`49087d1`](https://github.com/netbirdio/netbird/commit/49087d14aecd2280eadf7c7927fc816741d5158e) Merge branch 'main' into feature/account-usage - [`2570afa`](https://github.com/netbirdio/netbird/commit/2570afacc6486252aee72d176fa7f94e213509e4) Lock account on usage retrieval - [`7422bf0`](https://github.com/netbirdio/netbird/commit/7422bf07bd9dcd90a6bbf2e5819ef91e3eb0bdf4) Use time.Compare ### 📊 Changes **21 files changed** (+576 additions, -30 deletions) <details> <summary>View changed files</summary> 📝 `go.mod` (+2 -2) 📝 `go.sum` (+4 -4) 📝 `management/cmd/management.go` (+4 -3) 📝 `management/server/account.go` (+23 -2) 📝 `management/server/activity/codes.go` (+11 -4) 📝 `management/server/activity/event.go` (+7 -1) 📝 `management/server/event.go` (+1 -2) 📝 `management/server/file_store.go` (+39 -0) 📝 `management/server/file_store_test.go` (+30 -0) 📝 `management/server/http/handler.go` (+11 -3) 📝 `management/server/http/middleware/access_control.go` (+6 -1) 📝 `management/server/http/middleware/auth_middleware.go` (+6 -0) 📝 `management/server/http/middleware/auth_middleware_test.go` (+35 -2) ➕ `management/server/http/middleware/bypass/bypass.go` (+39 -0) ➕ `management/server/http/middleware/bypass/bypass_test.go` (+103 -0) 📝 `management/server/mock_server/account_mock.go` (+13 -3) 📝 `management/server/sqlite_store.go` (+49 -2) 📝 `management/server/sqlite_store_test.go` (+27 -0) 📝 `management/server/status/error.go` (+3 -1) 📝 `management/server/store.go` (+2 -0) _...and 1 more files_ </details> ### 📄 Description ## Describe your changes - Updated management-integrations - Implemented bypass middleware logic - Added logic to calculate active users ## Issue ticket number and link ### Checklist - [ ] Is it a bug fix - [x] Is a typo/documentation fix - [x] Is a feature enhancement - [x] It is a refactor - [x] 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 03:07:57 -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#15563