dotnet/aspnetcore

HttpRuleParser GetExpressionLength allows invalid characters.

Open

#2694 aperta il 2 gen 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
affected-fewarea-networkingbugfeature-http-abstractionshelp wantedseverity-minor

Metriche repository

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

Descrizione

From @jkotalik on Tuesday, August 22, 2017 4:19:40 PM

@Tratcher and I discovered that GetExpressionLength in HttpRuleParser allows invalid characters (including control characters in expressions. GetExpressionLength mentions that we don't really care about the content of a quoted string, however it seems appropriate that if a quoted string has an invalid character, it should throw on parsing here, not in Kestrel (or whatever server).

This would be a breaking change, as it would introduce a new place where an exception is thrown, however it is probably the right behavior.

Copied from original issue: aspnet/HttpAbstractions#923

Guida contributor

Direzione di ricerca
Ispeziona il metodo GetExpressionLength in HttpRuleParser.cs e i test correlati. Comprendi dove sono consentiti caratteri non validi e aggiungi la convalida per lanciare un'eccezione appropriata.
Tech stack
csharp
Dominio
backend
Tipo issue
Bug
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55