Skip to content

Conversation

@Unicode01
Copy link
Contributor

@Unicode01 Unicode01 commented Feb 26, 2025

完善错误提示 添加PreSend Packet 增加更改Host请求头

@Unicode01 Unicode01 changed the title 完善握手错误提示 添加PreSend Packet 完善握手错误提示 添加PreSend Packet 增加更改Host请求头 Feb 26, 2025
@lxzan
Copy link
Owner

lxzan commented Feb 27, 2025

周末看一下

@lxzan
Copy link
Owner

lxzan commented Feb 27, 2025

PreSend Packet做什么用的 ?

@Unicode01
Copy link
Contributor Author

PreSend Packet做什么用的 ?

握手包增加body :
http.NewRequestWithContext(ctx, http.MethodGet, c.option.Addr, nil)
http.NewRequestWithContext(ctx, http.MethodGet, c.option.Addr, bytes.NewReader(packet))

@lxzan
Copy link
Owner

lxzan commented Feb 27, 2025

什么场景会用到这个?

@Unicode01
Copy link
Contributor Author

什么场景会用到这个?

0RTT

@lxzan
Copy link
Owner

lxzan commented Feb 27, 2025

优化错误提示是可以接受的

WebSocket协议之外的东西原则上不再添加,除非特别有用,使用广泛

@Unicode01
Copy link
Contributor Author

优化错误提示是可以接受的

WebSocket协议之外的东西原则上不再添加,除非特别有用,使用广泛

可以,那我特殊场合自己改一下

@lxzan
Copy link
Owner

lxzan commented Feb 28, 2025

修改下PR, 仅保留错误优化吧

@Unicode01
Copy link
Contributor Author

修改下PR, 仅保留错误优化吧

修改了,保留了Host修改,不然有些场景Host不改不行

@lxzan lxzan changed the base branch from main to testing March 1, 2025 04:49
@lxzan lxzan merged commit c3db8e7 into lxzan:testing Mar 1, 2025
1 check failed
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