Skip to content

Comments

DNS: Check err for UDP dns.PackMessage(req.msg)#5512

Merged
RPRX merged 1 commit intomainfrom
dns-invalid-fqdn
Jan 9, 2026
Merged

DNS: Check err for UDP dns.PackMessage(req.msg)#5512
RPRX merged 1 commit intomainfrom
dns-invalid-fqdn

Conversation

@Fangliding
Copy link
Member

fix #5506

@RPRX
Copy link
Member

RPRX commented Jan 9, 2026

如果有应用对着某个 IP 发不符合规范的 SNI 然后 Xray 又 sniff+query 应该能被应用探测出来是 Xray。。。先不管这种情况吧

@RPRX RPRX changed the title Check err in udp dns DNS: Check err for UDP dns.PackMessage(req.msg) Jan 9, 2026
@RPRX RPRX merged commit 07a0daf into main Jan 9, 2026
78 checks passed
@Fangliding
Copy link
Member Author

Fangliding commented Jan 9, 2026

如果有应用对着某个 IP 发不符合规范的 SNI 然后 Xray 又 sniff+query 应该能被应用探测出来是 Xray。。。先不管这种情况吧

那么麻烦干什么 对着自家服务器发一个tls sni请求发现被重解析了就99%是翻墙软件环境了 还有经典的国内外不同出口检测

目前个人标准是全局且浏览器环境下不要能稳定探测出来(eg. 活生生扫出本地clash面板 不过最近chrome好像在采取措施) 就好了 这个不影响(非法的域名本来就是连不上)

顺便其他DNS类型都没这个问题 就classic模式忽略了这个err返回

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.

Malformed domain in the inbound request causes a panic

2 participants