fix: hotStandby - set heartbeats state no matter secondary's state

nrkno · Oct 28, 2024 · c2b35be · c2b35be
1 parent f550bd5
commit c2b35be
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/packages/connector/src/MosConnection.ts b/packages/connector/src/MosConnection.ts
@@ -178,15 +178,16 @@ export class MosConnection extends EventEmitter<MosConnectionEvents> implements
 			// the secondary connection when the primary is connected
 			// And disable heartbeats on the primary when the primary is disconnected
 			if (connectionOptions.secondary?.openMediaHotStandby) {
+				// Initially disable heartbeats on secondary since primary should be attempted first
+				secondary.disableHeartbeats()
+
 				primary.on('connectionChanged', () => {
-					if (secondary) {
-						if (primary.connected) {
-							secondary.disableHeartbeats()
-							primary.enableHeartbeats()
-						} else {
-							secondary.enableHeartbeats()
-							primary.disableHeartbeats()
-						}
+					if (primary.connected) {
+						secondary?.disableHeartbeats()
+						primary.enableHeartbeats()
+					} else {
+						secondary?.enableHeartbeats()
+						primary.disableHeartbeats()
 					}
 				})
 			}