Redact authentication logs

Add a boolean flag `authIsActive` to manage redaction of sensitive authentication information in debug logs. When this flag is true, authentication details are replaced with `<auth redacted>`.
This commit is contained in:
Winni Neessen 2024-10-12 20:53:58 +02:00
parent 8faac3d101
commit 7acfe8015d
Signed by: wneessen
GPG key ID: 385AC9889632126E

View file

@ -83,6 +83,7 @@ type Client struct {
// logger will be used for debug logging // logger will be used for debug logging
logger log.Logger logger log.Logger
authIsActive bool
// mutex is used to synchronize access to shared resources, ensuring that only one goroutine can access // mutex is used to synchronize access to shared resources, ensuring that only one goroutine can access
// the resource at a time. // the resource at a time.
@ -174,7 +175,12 @@ func (c *Client) Hello(localName string) error {
func (c *Client) cmd(expectCode int, format string, args ...interface{}) (int, string, error) { func (c *Client) cmd(expectCode int, format string, args ...interface{}) (int, string, error) {
c.mutex.Lock() c.mutex.Lock()
c.debugLog(log.DirClientToServer, format, args...) var logMsg []interface{}
logMsg = args
if c.authIsActive {
logMsg = []interface{}{"<auth redacted>"}
}
c.debugLog(log.DirClientToServer, format, logMsg...)
id, err := c.Text.Cmd(format, args...) id, err := c.Text.Cmd(format, args...)
if err != nil { if err != nil {
c.mutex.Unlock() c.mutex.Unlock()
@ -182,7 +188,12 @@ func (c *Client) cmd(expectCode int, format string, args ...interface{}) (int, s
} }
c.Text.StartResponse(id) c.Text.StartResponse(id)
code, msg, err := c.Text.ReadResponse(expectCode) code, msg, err := c.Text.ReadResponse(expectCode)
c.debugLog(log.DirServerToClient, "%d %s", code, msg)
logMsg = []interface{}{code, msg}
if c.authIsActive && code >= 300 {
logMsg = []interface{}{code, "<auth redacted>"}
}
c.debugLog(log.DirServerToClient, "%d %s", logMsg...)
c.Text.EndResponse(id) c.Text.EndResponse(id)
c.mutex.Unlock() c.mutex.Unlock()
return code, msg, err return code, msg, err
@ -256,6 +267,16 @@ func (c *Client) Auth(a Auth) error {
if err := c.hello(); err != nil { if err := c.hello(); err != nil {
return err return err
} }
c.mutex.Lock()
c.authIsActive = true
c.mutex.Unlock()
defer func() {
c.mutex.Lock()
c.authIsActive = false
c.mutex.Unlock()
}()
encoding := base64.StdEncoding encoding := base64.StdEncoding
mech, resp, err := a.Start(&ServerInfo{c.serverName, c.tls, c.auth}) mech, resp, err := a.Start(&ServerInfo{c.serverName, c.tls, c.auth})
if err != nil { if err != nil {