dotnet/aspnetcore

Write more content directly to StreamWriter in W3CLogger

Aperta

#33.946 aperta il 29 giu 2021

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
Perfarea-networkinghelp wanted

Metriche repository

Star
 (37.933 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

See https://github.com/dotnet/aspnetcore/pull/33251/#discussion_r655734742, https://github.com/dotnet/aspnetcore/pull/33251/#discussion_r656445769.

https://github.com/dotnet/aspnetcore/blob/3177376dca6ca0a6bba1d4ab946e77f02547c167/src/Middleware/HttpLogging/src/W3CLogger.cs#L49

Since we know the only consumer of this data is StreamWriter, can you delay the Format, enqueue the elements array, and format when writing to the stream? This intermediate string is likely large and useless.

https://github.com/dotnet/aspnetcore/blob/3177376dca6ca0a6bba1d4ab946e77f02547c167/src/Middleware/HttpLogging/src/W3CLoggerProcessor.cs#L35

More content that should be written directly to the streamWriter.

Guida contributor

Direzione di ricerca
Il problema suggerisce di eliminare la formattazione intermedia delle stringhe in W3CLogger scrivendo direttamente nello StreamWriter. Sono coinvolti due file chiave: src/Middleware/HttpLogging/src/W3CLogger.cs (riga 49) e src/Middleware/HttpLogging/src/W3CLoggerProcessor.cs (riga 35). L'approccio consiste nel ritardare la formattazione e accodare gli elementi di dati, quindi scriverli direttamente nello stream durante il flush. Rivedere la discussione PR collegata (#33251) per il contesto e assicurarsi che la modifca mantenga la sicurezza del thread e la conformità al formato di log.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65