[PR #188] feat: add AppTokens authentication and bulk permission operations #179

New issue

Closed

opened 2026-02-26 17:37:38 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 17:37:38 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Corsinvest/cv4pve-admin/pull/188
Author: @franklupo
Created: 2/23/2026
Status: ✅ Merged
Merged: 2/23/2026
Merged by: @franklupo

Base: main ← Head: feat/app-tokens-permission-bulk

📝 Commits (1)

2a02a12 feat: add AppTokens authentication and bulk permission operations

📊 Changes

21 files changed (+1602 additions, -264 deletions)

View changed files

📝 src/Corsinvest.ProxmoxVE.Admin.Core/Extensions/IdentityExtensions.cs (+3 -3)
➕ src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/AppTokens/IAppTokenService.cs (+38 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/Permissions/IPermissionService.cs (+13 -8)
➕ src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/Permissions/PermissionData.cs (+11 -0)
➕ src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppToken.cs (+28 -0)
➕ src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppTokenPermission.cs (+11 -0)
➕ src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppTokenRole.cs (+14 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/ApplicationClaimTypes.cs (+1 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/ApplicationRole.cs (+1 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/BasePermission.cs (+1 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/GlobalUsings.cs (+0 -1)
➕ src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/20260221112216_AddAppTokens.Designer.cs (+716 -0)
➕ src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/20260221112216_AddAppTokens.cs (+139 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/ModuleDbContextModelSnapshot.cs (+147 -7)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Module.cs (+7 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Persistence/ModuleDbContext.cs (+40 -0)
➕ src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/AppTokenAuthenticationHandler.cs (+54 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Permissions.cs (+6 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/ServiceCollectionExtensions.cs (+5 -1)
➕ src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Services/AppTokenService.cs (+108 -0)

...and 1 more files

📄 Description

Summary

Add AppToken, AppTokenPermission, AppTokenRole entities with EF Core migration
Add AppTokenService and AppTokenAuthenticationHandler for token-based authentication
Add PermissionData DTO and bulk sync operations (SyncForRoleAsync, SyncForUserAsync) to IPermissionService
Refactor PermissionService to support efficient bulk permission management

Test plan

Create an AppToken and verify authentication works
Verify role permissions sync correctly via SyncForRoleAsync
Verify user permissions sync correctly via SyncForUserAsync
Verify EF migration applies cleanly

_{🔄 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/Corsinvest/cv4pve-admin/pull/188 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/23/2026 **Status:** ✅ Merged **Merged:** 2/23/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `main` ← **Head:** `feat/app-tokens-permission-bulk` --- ### 📝 Commits (1) - [`2a02a12`](https://github.com/Corsinvest/cv4pve-admin/commit/2a02a129f8b3c892ce6169ca6b71d911e8df1946) feat: add AppTokens authentication and bulk permission operations ### 📊 Changes **21 files changed** (+1602 additions, -264 deletions) <details> <summary>View changed files</summary> 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Extensions/IdentityExtensions.cs` (+3 -3) ➕ `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/AppTokens/IAppTokenService.cs` (+38 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/Permissions/IPermissionService.cs` (+13 -8) ➕ `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Auth/Permissions/PermissionData.cs` (+11 -0) ➕ `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppToken.cs` (+28 -0) ➕ `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppTokenPermission.cs` (+11 -0) ➕ `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/AppTokenRole.cs` (+14 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/ApplicationClaimTypes.cs` (+1 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/ApplicationRole.cs` (+1 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Security/Identity/BasePermission.cs` (+1 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/GlobalUsings.cs` (+0 -1) ➕ `src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/20260221112216_AddAppTokens.Designer.cs` (+716 -0) ➕ `src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/20260221112216_AddAppTokens.cs` (+139 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Migrations/ModuleDbContextModelSnapshot.cs` (+147 -7) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Module.cs` (+7 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Persistence/ModuleDbContext.cs` (+40 -0) ➕ `src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/AppTokenAuthenticationHandler.cs` (+54 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Permissions.cs` (+6 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/ServiceCollectionExtensions.cs` (+5 -1) ➕ `src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Services/AppTokenService.cs` (+108 -0) _...and 1 more files_ </details> ### 📄 Description ## Summary - Add `AppToken`, `AppTokenPermission`, `AppTokenRole` entities with EF Core migration - Add `AppTokenService` and `AppTokenAuthenticationHandler` for token-based authentication - Add `PermissionData` DTO and bulk sync operations (`SyncForRoleAsync`, `SyncForUserAsync`) to `IPermissionService` - Refactor `PermissionService` to support efficient bulk permission management ## Test plan - [ ] Create an AppToken and verify authentication works - [ ] Verify role permissions sync correctly via `SyncForRoleAsync` - [ ] Verify user permissions sync correctly via `SyncForUserAsync` - [ ] Verify EF migration applies cleanly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 17:37:38 +03:00

closed this issue
added the
pull-request
label

kerem referenced this issue

2026-03-07 19:57:19 +03:00

[PR #179] feat: misc fixes and improvements #171

kerem changed title from ~~[PR #188] [MERGED] feat: add AppTokens authentication and bulk permission operations~~ to [PR #188] feat: add AppTokens authentication and bulk permission operations

2026-03-07 19:57:21 +03:00