dotnet/aspnetcore

LoadAsync on Session - when _isNewSessionKey is true

Aperta

#20.898 aperta il 16 apr 2020

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (0 assegnatari)C# (10.653 fork)batch import
Perfaffected-fewarea-middlewarebreaking-changeenhancementhelp wantedseverity-nice-to-have

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina il file DistributedSession.cs, in particolare il metodo LoadAsync e il flag isNewSessionKey. Rivedi i commenti esistenti nell'issue per eventuali indicazioni dei manutentori. Determina l'impatto di rendere LoadAsync un'operazione nulla quando la sessione è nuova e assicurati che le chiamate successive a Get/TryGetValue/Set funzionino ancora correttamente.
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
C#ASP.NET Core
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70