wneessen/go-mail
1
1
Fork 0
mirror of https://github.com/wneessen/go-mail.git synced 2024-11-15 02:12:55 +01:00
Code Issues Projects Releases Packages Wiki Activity

Remove redundant empty string check in SCRAM normalization

Browse source 
The existing check for an empty normalized string is unnecessary because the OpaqueString profile in precis already throws an error if an empty string is returned: https://cs.opensource.google/go/x/text/+/master:secure/precis/profiles.go;l=66
This commit is contained in:
Winni Neessen 2024-11-08 22:44:10 +01:00
parent d6f256c29e
commit a1efa1a1ca
Signed by: wneessen
GPG key ID: 385AC9889632126E
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
smtp/auth_scram.go
View file

@ -310,8 +310,5 @@ func (a *scramAuth) normalizeString(s string) (string, error) {
if err != nil { if err != nil {
return "", fmt.Errorf("failed to normalize string: %w", err) return "", fmt.Errorf("failed to normalize string: %w", err)
} }
if s == "" {
return "", errors.New("normalized string is empty")
}
return s, nil return s, nil
} }