[PR #3686] [CLOSED] [management] user info with role permissions #23378

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

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/3686
Author: @pnmcosta
Created: 4/15/2025
Status: Closed

Base: feature/additional-rolesHead: feature/user-info-with-role-permissions


📝 Commits (10+)

  • 5e989df [management] user info with role permissions
  • f6d6cd6 Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions
  • 467b0f9 better naming of exposed user permissions
  • 96b22d5 Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions
  • ca356f1 fix rest test
  • 974f297 Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions
  • ab391ed [management] restrict dashboard only to restrictable roles
  • e42d4c0 fix rest test
  • 82ab78d make user permissions optional
  • a01a817 Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions

📊 Changes

14 files changed (+398 additions, -258 deletions)

View changed files

📝 management/client/rest/users_test.go (+2 -5)
📝 management/server/account/manager.go (+2 -1)
📝 management/server/http/api/openapi.yml (+23 -5)
📝 management/server/http/api/types.gen.go (+3 -12)
📝 management/server/http/handlers/users/users_handler.go (+22 -4)
📝 management/server/http/handlers/users/users_handler_test.go (+154 -39)
📝 management/server/mock_server/account_mock.go (+3 -2)
📝 management/server/permissions/manager.go (+21 -0)
📝 management/server/permissions/manager_mock.go (+15 -0)
📝 management/server/permissions/modules/module.go (+16 -0)
📝 management/server/types/user.go (+6 -20)
📝 management/server/user.go (+24 -32)
📝 management/server/user_test.go (+93 -138)
management/server/users/user.go (+14 -0)

📄 Description

Describe your changes

Get current user response info with role permissions by module and operation

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/3686 **Author:** [@pnmcosta](https://github.com/pnmcosta) **Created:** 4/15/2025 **Status:** ❌ Closed **Base:** `feature/additional-roles` ← **Head:** `feature/user-info-with-role-permissions` --- ### 📝 Commits (10+) - [`5e989df`](https://github.com/netbirdio/netbird/commit/5e989dff4ff044994f71f73215208bd3bb0429ec) [management] user info with role permissions - [`f6d6cd6`](https://github.com/netbirdio/netbird/commit/f6d6cd61a5fed9b6036f39a16c224f40257968fd) Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions - [`467b0f9`](https://github.com/netbirdio/netbird/commit/467b0f94b161f59def3e2f9a19a0dc2e995e0989) better naming of exposed user permissions - [`96b22d5`](https://github.com/netbirdio/netbird/commit/96b22d5140d0f30e3593eed8e7cabec0bf289c90) Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions - [`ca356f1`](https://github.com/netbirdio/netbird/commit/ca356f1dae9bb5c84df5c380f12724390525af48) fix rest test - [`974f297`](https://github.com/netbirdio/netbird/commit/974f29702201af587fa4dd4c2cdfd330bf4b1e62) Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions - [`ab391ed`](https://github.com/netbirdio/netbird/commit/ab391ed9faed349812414ee60d22bd31d9af2f0a) [management] restrict dashboard only to restrictable roles - [`e42d4c0`](https://github.com/netbirdio/netbird/commit/e42d4c06c2b1d2432ae1d372b49da9042b53b783) fix rest test - [`82ab78d`](https://github.com/netbirdio/netbird/commit/82ab78d176cbe1376082cca1ec74f201893b6386) make user permissions optional - [`a01a817`](https://github.com/netbirdio/netbird/commit/a01a8174ba9333c8011908f721c13653481c453d) Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions ### 📊 Changes **14 files changed** (+398 additions, -258 deletions) <details> <summary>View changed files</summary> 📝 `management/client/rest/users_test.go` (+2 -5) 📝 `management/server/account/manager.go` (+2 -1) 📝 `management/server/http/api/openapi.yml` (+23 -5) 📝 `management/server/http/api/types.gen.go` (+3 -12) 📝 `management/server/http/handlers/users/users_handler.go` (+22 -4) 📝 `management/server/http/handlers/users/users_handler_test.go` (+154 -39) 📝 `management/server/mock_server/account_mock.go` (+3 -2) 📝 `management/server/permissions/manager.go` (+21 -0) 📝 `management/server/permissions/manager_mock.go` (+15 -0) 📝 `management/server/permissions/modules/module.go` (+16 -0) 📝 `management/server/types/user.go` (+6 -20) 📝 `management/server/user.go` (+24 -32) 📝 `management/server/user_test.go` (+93 -138) ➕ `management/server/users/user.go` (+14 -0) </details> ### 📄 Description ## Describe your changes Get current user response info with role permissions by module and operation ## Issue ticket number and link ## Stack - `feature/additional-roles` - :warning: No PR associated with branch <!-- branch-stack --> - \#3686 :point\_left: - \#3691 ### 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 06:07:09 -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#23378