登录
注册
开源
企业版
高校版
搜索
帮助中心
使用条款
关于我们
开源
企业版
高校版
私有云
模力方舟
AI 队友
登录
注册
轻量养虾,开箱即用!低 Token + 稳定算力,Gitee & 模力方舟联合出品的 PocketClaw 正式开售!点击了解详情
代码拉取完成,页面将自动刷新
开源项目
>
程序开发
>
网络开发包
&&
捐赠
捐赠前请先登录
取消
前往登录
扫描微信二维码支付
取消
支付完成
支付提示
将跳转至支付宝完成支付
确定
取消
Watch
不关注
关注所有动态
仅关注版本发行动态
关注但不提醒动态
351
Star
2.9K
Fork
850
GVP
若汝棋茗
/
TouchSocket
代码
Issues
27
统计
流水线
服务
质量分析
Jenkins for Gitee
腾讯云托管
腾讯云 Serverless
悬镜安全
阿里云 SAE
Codeblitz
SBOM
开发画像分析
我知道了,不再自动展开
更新失败,请稍后重试!
移除标识
内容风险标识
本任务被
标识为内容中包含有代码安全 Bug 、隐私泄露等敏感信息,仓库外成员不可访问
[Tcp]TryShutdown的关闭机制没有考虑异步发送队列的情况
已完成
#IBTKMP
若汝棋茗
拥有者
创建于
2025-03-15 11:58
### 组件 Tcp ### 版本号 3.0.17 ### .NET SDK .NET8 ### 项目类型 Console ### 操作系统 Windows 11 ### 运行环境 开发环境 (Development) ### 这个问题是否已经存在? - [x] 我已经搜索过现有的问题 (https://gitee.com/RRQM_Home/TouchSocket/issues) ### 如何复现 框架使用的TryShutdown大致代码如下: ``` public static bool TryShutdown<TClient>(this TClient client, SocketShutdown how = SocketShutdown.Both) where TClient : class, ITcpSession { // 检查客户端对象是否为null或默认值 if (client == default) { return false; } try { // 获取客户端的主要Socket对象 var socket = client.MainSocket; // 检查Socket对象是否为空 if (socket == null) { return false; } // 检查Socket连接是否已经建立 if (!socket.Connected) { return false; } // 发送关闭报文 socket.Shutdown(how); return true; } catch { return false; } } ``` 上述逻辑中没有检查异步发送队列是否为空的情况。例如: 对于WebApiParserPlugin插件,在响应时,会先执行AnswerAsync。 然后如果判断为短链接的话,应该需要关闭会话。关闭方式应该是等待缓存区数据发送完成后关闭。 一般使用下方编码,逻辑视为正确。 ``` await httpResponse.AnswerAsync().ConfigureAwait(EasyTask.ContinueOnCapturedContext); if (!httpContext.Request.KeepAlive) { client.TryShutdown(SocketShutdown.Both); } ``` 但实际上。因为有异步发送队列的存在。会导致`await AnswerAsync()`之后,仅仅表示数据放到了发送队列,而不是已在tcp发送区。 所以会导致短链接数据无法成功发送。 ### 预期结果 正确处理Shutdown的逻辑。应该考虑异步发送队列。 ### 实际结果 错误的Shutdown的逻辑 ### 异常信息 无 ### Demo 地址(仅限Git地址) 无 ### 承诺支持 - [x] 我确定我已经对TouchSocket项目进行了“Star”操作。 ### 承诺规范 - [x] 我确定已完整阅读[Issue提问规范](https://touchsocket.net/docs/current/troubleshootissue),并按照要求填写。 ### 承诺友好 - [x] 我承诺将本着相互尊重、理解和友善的态度进行交流,共同维护好 TouchSocket 来之不易的良好的社区氛围。
### 组件 Tcp ### 版本号 3.0.17 ### .NET SDK .NET8 ### 项目类型 Console ### 操作系统 Windows 11 ### 运行环境 开发环境 (Development) ### 这个问题是否已经存在? - [x] 我已经搜索过现有的问题 (https://gitee.com/RRQM_Home/TouchSocket/issues) ### 如何复现 框架使用的TryShutdown大致代码如下: ``` public static bool TryShutdown<TClient>(this TClient client, SocketShutdown how = SocketShutdown.Both) where TClient : class, ITcpSession { // 检查客户端对象是否为null或默认值 if (client == default) { return false; } try { // 获取客户端的主要Socket对象 var socket = client.MainSocket; // 检查Socket对象是否为空 if (socket == null) { return false; } // 检查Socket连接是否已经建立 if (!socket.Connected) { return false; } // 发送关闭报文 socket.Shutdown(how); return true; } catch { return false; } } ``` 上述逻辑中没有检查异步发送队列是否为空的情况。例如: 对于WebApiParserPlugin插件,在响应时,会先执行AnswerAsync。 然后如果判断为短链接的话,应该需要关闭会话。关闭方式应该是等待缓存区数据发送完成后关闭。 一般使用下方编码,逻辑视为正确。 ``` await httpResponse.AnswerAsync().ConfigureAwait(EasyTask.ContinueOnCapturedContext); if (!httpContext.Request.KeepAlive) { client.TryShutdown(SocketShutdown.Both); } ``` 但实际上。因为有异步发送队列的存在。会导致`await AnswerAsync()`之后,仅仅表示数据放到了发送队列,而不是已在tcp发送区。 所以会导致短链接数据无法成功发送。 ### 预期结果 正确处理Shutdown的逻辑。应该考虑异步发送队列。 ### 实际结果 错误的Shutdown的逻辑 ### 异常信息 无 ### Demo 地址(仅限Git地址) 无 ### 承诺支持 - [x] 我确定我已经对TouchSocket项目进行了“Star”操作。 ### 承诺规范 - [x] 我确定已完整阅读[Issue提问规范](https://touchsocket.net/docs/current/troubleshootissue),并按照要求填写。 ### 承诺友好 - [x] 我承诺将本着相互尊重、理解和友善的态度进行交流,共同维护好 TouchSocket 来之不易的良好的社区氛围。
评论 (
0
)
登录
后才可以发表评论
状态
已完成
待办的
进行中
已完成
已关闭
负责人
未设置
标签
bug
未设置
标签管理
里程碑
未关联里程碑
未关联里程碑
Pull Requests
未关联
未关联
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
未关联
分支 (
-
)
标签 (
-
)
开始日期   -   截止日期
-
置顶选项
不置顶
置顶等级:高
置顶等级:中
置顶等级:低
优先级
不指定
严重
主要
次要
不重要
参与者(1)
C#
1
https://gitee.com/RRQM_Home/TouchSocket.git
git@gitee.com:RRQM_Home/TouchSocket.git
RRQM_Home
TouchSocket
TouchSocket
点此查找更多帮助
搜索帮助
Git 命令在线学习
如何在 Gitee 导入 GitHub 仓库
Git 仓库基础操作
企业版和社区版功能对比
SSH 公钥设置
如何处理代码冲突
仓库体积过大,如何减小?
如何找回被删除的仓库数据
Gitee 产品配额说明
GitHub仓库快速导入Gitee及同步更新
什么是 Release(发行版)
将 PHP 项目自动发布到 packagist.org
评论
仓库举报
回到顶部
登录提示
该操作需登录 Gitee 帐号,请先登录后再操作。
立即登录
没有帐号,去注册