Skip to content

Conversation

@arkxfly
Copy link
Contributor

@arkxfly arkxfly commented Sep 20, 2025

No description provided.

@codecov
Copy link

codecov bot commented Sep 20, 2025

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
clash-lib/src/app/dns/config.rs 33.33% 0 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

@Itsusinn
Copy link
Member

Itsusinn commented Sep 21, 2025

目前dns还没支持分阶段的解析(用于初始化的dns resolver和正常resolver)

考虑到还有通过proxy进行dns resolve的行为,dns resolver自身的初始化可能更加复杂了

@ibigbug ibigbug merged commit f613a53 into Watfaq:master Sep 30, 2025
33 of 34 checks passed
@ibigbug
Copy link
Member

ibigbug commented Sep 30, 2025

再发一个新的pr吧

@ibigbug
Copy link
Member

ibigbug commented Oct 1, 2025

什么意思?

@arkxfly arkxfly deleted the fix_dns branch October 1, 2025 04:40
@ibigbug
Copy link
Member

ibigbug commented Oct 1, 2025

Sorry I don't get it.

@arkxfly arkxfly changed the title fix(dns): Fix ip address nameserver checking . Oct 1, 2025
@ibigbug
Copy link
Member

ibigbug commented Oct 1, 2025

You can create a PR to revert it. Unfortunately I won't a dirty force push to master branch for a repository like this that already has many contributors.

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.

4 participants