Skip to content

Comments

XICMP finalmask: Refine seq#5652

Merged
RPRX merged 4 commits intoXTLS:mainfrom
LjhAUMEM:icmp2
Feb 6, 2026
Merged

XICMP finalmask: Refine seq#5652
RPRX merged 4 commits intoXTLS:mainfrom
LjhAUMEM:icmp2

Conversation

@LjhAUMEM
Copy link
Contributor

@LjhAUMEM LjhAUMEM commented Feb 4, 2026

给 seq 加个超时机制,聊胜于无

本来想用 time.Time,但是一想到每次 read 都 time.Now 一下就有点难崩,那还是用窗口模拟吧

之前还想除了回复的第一个随机字节后面再加四字节 crc32 校验,想了想还是算了,是不是合法数据还是交给上层,而且 hash 验证也只能缓解

@RPRX
Copy link
Member

RPRX commented Feb 5, 2026

Ready 了吗

@LjhAUMEM
Copy link
Contributor Author

LjhAUMEM commented Feb 6, 2026

Ready 了

@RPRX RPRX changed the title refine seq XICMP finalmask: Refine seq Feb 6, 2026
@RPRX RPRX merged commit 957e5a6 into XTLS:main Feb 6, 2026
39 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.

2 participants