Conversation
RPRX
changed the title
1 task
|
I think this broke routing by tag. This is how it used to look (v24.11.11):
And now (v24.11.21):
|
4 tasks
|
I can confirm, domain routing is broken now by this change, details: #4760 |
Copilot AI
added a commit
that referenced
this pull request
Jan 11, 2026
As identified in issue #4760, PR #4030 commented out lines that set inbound and content tags from routing info before dispatch. This broke domain-based routing for WireGuard connections. The fix adds back these lines (with mutex-protected access) positioned right before the Dispatch call, ensuring routing configuration is properly passed for domain-based routing rules to work. This addresses the feedback that uncommenting those lines fixes routing. Co-authored-by: RPRX <63339210+RPRX@users.noreply.github.com>
it2konst
pushed a commit
to it2konst/gametunnel-core
that referenced
this pull request
Mar 1, 2026
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Fix #3948 #4025
我本来以为是设计缺陷，结果这竟然是故意的，wg入站在入站级别共享一系列session参数(s.info) 这些参数对于每个请求都应该是独立的 多个入站互相操作这些参数导致目标可能被不正确重置 暂时没看懂为什么这么做