dotnet/aspnetcore

Perf: add API to avoid regex matching rewrite rules unless necessary

Open

#5984 aperta il 30 ago 2016

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
api-suggestionarea-middlewareenhancementfeature-rewrite-middlewarehelp wanted

Metriche repository

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

Descrizione

Regex is an expensive part of the rewrite middleware a lot in order to do pattern matching. It is common for rewrite patterns to be led by a static prefix.

Example: "blogs/(.*)" => "api/blogs?id=$1"

For this rule, the middleware can avoid unnecessary regex allocations by only running the rule when the path begins with "/blogs/".

Possible API design:

RewriteOptions.AddRewrite(string prefix, string pattern, string urlResult) // prefix is static, doesn't container rules. If uri doesn't start with the prefix, skip this rule
RewriteOptions.Ignore(string prefix); // always skip rule processing for urls beginning with this prefix

Usage

RewriteOptions.AddRewrite(prefix: "/blogs/", pattern: "(.*)", urlResult: "/api/blogs?id=$1")
RewriteOptions.Ignore("/images/")

Guida contributor

Direzione di ricerca
Implementa i metodi AddRewrite e Ignore nella classe RewriteOptions per consentire il salto della corrispondenza regex basato sul prefisso. Il metodo AddRewrite deve memorizzare un prefisso statico e un pattern; se il percorso della richiesta non inizia con il prefisso, salta la regola. Il metodo Ignore deve saltare tutte le regole per i percorsi che iniziano con il prefisso specificato. Assicurati che i test esistenti passino e aggiungi nuovi test per la corrispondenza del prefisso.
Tech stack
csharp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
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.
60