diff options
| author | Neil Alexander <neilalexander@users.noreply.github.com> | 2020-07-07 16:53:10 +0100 |
|---|---|---|
| committer | Neil Alexander <neilalexander@users.noreply.github.com> | 2020-07-07 16:53:10 +0100 |
| commit | 51fd53294091bcfc45d42dad24cb24e7b24fbd72 (patch) | |
| tree | 7f7408a99b4a8a46dc91cbe342b8de9b8f6ed457 /federationsender | |
| parent | 8e9580852d32f870693103c424a71a30c811fcca (diff) | |
Fix error handling in federationsender
Diffstat (limited to 'federationsender')
| -rw-r--r-- | federationsender/queue/destinationqueue.go | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/federationsender/queue/destinationqueue.go b/federationsender/queue/destinationqueue.go index 845600c9..2e5916e8 100644 --- a/federationsender/queue/destinationqueue.go +++ b/federationsender/queue/destinationqueue.go @@ -397,15 +397,8 @@ func (oq *destinationQueue) nextTransaction( } return true, nil case gomatrix.HTTPError: - // We received a HTTP error back. In this instance we only - // should report an error if - if e.Code >= 400 && e.Code <= 499 { - // We tried but the remote side has sent back a client error. - // It's no use retrying because it will happen again. - return true, nil - } - // Otherwise, report that we failed to send the transaction - // and we will retry again. + // Report that we failed to send the transaction and we + // will retry again, subject to backoff. return false, err default: log.WithFields(log.Fields{ |