dotnet/aspnetcore

Write more content directly to StreamWriter in W3CLogger

Ouverte

#33 946 ouverte le 29 juin 2021

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)C# (10 653 forks)batch import
Perfarea-networkinghelp wanted

Métriques du dépôt

Stars
 (37 933 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Le problème suggère d'éliminer la mise en forme intermédiaire des chaînes dans W3CLogger en écrivant directement dans le StreamWriter. Deux fichiers clés sont concernés : src/Middleware/HttpLogging/src/W3CLogger.cs (ligne 49) et src/Middleware/HttpLogging/src/W3CLoggerProcessor.cs (ligne 35). L'approche consiste à retarder la mise en forme et à mettre en file d'attente les éléments de données, puis à les écrire directement dans le flux lors du vidage. Consultez la discussion PR liée (#33251) pour le contexte et assurez vous que le changement maintient la sécurité des threads et la conformité au format de journal.
Stack technique
csharp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65