cqrenet/relay
5
0
Fork 0
mirror of https://github.com/chatmail/relay.git synced 2026-08-10 18:40:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Implement missing "implicit MX" rule

Browse Source 
Implements "implicit MX" as described by
https://datatracker.ietf.org/doc/html/rfc5321#section-5.1

Fixes: #126
Signed-off-by: Jagoda Ślązak <jslazak@jslazak.com>
This commit is contained in:
Jagoda Ślązak
2026-04-27 14:44:21 +02:00
parent c1a4d99d2e
commit 21aaea0d92
1 changed files with 26 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files
filtermail/src/transport.rs
+26 -18
View File
@@ -59,25 +59,33 @@ impl TransportHandler {
allow_invalid_cert = true;
}
let query = format!("{mx_domain}.");
let mx_records = dns_resolver.mx_lookup(query).await.map_err(|e| {
if e.is_no_records_found() {
format!("512 No MX records for {mx_domain}")
} else if e.is_nx_domain() {
format!("512 Domain {mx_domain} does not exist")
} else {
format!("421 DNS resolution failed for {mx_domain}")
}
})?;
let mut hosts: Vec<(u16, String)> = mx_records
.iter()
.map(|mx| {
let host = mx.exchange().to_string().trim_end_matches('.').to_string();
(mx.preference(), host)
})
.collect();
hosts.sort();
hosts
match dns_resolver.mx_lookup(query).await {
Ok(mx_records) => {
let mut hosts: Vec<(u16, String)> = mx_records
.iter()
.map(|mx| {
let host =
mx.exchange().to_string().trim_end_matches('.').to_string();
(mx.preference(), host)
})
.collect();
hosts.sort();
hosts
}
Err(e) => {
if e.is_no_records_found() {
// "implicit MX" as described by section 5.1 of RFC5321
// https://datatracker.ietf.org/doc/html/rfc5321#section-5.1
log::debug!("No MX record found, using implicit MX: {mx_domain}");
vec![(0, mx_domain)]
} else if e.is_nx_domain() {
return Err(format!("512 Domain {mx_domain} does not exist"));
} else {
return Err(format!("421 DNS resolution failed for {mx_domain}"));
}
}
}
}
};