精简trojan udp的处理代码 #142
Merged
精简trojan udp的处理代码 #142
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
|
感谢 PR，虽然也看到 v2ray 原本有 UDP 地址结构，但整个软件要兼容传递此结构所需的工程量过大，所以直接改 buffer 的结构了 不过剩下问题是并不是所有的 UDP buffer 都带地址（比如 VLESS、VMess 的暂无），我会看一下这里是否保持了兼容性 |
|
引入了两个问题，已通过 fb0e517 修复 |
github-actions bot
added a commit
to sbily1988/Xray-core
that referenced
this pull request
Jan 12, 2021
* https://github.com/XTLS/Xray-core: (31 commits) Support for VLESS & VMess UUID v5 mapping standard Trojan: Do not panic when UDP dispatcher failed to write response (XTLS#153) v1.2.1 Improve configuration detector (cone or symmetric) 16 -> 60, 8 -> 300 Improve the response to UDP Associate in Socks5 Adjust Trojan Outbound postRequest Adjust Trojan & Socks handleUDPPayload Refine Trojan packet reader & writer (XTLS#142) Fix TPROXY UDP/IPv6 Optimized log (XTLS#121) v1.2.0 Enable loading TOML & YAML by confdir (XTLS#120) Change TOML package to github.com/pelletier/go-toml (XTLS#119) Disable session resumption by default Refactor: TPROXY inbound UDP write back VLESS & VMess are not ready to accept FullCone yet Refactor: FullCone TPROXY Inbound & Socks Outbound Refactor: *net.UDPAddr -> *net.Destination Move common/net/connection.go into cnc folder ...
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.
既然buf.Buffer里面已经带了UDPDestination，可以精简掉一些不必要的代码