starred/shadowsocks-rust
1
0
Fork 0
mirror of https://github.com/shadowsocks/shadowsocks-rust.git synced 2026-11-06 08:14:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: domain name comparison in should_forward_by_query (#1953)

Browse Source
This commit is contained in:
Mygod
2025-05-12 09:57:07 +08:00
committed by GitHub
parent 34ebd07a4d
commit 07caa8a96a
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/shadowsocks-service/src/local/dns/server.rs
+7 -1
View File
@@ -594,13 +594,19 @@ fn should_forward_by_query(context: &ServiceContext, balancer: &PingBalancer, qu
if let ServerAddr::DomainName(dn, ..) = svr_cfg.addr() {
// Convert domain name to `Name`
// Ignore it if error occurs
if let Ok(name) = Name::from_str(dn) {
if let Ok(mut name) = Name::from_str(dn) {
// cmp will handle FQDN in case insensitive way
if let Ordering::Equal = query.name().cmp(&name) {
// It seems that query is for this server, just bypass it to local resolver
trace!("DNS querying name {} of server {:?}", query.name(), svr_cfg);
return Some(false);
}
// test it again with fqdn set
name.set_fqdn(true);
if let Ordering::Equal = query.name().cmp(&name) {
trace!("DNS querying name {} of server {:?}", query.name(), svr_cfg);
return Some(false);
}
}
}
}