Mirrors/chatmail-core
1
0
Fork 0
mirror of https://github.com/chatmail/core.git synced 2026-05-08 17:36:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

Implement immediate failing for x.1.1 and x.1.2 status codes too.

Browse Source
This commit is contained in:
jikstra
2021-02-01 13:07:43 +01:00
committed by link2xt
parent 93ba6c1ce8
commit e741cb3646
1 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/job.rs
View File

@@ -291,13 +291,20 @@ impl Job {
// We got a transient 4xx response from SMTP server. // We got a transient 4xx response from SMTP server.
// Give some time until the server-side error maybe goes away. // Give some time until the server-side error maybe goes away.
if let Some(first_word) = response.first_word() {
if response.first_word() == Some("4.1.2") { if first_word.ends_with(".1.1")
// Sometimes we receive the 4.1.2 error, which stays for "Bad destination system address" || first_word.ends_with(".1.2")
// or more simple, the domain doesn't exist or is invalid. This should be a permanent error || first_word.ends_with(".1.3")
// and fail immediatly. {
info!(context, "Smtp-job #{} is a transient \"bad destination\" error. Lets let it fail immediatly.", self.job_id); // Sometimes we receive transient errors that should be permanent.
Status::Finished(Err(format_err!("Permanent SMTP error: {}", err))) // Any extended smtp status codes like x.1.1, x.1.2 or x.1.3 that we
// receive as a transient error are misconfigurations of the smtp server.
// See https://tools.ietf.org/html/rfc3463#section-3.2
info!(context, "Smtp-job #{} Received extended status code {} for a transient error. This looks like a misconfigured smtp server, let's fail immediatly", self.job_id, first_word);
Status::Finished(Err(format_err!("Permanent SMTP error: {}", err)))
} else {
Status::RetryLater
}
} else { } else {
Status::RetryLater Status::RetryLater
} }