From b4d157c6cfc73220207ce8ba123bea553e0b24ab Mon Sep 17 00:00:00 2001 From: Neil Alexander Date: Fri, 7 Aug 2020 17:20:17 +0100 Subject: [PATCH] Log why we're blacklisting --- federationsender/queue/destinationqueue.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/federationsender/queue/destinationqueue.go b/federationsender/queue/destinationqueue.go index 021444d29..0b6f539df 100644 --- a/federationsender/queue/destinationqueue.go +++ b/federationsender/queue/destinationqueue.go @@ -275,7 +275,7 @@ func (oq *destinationQueue) backgroundSend() { // has exceeded a maximum allowable value. Clean up the in-memory // buffers at this point. The PDU clean-up is already on a defer. oq.cleanPendingInvites() - log.Infof("Blacklisting %q due to errors", oq.destination) + log.WithError(terr).Warnf("Blacklisting %q due to error", oq.destination) return } else { // We haven't been told to give up terminally yet but we still have @@ -303,7 +303,7 @@ func (oq *destinationQueue) backgroundSend() { if giveUp := oq.statistics.Failure(); giveUp { // It's been suggested that we should give up because // the backoff has exceeded a maximum allowable value. - log.Infof("Blacklisting %q due to errors", oq.destination) + log.WithError(ierr).Warnf("Blacklisting %q due to error", oq.destination) return } } else if sent > 0 {