dotnet/aspnetcore

LoadAsync on Session - when _isNewSessionKey is true

Open

#20,898 opened on 2020年4月16日

GitHub で見る
 (10 comments) (0 reactions) (0 assignees)C# (10,653 forks)batch import
Perfaffected-fewarea-middlewarebreaking-changeenhancementhelp wantedseverity-nice-to-have

Repository metrics

Stars
 (37,933 stars)
PR merge metrics
 (平均マージ 16d 9h) (30d で 258 merged PRs)

説明

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.

コントリビューターガイド

調査方針
DistributedSession.cs ファイル、特に LoadAsync メソッドと isNewSessionKey フラグを調査します。メンテナのガイダンスについて、この問題の既存のコメントを確認します。セッションが新しい場合に LoadAsync を無操作にすることの影響を判断し、Get/TryGetValue/Set への後続の呼び出しが引き続き正しく機能することを確認します。
技術スタック
csharp
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C#ASP.NET Core
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70