Analysis of Naive Inbound Implementation #1
Conversation
We mistakenly believed that `libresolv`'s `search` function worked correctly in NetworkExtension, but it seems only `getaddrinfo` does. This commit changes the behavior of the `local` DNS server in NetworkExtension to prefer DHCP, falling back to `getaddrinfo` if DHCP servers are unavailable. It's worth noting that `prefer_go` does not disable DHCP since it respects Dial Fields, but `getaddrinfo` does the opposite. The new behavior only applies to NetworkExtension, not to all scenarios (primarily command-line binaries) as it did previously. In addition, this commit also improves the DHCP DNS server to use the same robust query logic as `local`.
We do not have the `com.apple.developer.networking.multicast` entitlement and are unable to obtain it for non-technical reasons.
…for local DNS server
Provides a detailed analysis of the `naive` inbound in `sing-box`. The analysis confirms that the `naive` inbound is a pure Go implementation and does not depend on any external C++ libraries. It re-implements the NaiveProxy protocol to handle traffic obfuscation.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. New to Jules? Learn more at jules.google/docs. |
d3vw
force-pushed
the
dev-next
branch
2 times, most recently
from
23ce4e6 to
5f739c4
Compare
This submission provides a detailed analysis of how the
naiveinbound is handled insing-box, confirming it is a pure Go implementation without any C++ dependencies.PR created automatically by Jules for task 18209831349324103181 started by @juiiyang