From f5b169cbd4125004251dddbf66967dd4a3824398 Mon Sep 17 00:00:00 2001 From: Julian Kornberger Date: Tue, 4 Feb 2020 01:52:37 +0100 Subject: [PATCH] Trim SMTP response (#237) this avoids to have \r\n in the error message --- modules/smtp/scanner.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/modules/smtp/scanner.go b/modules/smtp/scanner.go index 3c9cc82..9ff5d7d 100644 --- a/modules/smtp/scanner.go +++ b/modules/smtp/scanner.go @@ -29,6 +29,7 @@ import ( "errors" "fmt" "strconv" + "strings" log "github.com/sirupsen/logrus" "github.com/zmap/zgrab2" @@ -265,7 +266,7 @@ func (scanner *Scanner) Scan(target zgrab2.ScanTarget) (zgrab2.ScanStatus, inter return zgrab2.TryGetScanStatus(err), result, err } if code < 200 || code >= 300 { - return zgrab2.SCAN_APPLICATION_ERROR, result, fmt.Errorf("SMTP error code %d returned from STARTTLS command (%s)", code, ret) + return zgrab2.SCAN_APPLICATION_ERROR, result, fmt.Errorf("SMTP error code %d returned from STARTTLS command (%s)", code, strings.TrimSpace(ret)) } tlsConn, err := scanner.config.TLSFlags.GetTLSConnection(conn.Conn) if err != nil {