Merge pull request #6 from wneessen/fix_sendmailpipe

Address #5
This commit is contained in:
Winni Neessen 2022-05-25 10:53:53 +02:00 committed by GitHub
commit db9358f8b8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

7
msg.go
View file

@ -58,7 +58,7 @@ type Msg struct {
} }
// SendmailPath is the default system path to the sendmail binary // SendmailPath is the default system path to the sendmail binary
const SendmailPath = "/usr/bin/sendmail" const SendmailPath = "/usr/sbin/sendmail"
// MsgOption returns a function that can be used for grouping Msg options // MsgOption returns a function that can be used for grouping Msg options
type MsgOption func(*Msg) type MsgOption func(*Msg)
@ -547,7 +547,10 @@ func (m *Msg) WriteToSendmailWithContext(ctx context.Context, sp string, a ...st
return fmt.Errorf("sendmail command failed: %s", serr) return fmt.Errorf("sendmail command failed: %s", serr)
} }
// Wait for completion or cancellation of the sendmail executable // Close STDIN and wait for completion or cancellation of the sendmail executable
if err := si.Close(); err != nil {
return fmt.Errorf("failed to close STDIN pipe: %w", err)
}
if err := ec.Wait(); err != nil { if err := ec.Wait(); err != nil {
return fmt.Errorf("sendmail command execution failed: %w", err) return fmt.Errorf("sendmail command execution failed: %w", err)
} }