Skip to content

Fix issues related to android client#4616

Merged
yuhan6665 merged 2 commits intoXTLS:mainfrom
Cl-He-O:xivpn
Apr 14, 2025
Merged

Fix issues related to android client#4616
yuhan6665 merged 2 commits intoXTLS:mainfrom
Cl-He-O:xivpn

Conversation

@Cl-He-O
Copy link
Contributor

@Cl-He-O Cl-He-O commented Apr 14, 2025

Commit https://github.com/Cl-He-O/Xray-core/commit/690d76fb9ea87fcb18ee50a6d938df99de1c4117 fix the issue described in #4613.
Commit https://github.com/Cl-He-O/Xray-core/commit/115fa4449b36e81751637600d9254ac11c21d500 fix issue that dialer controllers are not called, introduced by 8284a0e. Dialer controllers are used in an android client to protect socket from looping back.

@Fangliding
Copy link
Member

controllers 确实是这样
4613 测试过了吗?

@Cl-He-O
Copy link
Contributor Author

Cl-He-O commented Apr 14, 2025

测过,确定修了。

@Fangliding Fangliding linked an issue Apr 14, 2025 that may be closed by this pull request
4 tasks
@yuhan6665 yuhan6665 merged commit 7a2f42f into XTLS:main Apr 14, 2025
35 checks passed
@yuhan6665
Copy link
Member

感谢佬!
这些修复完全可以发两个pr

Exclude0122 pushed a commit to Exclude0122/Xray-core that referenced this pull request Apr 14, 2025
* WireGuard: Fix tunnel not closed

* Dialer: Apply controllers in lc.Control
Exclude0122 added a commit to Exclude0122/Xray-core that referenced this pull request Apr 14, 2025
* WireGuard: Fix tunnel not closed

* Dialer: Apply controllers in lc.Control
maoxikun pushed a commit to maoxikun/Xray-core that referenced this pull request Aug 23, 2025
* WireGuard: Fix tunnel not closed

* Dialer: Apply controllers in lc.Control

(cherry picked from commit 7a2f42f)
it2konst pushed a commit to it2konst/gametunnel-core that referenced this pull request Mar 1, 2026
* WireGuard: Fix tunnel not closed

* Dialer: Apply controllers in lc.Control
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.

core.Instance.Close() does not close wireguard client

3 participants