feat: implement port hopping #1064
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wwqgtxx
force-pushed
the
Alpha
branch
4 times, most recently
from
6321cfb
to
092e535
Compare
wwqgtxx
reviewed
Mar 8, 2024
| @@ -82,6 +91,41 @@ func closeHysteria2(h *Hysteria2) { | |||
| } | |||
| } | |||
|
|
|||
| func parsePorts(portStr string) (ports []uint16) { | |||
There was a problem hiding this comment.
common.utils中有NewUnsignedRanges可以直接用，不需要重新写一个新的
There was a problem hiding this comment.
主要是这里接收的参数不是 []uint16，而是 string，格式是 port1,port2-port3,port4，需要解析成前者。
我看 transport/hysteria/conns/udp/hop.go 也是自己实现的 parsePorts
H1JK
pushed a commit
that referenced
this pull request
Mar 8, 2024
* implement port hopping using sing and sing-quic * 更新quic-go * 更新sing * Update go.sum --------- Co-authored-by: wwqgtxx <wwqgtxx@gmail.com>
wwqgtxx
added a commit
that referenced
this pull request
Mar 8, 2024
* implement port hopping using sing and sing-quic * 更新quic-go * 更新sing * Update go.sum --------- Co-authored-by: wwqgtxx <wwqgtxx@gmail.com>
|
在使用固定端口时，更新到此版本会产生 配置文件中有这种写法 |
|
@YaeSakuraQvQ 目前server为域名事暂时不可用，请等待修复 |
|
昨天好像忘记提交这个了，刚才切换分支时才发现 然后测了下老版本是支持域名的，估计是改成 ListenPacket 后解析有问题。我现在得出门了，晚上再研究。 |
|
@keakon 明天晚上我来修吧，其实不是ListenPacket本身不支持域名，是WriteTo不支持。原来的Unbind是调用Write的，所以在DialContext的时候就已经解析完了 |
This was referenced Mar 9, 2024
|
@YaeSakuraQvQ @666dechaoge 你们可以试一下7ad37ca |
可以的，单端口域名的也能hy2节点能联通 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.