Fix bug in calculating the federation retry backoff period (#6025)

matrix-org · Feb 25, 2020 · d8e34ad · d8e34ad
2 parents 1b2c568 + 0388bea
commit d8e34ad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/changelog.d/6025.bugfix b/changelog.d/6025.bugfix
@@ -0,0 +1 @@
+Fix bug in calculating the federation retry backoff period.
diff --git a/synapse/util/retryutils.py b/synapse/util/retryutils.py
@@ -193,8 +193,9 @@ def __exit__(self, exc_type, exc_val, exc_tb):
         else:
             # We couldn't connect.
             if self.retry_interval:
-                self.retry_interval *= RETRY_MULTIPLIER
-                self.retry_interval *= int(random.uniform(0.8, 1.4))
+                self.retry_interval = int(
+                    self.retry_interval * RETRY_MULTIPLIER * random.uniform(0.8, 1.4)
+                )
 
                 if self.retry_interval >= MAX_RETRY_INTERVAL:
                     self.retry_interval = MAX_RETRY_INTERVAL
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix bug in calculating the federation retry backoff period.