Mirrors/go-chasquid-smtp
1
0
Fork 0
mirror of https://blitiri.com.ar/repos/chasquid synced 2025-12-17 14:37:02 +00:00
Code Releases Activity

smtp-check: Improve error reporting

Browse Source 
smtp-check exits on the first error, which is not ideal when
troubleshooting, as seeing only one error can mask others, or make it
more difficult to find the underlying cause.

This patch improves how smtp-check reports errors by tweaking the
presentation a bit, as well as perform almost all checks regardless of
whether they pass or not.
This commit is contained in:
Alberto Bertogli
2019-10-19 14:06:26 +01:00
parent 0bf8f72c73
commit 27227986ab
1 changed files with 24 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
cmd/smtp-check/smtp-check.go
View File

@@ -8,6 +8,7 @@ import (
"context" "context"
"crypto/tls" "crypto/tls"
"flag" "flag"
"fmt"
"log" "log"
"net" "net"
"net/smtp" "net/smtp"
@@ -54,6 +55,7 @@ func main() {
} }
log.Printf("OK") log.Printf("OK")
} }
log.Printf("")
mxs, err := net.LookupMX(domain) mxs, err := net.LookupMX(domain)
if err != nil { if err != nil {
@@ -64,8 +66,9 @@ func main() {
log.Fatalf("MX lookup returned no results") log.Fatalf("MX lookup returned no results")
} }
errs := []error{}
for _, mx := range mxs { for _, mx := range mxs {
log.Printf("=== Testing MX: %2d %s", mx.Pref, mx.Host) log.Printf("=== MX: %2d %s", mx.Pref, mx.Host)
ips, err := net.LookupIP(mx.Host) ips, err := net.LookupIP(mx.Host)
if err != nil { if err != nil {
@@ -73,9 +76,10 @@ func main() {
} }
for _, ip := range ips { for _, ip := range ips {
result, err := spf.CheckHostWithSender(ip, domain, "test@"+domain) result, err := spf.CheckHostWithSender(ip, domain, "test@"+domain)
log.Printf("SPF %v for %v: %v", result, ip, err)
if result != spf.Pass { if result != spf.Pass {
log.Printf("SPF check != pass for IP %s: %s - %s", errs = append(errs,
ip, result, err) fmt.Errorf("%s: SPF failed (%v)", mx.Host, ip))
} }
} }
@@ -94,19 +98,21 @@ func main() {
} }
err = c.StartTLS(config) err = c.StartTLS(config)
if err != nil { if err != nil {
log.Fatalf("TLS error: %v", err) log.Printf("TLS error: %v", err)
errs = append(errs, fmt.Errorf("%s: TLS failed", mx.Host))
} else {
cstate, _ := c.TLSConnectionState()
log.Printf("TLS OK: %s - %s", tlsconst.VersionName(cstate.Version),
tlsconst.CipherSuiteName(cstate.CipherSuite))
} }
cstate, _ := c.TLSConnectionState()
log.Printf("TLS OK: %s - %s", tlsconst.VersionName(cstate.Version),
tlsconst.CipherSuiteName(cstate.CipherSuite))
c.Close() c.Close()
} }
if policy != nil { if policy != nil {
if !policy.MXIsAllowed(mx.Host) { if !policy.MXIsAllowed(mx.Host) {
log.Fatalf("NOT allowed by STS policy") log.Printf("NOT allowed by STS policy")
errs = append(errs, fmt.Errorf("%s: STS failed", mx.Host))
} }
log.Printf("Allowed by policy") log.Printf("Allowed by policy")
} }
@@ -114,5 +120,13 @@ func main() {
log.Printf("") log.Printf("")
} }
log.Printf("=== Success") if len(errs) == 0 {
log.Printf("=== Success")
} else {
log.Printf("=== FAILED")
for _, err := range errs {
log.Printf("%v", err)
}
log.Fatal("")
}
} }