From ac7c1f28cfaedaf45c8859d3654576fa7a5795af Mon Sep 17 00:00:00 2001 From: Jason Kulatunga Date: Mon, 5 Oct 2020 08:56:43 -0600 Subject: [PATCH] ignore empty ("") errors. --- webapp/backend/pkg/notify/notify.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/webapp/backend/pkg/notify/notify.go b/webapp/backend/pkg/notify/notify.go index c2ae507..3c7a924 100644 --- a/webapp/backend/pkg/notify/notify.go +++ b/webapp/backend/pkg/notify/notify.go @@ -202,22 +202,21 @@ func (n *Notify) SendShoutrrrNotification(shoutrrrUrl string) error { errs := sender.Send(n.Payload.Message, params) if len(errs) > 0 { - n.Logger.Errorf("One or more errors occurred occurred while sending notifications for %s:", shoutrrrUrl) var errstrings []string for _, err := range errs { if err == nil || err.Error() == "" { continue } - n.Logger.Error(err) errstrings = append(errstrings, err.Error()) } //sometimes there are empty errs, we're going to skip them. if len(errstrings) == 0 { return nil } else { + n.Logger.Errorf("One or more errors occurred while sending notifications for %s:", shoutrrrUrl) + n.Logger.Error(errs) return errors.New(strings.Join(errstrings, "\n")) - } } return nil