Skip to content

fix: solve the problem that DNS dialer not perform switching & udp traffic stucks #782

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 23, 2025

Conversation

LostAttractor
Copy link
Contributor

@LostAttractor LostAttractor commented Mar 13, 2025

  1. Fixed the regression introduced by feat(dns): support DoH, DoT, DoH3, DoQ #649
  2. Refactor udp_task_pool to avoid EmitTask stuck or task loss or incorrect task run order

Background

Checklist

Full Changelogs

  • [Implement ...]

Issue Reference

Closes #[issue number]

Test Result

@alexandr2666
Copy link

alexandr2666 commented Mar 16, 2025

DNS使用h3://1.1.1.1/dns-query或者h3://8.8.8.8/dns-query过一会就会出现这样的报错
error.log
https://1.1.1.1/dns-query过一会会出现:

Mar 16 11:15:09 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAB2FwLWdldzQHc3BvdGlmeQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"
Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"
Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AABwAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

@LostAttractor
Copy link
Contributor Author

LostAttractor commented Mar 16, 2025

DNS使用h3://1.1.1.1/dns-query或者h3://8.8.8.8/dns-query过一会就会出现这样的报错

error.log

https://1.1.1.1/dns-query过一会会出现:


Mar 16 11:15:09 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAB2FwLWdldzQHc3BvdGlmeQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AABwAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

从日志来看,优先考虑是你的网或者你的节点炸了
建议出现问题时直接使用

curl -H Host:google.com (google的ip) -v --insecure

看看节点通不通

@alexandr2666
Copy link

DNS使用h3://1.1.1.1/dns-query或者h3://8.8.8.8/dns-query过一会就会出现这样的报错
error.log
https://1.1.1.1/dns-query过一会会出现:


Mar 16 11:15:09 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAB2FwLWdldzQHc3BvdGlmeQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AAAEAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

Mar 16 11:15:34 route-box dae[1440]: level=warning msg="handlePkt: Get "https://1.1.1.1:443/dns-query?dns=AAABAAABAAAAAAAAA2Rucwhtc2Z0bmNzaQNjb20AABwAAQ": dial tcp <nil>->1.1.1.1:443: connect: connection timed out"

从日志来看,优先考虑是你的网或者你的节点炸了 建议出现问题时直接使用

curl -H Host:google.com (google的ip) -v --insecure

看看节点通不通

但炸掉的时候Telegram是能连上的

@LostAttractor LostAttractor marked this pull request as ready for review March 23, 2025 12:31
@LostAttractor LostAttractor requested a review from a team as a code owner March 23, 2025 12:31
@LostAttractor LostAttractor force-pushed the fix_dns branch 2 times, most recently from 16091c3 to 60adf55 Compare March 24, 2025 17:48
@LostAttractor LostAttractor changed the title fix: solve the problem that DNS dialer will not perform switching int… fix: solve the problem that DNS dialer not perform switching & udp traffic stucks Mar 24, 2025
Copy link
Contributor

@dae-prow dae-prow bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧪 Since the PR has been fully tested, please consider merging it.

@MarksonHon MarksonHon merged commit b5f9479 into daeuniverse:main May 23, 2025
34 checks passed
@dae-prow dae-prow bot mentioned this pull request May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants