hculpan/areweplaying
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Log error when sendEmailHandler fails to send an email

Browse source 
Send Reminder failures were silently swallowed, only surfacing in the
HTTP response and never in the application log, unlike the GM
notification email flows.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Harry Culpan 2026-07-04 21:33:09 -04:00
parent ee4581ed7e
commit 5f1a48b1be
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cmd/web/sendemailhandler.go
View file

@ -2,6 +2,7 @@ package main
import ( import (
"context" "context"
"log"
"net/http" "net/http"
"strings" "strings"
@ -29,6 +30,7 @@ func sendEmailHandler(w http.ResponseWriter, r *http.Request) {
// log.Default().Printf("got email: To: %s, From: %s, Subject: %q, Body: %q", to, from, subject, body) // log.Default().Printf("got email: To: %s, From: %s, Subject: %q, Body: %q", to, from, subject, body)
err := email.SendEmail(e) err := email.SendEmail(e)
if err != nil { if err != nil {
log.Default().Printf("ERROR sending email: %s", err)
http.Error(w, err.Error(), http.StatusInternalServerError) http.Error(w, err.Error(), http.StatusInternalServerError)
return return
} }