Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: SignServer TCP ping for custom port #2353

Merged
merged 1 commit into from
Aug 23, 2023
Merged

Fix: SignServer TCP ping for custom port #2353

merged 1 commit into from
Aug 23, 2023

Conversation

xiwangly2
Copy link
Contributor

@xiwangly2 xiwangly2 commented Aug 7, 2023

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

TCP ping
http:https://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
Contributor Author

同样的,ipv6测试通过#2358
[2023-08-10 16:06:02] [INFO]: 使用服务器 http:https://[2406:da1c:d78:5c01:ff44:315f:129e:b8cc]:8080/ 进行数据包签名
[2023-08-10 16:06:08] [INFO]: 连接至签名服务器: http:https://[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
11 checks passed
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.

None yet

2 participants