From 6749dd40b31dbcfae2eb91deb93989899b868617 Mon Sep 17 00:00:00 2001 From: Sean Liao Date: Tue, 8 Feb 2022 22:34:13 +0100 Subject: [PATCH] net/smtp: match actual behavior for Auth.Start Looking at history, it appears to never have worked as documented. Fixes #48759 Change-Id: I066307c28e3ed1875c1c4049bade62e2818dd400 Reviewed-on: https://go-review.googlesource.com/c/go/+/383998 Reviewed-by: Ian Lance Taylor Trust: Cherry Mui --- src/net/smtp/auth.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/net/smtp/auth.go b/src/net/smtp/auth.go index fd1a472f930..7a32ef6a2e6 100644 --- a/src/net/smtp/auth.go +++ b/src/net/smtp/auth.go @@ -16,8 +16,7 @@ type Auth interface { // Start begins an authentication with a server. // It returns the name of the authentication protocol // and optionally data to include in the initial AUTH message - // sent to the server. It can return proto == "" to indicate - // that the authentication should be skipped. + // sent to the server. // If it returns a non-nil error, the SMTP client aborts // the authentication attempt and closes the connection. Start(server *ServerInfo) (proto string, toServer []byte, err error)