From 47c39f6d382c496cb964262adcf76cc8dbb96da3 Mon Sep 17 00:00:00 2001 From: Elias Schneider Date: Fri, 24 Jan 2025 10:36:16 +0100 Subject: [PATCH] fix: use OS hostname for SMTP EHLO message --- backend/internal/service/email_service.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/backend/internal/service/email_service.go b/backend/internal/service/email_service.go index fbe2efc..7859dc1 100644 --- a/backend/internal/service/email_service.go +++ b/backend/internal/service/email_service.go @@ -14,6 +14,7 @@ import ( "net" "net/smtp" "net/textproto" + "os" ttemplate "text/template" "time" ) @@ -117,8 +118,12 @@ func SendEmail[V any](srv *EmailService, toEmail email.Address, template email.T } defer client.Close() - if err := client.Hello(common.EnvConfig.Host); err != nil { - return fmt.Errorf("failed to say hello to SMTP server: %w", err) + // Set the hello message manually as for example Google rejects the default "localhost" value + hostname, err := os.Hostname() + if err == nil { + if err := client.Hello(hostname); err != nil { + return fmt.Errorf("failed to say hello to SMTP server: %w", err) + } } smtpUser := srv.appConfigService.DbConfig.SmtpUser.Value