dotnet/aspnetcore

LoadAsync on Session - when _isNewSessionKey is true

开放

#20,898 创建于 2020年4月16日

在 GitHub 查看贡献者指南
 (10 条评论) (0 个反应) (0 位负责人)C# (10,653 个派生)batch import
Perfaffected-fewarea-middlewarebreaking-changeenhancementhelp wantedseverity-nice-to-have

仓库指标

星标
 (37,933 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C#ASP.NET Core
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70