feat/permissions-attribute #20

Merged
eliott merged 3 commits from feat/permissions-attribute into develop 2026-03-31 17:57:29 +02:00
Member
No description provided.
eliott left a comment

Tu peux traiter tous les retours sauf celui du PermissionHandler.cs il manque des data

Tu peux traiter tous les retours sauf celui du PermissionHandler.cs il manque des data
@ -0,0 +23,4 @@
{
// Mark the requirement as succeeded if the user has the necessary permission
context.Succeed(requirement);
}

Le début est bon, il faudrait en plus vérifier à partir de la DB si les rôles du user contiennent les permissions demandées. Mais pour le moment aucune des tâbles sont faites donc faut attendre que les autres tâches soient faites.

Le début est bon, il faudrait en plus vérifier à partir de la DB si les rôles du user contiennent les permissions demandées. Mais pour le moment aucune des tâbles sont faites donc faut attendre que les autres tâches soient faites.
@ -0,0 +4,4 @@
/// Defines the different permissions used in the application.
/// Permissions can be used in authorization requirements to control access.
/// </summary>
/// Note: Should these permissions be defined here, or stored in the database for flexibility?

Pour moi on peut les déclarer en DB avec un nouvelle table Permission. Donc tu peux delete la class. La table permission sera une autre tâche

Pour moi on peut les déclarer en DB avec un nouvelle table Permission. Donc tu peux delete la class. La table permission sera une autre tâche
@ -0,0 +1,31 @@
using Microsoft.AspNetCore.Authorization;

Les controllers de test faut pas les push, ça fait du code sur le serveur qui est pas du vrai code

Les controllers de test faut pas les push, ça fait du code sur le serveur qui est pas du vrai code
Program.cs Outdated
@ -12,0 +30,4 @@
ValidAudience = builder.Configuration["Jwt:Audience"],
IssuerSigningKey = new SymmetricSecurityKey(Encoding.UTF8.GetBytes(builder.Configuration["Jwt:Key"]))
};
});

Tout ce code est déjà déclaré dans le DependencyInjectionExtensions.cs donc ça ne sert à rien

Tout ce code est déjà déclaré dans le DependencyInjectionExtensions.cs donc ça ne sert à rien
Program.cs Outdated
@ -12,0 +45,4 @@
options.AddPolicy("ProductsEditPolicy", policy =>
policy.Requirements.Add(new PermissionRequirement(Permissions.ProductsEdit)));
});

Toutes les policy déplace les dans le DependencyInjectionExtensions pour que ce soit plus clean

Toutes les policy déplace les dans le DependencyInjectionExtensions pour que ce soit plus clean
eliott approved these changes 2026-03-31 17:56:49 +02:00
@ -118,0 +124,4 @@
{
builder.Services.AddAuthorization(options =>
{
options.AddPolicy("UsersReadPolicy", policy =>

Pour les AddPolicy j'y avais pas pensé avant mais il va falloir attendre aussi que la table des Permissions soit faites pour que tu puisses avoir une policy par ligne de la table permission (à voir comment faire jsp pour le coup).

Pour les AddPolicy j'y avais pas pensé avant mais il va falloir attendre aussi que la table des Permissions soit faites pour que tu puisses avoir une policy par ligne de la table permission (à voir comment faire jsp pour le coup).
Owner

Je valide la PR mais faudrai penser (toi ou n'importe qui d'ailleurs) à bien adapter l'attribute avec les données de la DB

Je valide la PR mais faudrai penser (toi ou n'importe qui d'ailleurs) à bien adapter l'attribute avec les données de la DB
eliott merged commit 5f2e97ef1c into develop 2026-03-31 17:57:29 +02:00
eliott deleted branch feat/permissions-attribute 2026-03-31 17:57:29 +02:00
Commenting is not possible because the repository is archived.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
cda-solutions/resources-relationnelles-api!20
No description provided.