vikunja
/
vikunja
6
43
Fork 70
Code Issues 150 Pull Requests 11 Packages Releases 32 Activity

fix(mail): don't try to authenticate against the mail server when no credentials are provided
continuous-integration/drone/push Build is passing Details 

Related to https://github.com/go-vikunja/api/issues/34
This commit is contained in:
kolaente 2022-07-12 15:46:21 +02:00
parent 4bb77b5539
commit c906fc2b07
Signed by: konrad
GPG Key ID: F40E70337AB24C9B
1 changed files with 14 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
pkg/mail/mail.go
View File

@ -47,17 +47,27 @@ func getClient() (*mail.Client, error) {
tlsPolicy = mail.TLSMandatory
}
return mail.NewClient(
config.MailerHost.GetString(),
opts := []mail.Option{
mail.WithSMTPAuth(authType),
mail.WithUsername(config.MailerUsername.GetString()),
mail.WithPassword(config.MailerPassword.GetString()),
mail.WithPort(config.MailerPort.GetInt()),
mail.WithTLSPolicy(tlsPolicy),
//#nosec G402
mail.WithTLSConfig(&tls.Config{
InsecureSkipVerify: config.MailerSkipTLSVerify.GetBool(),
}),
}
if config.MailerUsername.GetString() != "" {
opts = append(opts, mail.WithUsername(config.MailerUsername.GetString()))
}
if config.MailerPassword.GetString() != "" {
opts = append(opts, mail.WithPassword(config.MailerPassword.GetString()))
}
return mail.NewClient(
config.MailerHost.GetString(),
opts...,
)
}