dotnet/aspnetcore

Short circuit attribute

Fermée

#49 181 ouverte le 3 juil. 2023

Voir sur GitHubGuide contributeur
 (12 commentaires) (2 réactions) (1 personne assignée)C# (10 653 forks)batch import
api-approvedarea-middlewarehelp wanted

Métriques du dépôt

Stars
 (37 933 étoiles)
Métriques de merge PR
 (Merge moyen 16j 9h) (258 PRs mergées en 30 j)

Description

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.

The short circuit feature only has extension methods for adding the required metadata to endpoints. Should there be an attribute? That would allow MVC actions and gRPC methods to support short circuiting.

For example,

public class MonitoringController
{
    [ShortCircuit]
    [HttpGet("/status")]
    public ActionResult GetStatus()
    {
        // ...
    }
}

Brought up in YouTube video: https://youtu.be/rXdwX2X4-gw?t=422

Note that a short-circuited MVC action would still execute filters, and a gRPC method would still execute interceptors.

Describe the solution you'd like

  • Change internal ShortCircuitMetadata to an interface. (optional)
  • Change current metadata to an attribute and make public (implement the interface if present)

Additional context

No response

Guide contributeur

Direction de recherche
Le problème propose d'ajouter un attribut [ShortCircuit] pour MVC et gRPC. Examinez la classe ShortCircuitMetadata existante dans le code source d'ASP.NET Core (probablement dans src/Mvc ou src/Grpc). La solution consiste à la transformer en interface et à créer un attribut qui l'implémente. Les fichiers pertinents incluent les classes de métadonnées et les méthodes d'extension. La vidéo YouTube liée fournit le contexte. L'assigné travaille probablement dessus, alors vérifiez s'il y a des branches ou des commentaires liés pour suivre la progression.
Stack technique
csharp
Domaine
backendapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#ASP.NET Core basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40