Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix: SignServer TCP ping for custom port#2353

Merged
fumiama merged 1 commit into
Mrs4s:devfrom
xiwangly2:dev
Aug 23, 2023
Merged

Fix: SignServer TCP ping for custom port#2353
fumiama merged 1 commit into
Mrs4s:devfrom
xiwangly2:dev

Conversation

@xiwangly2

@xiwangly2 xiwangly2 commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

解决了对于使用了强制 https 的 SignServer 在进行 TCP ping 时 错误的只截取了 host ,导致判定使用 80 端口而始终 ping 不通
的问题

TCP ping
http://xxx.com
https://xxx.com:443
https://xxx.com ×

[2023-08-07 08:48:11] [INFO]: 使用服务器 https://xxx.com/ 进
行数据包签名
[2023-08-07 08:48:17] [WARNING]: 连接到签名服务器出现错误: 丢包4/4 时延9999ms
[2023-08-07 08:48:22] [WARNING]: 连接到签名服务器出现错误: 丢包4/4 时延9999ms
[2023-08-07 08:48:27] [WARNING]: 连接到签名服务器出现错误: 丢包4/4 时延9999ms
[2023-08-07 08:48:32] [WARNING]: 连接到签名服务器出现错误: 丢包4/4 时延9999ms
[2023-08-07 08:48:36] [FATAL]: 连接签名服务器失败

现在,它应该能正确的通过以上 TCP ping 的逻辑

同时,也确保了对于非 80/443 端口的 SignServer 服务器的 TCP ping 支持

[2023-08-07 08:58:53] [INFO]: 使用服务器 https://xxx.com/ 进
行数据包签名
[2023-08-07 08:59:00] [INFO]: 连接至签名服务器: https://xxx.com/
[2023-08-07 08:59:10] [INFO]: 注册QQ实例 10001成功: The QQ has already loaded an instance, so this time it is deleting the existing instance and creating a new one.
[2023-08-07 08:59:10] [INFO]: Bot将在5秒后登录并开始信息处理, 按 Ctrl+C 取消.
[2023-08-07 08:59:10] [INFO]: 每 30 分钟将刷新一次签名 token
[2023-08-07 08:59:15] [INFO]: 开始尝试登录并同步消息...
[2023-08-07 08:59:15] [INFO]: 使用协议: Android Pad 8.9.63.11390

@xiwangly2

Copy link
Copy Markdown
Contributor Author

同样的,ipv6测试通过#2358
[2023-08-10 16:06:02] [INFO]: 使用服务器 http://[2406:da1c:d78:5c01:ff44:315f:129e:b8cc]:8080/ 进行数据包签名
[2023-08-10 16:06:08] [INFO]: 连接至签名服务器: http://[2406:da1c:d78:5c01:ff44:315f:129e:b8cc]:8080/
[2023-08-10 16:06:19] [INFO]: 注册QQ实例 3547251134 成功: Instance loaded successfully.
[2023-08-10 16:06:19] [INFO]: Bot将在5秒后登录并开始信息处理, 按 Ctrl+C 取消.
[2023-08-10 16:06:19] [INFO]: 每 30 分钟将刷新一次签名 token
[2023-08-10 16:06:24] [INFO]: 开始尝试登录并同步消息...
[2023-08-10 16:06:24] [INFO]: 使用协议: Android Pad 8.9.63.11390

@KewenCode KewenCode mentioned this pull request Aug 22, 2023
4 tasks
@fumiama fumiama merged commit d85d697 into Mrs4s:dev Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants