feat/permissions-attribute #20
Loading…
Reference in a new issue
No description provided.
Delete branch "feat/permissions-attribute"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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 permissioncontext.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.
@ -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
@ -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
@ -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
@ -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
@ -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).
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