dotnet/aspnetcore

IClaimsTransformation do not automagically compose

Open

#46.120 aperta il 16 gen 2023

Vedi su GitHub
 (19 commenti) (25 reazioni) (0 assegnatari)C# (10.653 fork)batch import
Pillar: Dev Experiencearea-authenhancementhelp wanteduntriaged

Metriche repository

Star
 (37.933 star)
Metriche merge PR
 (Merge medio 16g 9h) (258 PR mergiate in 30 g)

Descrizione

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

Multiple registrations on IClaimsTransformation should compose. For pete's sake, the signature for IClaimsTransformation is literally an async reduce function.

Describe the solution you'd like

A ReducerClaimsTransformer should be injected into AuthenticationService.Transform

public class ReducerClaimsTransformation : IClaimsTransformation
{
    public async Task<ClaimsPrincipal> TransformAsync(ClaimsPrincipal principal)
    {
        foreach (var transformation in _transformations)
        {
            principal = await transformation.TransformAsync(principal);
        }

        return principal;
    }
}

Additional context

No response

Guida contributor

Direzione di ricerca
Indaga la registrazione corrente di IClaimsTransformation e implementa un pattern reducer per concatenare più trasformazioni.
Tech stack
csharp
Dominio
backendauthentication
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#ASP.NET Core
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70