~bigbes/core-go

a941162edd28c03abb3a10fcddd4ec4bdf651946 — Simon Ser 3 years ago 1fd2476
server/email: don't panic in EmailRecover

A panic in here is not great, because this function is supposed to
be the panic handler. Let's just log the error instead.
1 files changed, 2 insertions(+), 1 deletions(-)

M server/email.go
M server/email.go => server/email.go +2 -1
@@ 60,7 60,8 @@ func EmailRecover(ctx context.Context, _origErr interface{}) error {
	}
	rcpt, err := gomail.ParseAddress(to)
	if err != nil {
		panic(errors.New("Failed to parse sender address"))
		log.Printf("Warning: mail::error-to is invalid: %v", err)
		return fmt.Errorf("internal system error")
	}
	addr := mail.Address(*rcpt)
	header.SetAddressList("To", []*mail.Address{&addr})