[PR #776] [MERGED] feature/activity_events_for_pat #14226

Closed
opened 2026-08-05 02:09:38 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/776
Author: @pascal-fischer
Created: 3/31/2023
Status: Merged
Merged: 4/3/2023
Merged by: @pascal-fischer

Base: mainHead: feature/activity_events_for_pat


📝 Commits (10+)

  • db3a9f0 refactor jwt token validation and add PAT to middleware auth
  • 9b000b8 Merge branch 'feature/add_rest_endpoints_for_pat' into feature/add_pat_middleware
  • 4d7029d Merge branch 'main' into feature/add_pat_middleware
  • 6c8bb60 fix merge
  • e869882 fix merge
  • 2a79995 fix linter
  • 1343a3f add test + codacy
  • 454240c comments for codacy
  • e08af7f codacy
  • f273fe9 revert codacy

📊 Changes

17 files changed (+575 additions, -300 deletions)

View changed files

📝 management/cmd/management.go (+18 -6)
📝 management/server/account.go (+34 -0)
📝 management/server/account_test.go (+38 -0)
📝 management/server/activity/codes.go (+16 -0)
📝 management/server/grpcserver.go (+8 -9)
📝 management/server/http/handler.go (+10 -10)
📝 management/server/http/middleware/access_control.go (+16 -1)
management/server/http/middleware/auth_middleware.go (+175 -0)
management/server/http/middleware/auth_middleware_test.go (+123 -0)
management/server/http/middleware/jwt.go (+0 -254)
📝 management/server/http/util/util.go (+6 -2)
📝 management/server/jwtclaims/extractor.go (+10 -5)
📝 management/server/jwtclaims/extractor_test.go (+1 -1)
📝 management/server/jwtclaims/jwtValidator.go (+88 -5)
📝 management/server/mock_server/account_mock.go (+9 -0)
📝 management/server/user.go (+7 -0)
📝 management/server/user_test.go (+16 -7)

📄 Description

Describe your changes

Sending events for PAT creation and deletion.

https://github.com/netbirdio/netbird/issues/718

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/776 **Author:** [@pascal-fischer](https://github.com/pascal-fischer) **Created:** 3/31/2023 **Status:** ✅ Merged **Merged:** 4/3/2023 **Merged by:** [@pascal-fischer](https://github.com/pascal-fischer) **Base:** `main` ← **Head:** `feature/activity_events_for_pat` --- ### 📝 Commits (10+) - [`db3a9f0`](https://github.com/netbirdio/netbird/commit/db3a9f0aa2d294c4a010f70b60dc9226d7aaf7ee) refactor jwt token validation and add PAT to middleware auth - [`9b000b8`](https://github.com/netbirdio/netbird/commit/9b000b89d502be864bf7927a2ded6ad21bbee128) Merge branch 'feature/add_rest_endpoints_for_pat' into feature/add_pat_middleware - [`4d7029d`](https://github.com/netbirdio/netbird/commit/4d7029d80c64aab87229343bdbe2a34cb647d031) Merge branch 'main' into feature/add_pat_middleware - [`6c8bb60`](https://github.com/netbirdio/netbird/commit/6c8bb6063278a21b78915aa1e0ee9c3407861b91) fix merge - [`e869882`](https://github.com/netbirdio/netbird/commit/e869882da11bb9e77627219264ad323f1c7f43f6) fix merge - [`2a79995`](https://github.com/netbirdio/netbird/commit/2a799957064de411d4d962d94be418071ea601d1) fix linter - [`1343a3f`](https://github.com/netbirdio/netbird/commit/1343a3f00ebbdac6e4eb91fa66076986400daf40) add test + codacy - [`454240c`](https://github.com/netbirdio/netbird/commit/454240ca05f0e3ff106e4c596c79ff937aaca42c) comments for codacy - [`e08af7f`](https://github.com/netbirdio/netbird/commit/e08af7fcdff28219419e566c90045f7aee3827dc) codacy - [`f273fe9`](https://github.com/netbirdio/netbird/commit/f273fe9f519621140420e5bb9f8a200c5de899b3) revert codacy ### 📊 Changes **17 files changed** (+575 additions, -300 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/management.go` (+18 -6) 📝 `management/server/account.go` (+34 -0) 📝 `management/server/account_test.go` (+38 -0) 📝 `management/server/activity/codes.go` (+16 -0) 📝 `management/server/grpcserver.go` (+8 -9) 📝 `management/server/http/handler.go` (+10 -10) 📝 `management/server/http/middleware/access_control.go` (+16 -1) ➕ `management/server/http/middleware/auth_middleware.go` (+175 -0) ➕ `management/server/http/middleware/auth_middleware_test.go` (+123 -0) ➖ `management/server/http/middleware/jwt.go` (+0 -254) 📝 `management/server/http/util/util.go` (+6 -2) 📝 `management/server/jwtclaims/extractor.go` (+10 -5) 📝 `management/server/jwtclaims/extractor_test.go` (+1 -1) 📝 `management/server/jwtclaims/jwtValidator.go` (+88 -5) 📝 `management/server/mock_server/account_mock.go` (+9 -0) 📝 `management/server/user.go` (+7 -0) 📝 `management/server/user_test.go` (+16 -7) </details> ### 📄 Description ## Describe your changes Sending events for PAT creation and deletion. ## Issue ticket number and link https://github.com/netbirdio/netbird/issues/718 ### 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 02:09:38 -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#14226