dotnet/aspnetcore

Short circuit attribute

Geschlossen

#49.181 geöffnet am 03.07.2023

Auf GitHub ansehenContributor Guide
 (12 Kommentare) (2 Reaktionen) (1 zugewiesene Person)C# (10.653 Forks)batch import
api-approvedarea-middlewarehelp wanted

Repository-Metriken

Stars
 (37.933 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 16T 9h) (258 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Issue schlägt vor, ein [ShortCircuit] Attribut für MVC und gRPC hinzuzufügen. Untersuche die vorhandene ShortCircuitMetadata Klasse im ASP.NET Core Quellcode (wahrscheinlich in src/Mvc oder src/Grpc). Die Lösung besteht darin, sie in eine Schnittstelle umzuwandeln und ein Attribut zu erstellen, das diese implementiert. Relevante Dateien umfassen Metadatenklassen und Erweiterungsmethoden. Das verlinkte YouTube Video liefert Kontext. Der Zugewiesene arbeitet wahrscheinlich daran, also prüfe auf verknüpfte Branches oder Kommentare, um den Fortschritt zu überprüfen.
Tech Stack
csharp
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#ASP.NET Core basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40