dotnet/aspnetcore

NotifyOnCompleted does not catch exceptions, may abort the response/connection

Open

#5885 aperta il 7 mar 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
affected-very-fewarea-networkingbugfeature-httpsyshelp wantedseverity-nice-to-have

Metriche repository

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

Descrizione

NotifyOnCompleted should catch and log its own exceptions as to not disrupt the normal request flow. It doesn't.

https://github.com/aspnet/HttpSysServer/blob/336e85d19abcdf3ec3a024cb9431c768a4e5d494/src/Microsoft.AspNetCore.Server.HttpSys/MessagePump.cs#L204-L214 https://github.com/aspnet/HttpSysServer/blob/336e85d19abcdf3ec3a024cb9431c768a4e5d494/src/Microsoft.AspNetCore.Server.HttpSys/FeatureContext.cs#L586-L591

It also fires before the response is flushed/disposed so these exceptions may abort the response/connection. Compare to https://github.com/aspnet/KestrelHttpServer/pull/2324

Guida contributor

Direzione di ricerca
Esamina le implementazioni del metodo NotifyOnCompleted in MessagePump.cs e FeatureContext.cs. Aggiungi blocchi try catch per registrare le eccezioni e assicurati che non interrompano la risposta o la connessione. Segui il pattern della pull request #2324 di KestrelHttpServer.
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70