From bb3d51652df6b28f9298afc990010c76886e2d99 Mon Sep 17 00:00:00 2001
From: Kegan Dougal <kegan@matrix.org>
Date: Thu, 13 Oct 2022 10:52:40 +0100
Subject: [PATCH] Consolidate error handling retry logic

---
 src/sliding-sync.ts | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/sliding-sync.ts b/src/sliding-sync.ts
index 235f8ad40..9479991aa 100644
--- a/src/sliding-sync.ts
+++ b/src/sliding-sync.ts
@@ -846,17 +846,15 @@ export class SlidingSync extends TypedEventEmitter<SlidingSyncEvent, SlidingSync
                         currentPos = undefined;
                         await sleep(50); // in case the 400 was for something else; don't tightloop
                         continue;
-                    }
-                    await sleep(5000);
+                    } // else fallthrough to generic error handling
                 } else if (this.needsResend || err === "aborted") {
                     // don't sleep as we caused this error by abort()ing the request.
                     // we check for 'aborted' because that's the error Jest returns and without it
                     // we get warnings about not exiting fast enough.
                     continue;
-                } else {
-                    logger.error(err);
-                    await sleep(5000);
                 }
+                logger.error(err);
+                await sleep(5000);
             }
             if (!resp) {
                 continue;