dotnet/aspnetcore

LoadAsync on Session - when _isNewSessionKey is true

Open

#20 898 ouverte le 16 avr. 2020

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)C# (10 653 forks)batch import
Perfaffected-fewarea-middlewarebreaking-changeenhancementhelp wantedseverity-nice-to-have

Métriques du dépôt

Stars
 (37 933 stars)
Métriques de merge PR
 (Merge moyen 16j 9h) (258 PRs mergées en 30 j)

Description

https://github.com/dotnet/aspnetcore/blob/8b000d961cd3ccfcc8090fb8368fd6598bace978/src/Middleware/Session/src/DistributedSession.cs

Should Load/LoadAsync be a no-op when _isNewSessionKey is true. Taking the advice to call LoadAsync prior to accessing the session with get/trygetvalue/set I would of thought we'd guard against unnecsssary chitchat to the cache if we know the session is new?

I though about doing Request.Cookies["sesscoookie"] != null then await LoadAsync but then doing Get/Set/Id/IsAvailable will simply call load anyway if not _loaded isnt marked.

Guide contributeur

Direction de recherche
Examinez le fichier DistributedSession.cs, en particulier la méthode LoadAsync et le drapeau isNewSessionKey. Passez en revue les commentaires existants dans l'issue pour toute orientation des mainteneurs. Déterminez l'impact de faire de LoadAsync une opération nulle lorsque la session est nouvelle et assurez vous que les appels ultérieurs à Get/TryGetValue/Set fonctionnent toujours correctement.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
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.
1-3 heures
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.
Claire
Prérequis
C#ASP.NET Core
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70