[PR #3729] [CLOSED] [management] users roles endpoint #18396

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

📋 Pull Request Information

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

Base: mainHead: feature/users-roles-endpoint


📝 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
  • 42f088b [management] users roles endpoint
  • ca356f1 fix rest test
  • f8804ec Merge branch 'feature/user-info-with-role-permissions' into feature/users-roles-endpoint
  • 974f297 Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions
  • 22b06ff Merge branch 'feature/user-info-with-role-permissions' into feature/users-roles-endpoint
  • ab391ed [management] restrict dashboard only to restrictable roles

📊 Changes

10 files changed (+262 additions, -39 deletions)

View changed files

📝 management/cmd/management.go (+3 -2)
📝 management/server/http/api/openapi.yml (+61 -0)
📝 management/server/http/api/types.gen.go (+7 -0)
📝 management/server/http/handler.go (+3 -1)
📝 management/server/http/handlers/users/users_handler.go (+61 -11)
📝 management/server/http/testing/testing_tools/tools.go (+3 -3)
📝 management/server/permissions/manager.go (+15 -0)
📝 management/server/permissions/manager_mock.go (+14 -0)
📝 management/server/users/manager.go (+28 -22)
management/server/users/manager_mock.go (+67 -0)

📄 Description

Describe your changes

Dedicated endpoint to retrieve roles and permissions

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/3729 **Author:** [@pnmcosta](https://github.com/pnmcosta) **Created:** 4/23/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `feature/users-roles-endpoint` --- ### 📝 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 - [`42f088b`](https://github.com/netbirdio/netbird/commit/42f088b80f70e9db0365e0c2da1f35458dead1d3) [management] users roles endpoint - [`ca356f1`](https://github.com/netbirdio/netbird/commit/ca356f1dae9bb5c84df5c380f12724390525af48) fix rest test - [`f8804ec`](https://github.com/netbirdio/netbird/commit/f8804ec6d874e4b5ca022f72128aa342525355d4) Merge branch 'feature/user-info-with-role-permissions' into feature/users-roles-endpoint - [`974f297`](https://github.com/netbirdio/netbird/commit/974f29702201af587fa4dd4c2cdfd330bf4b1e62) Merge branch 'feature/additional-roles' into feature/user-info-with-role-permissions - [`22b06ff`](https://github.com/netbirdio/netbird/commit/22b06fff0bd6f9eda46276a8af1708ff897c4bfd) Merge branch 'feature/user-info-with-role-permissions' into feature/users-roles-endpoint - [`ab391ed`](https://github.com/netbirdio/netbird/commit/ab391ed9faed349812414ee60d22bd31d9af2f0a) [management] restrict dashboard only to restrictable roles ### 📊 Changes **10 files changed** (+262 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `management/cmd/management.go` (+3 -2) 📝 `management/server/http/api/openapi.yml` (+61 -0) 📝 `management/server/http/api/types.gen.go` (+7 -0) 📝 `management/server/http/handler.go` (+3 -1) 📝 `management/server/http/handlers/users/users_handler.go` (+61 -11) 📝 `management/server/http/testing/testing_tools/tools.go` (+3 -3) 📝 `management/server/permissions/manager.go` (+15 -0) 📝 `management/server/permissions/manager_mock.go` (+14 -0) 📝 `management/server/users/manager.go` (+28 -22) ➕ `management/server/users/manager_mock.go` (+67 -0) </details> ### 📄 Description ## Describe your changes Dedicated endpoint to retrieve roles and permissions ## Issue ticket number and link ## Stack <!-- branch-stack --> ### 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 04:08:28 -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#18396