fix: getHealth returns 206 status code if EL is optimistic or offline (…

…#6920) * fix: getHealth returns 206 status code if EL is syncing / optimistic * Report 206 if EL is offline
ChainSafe · Jul 5, 2024 · d56e871 · d56e871
1 parent 0c810b8
commit d56e871
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/beacon-node/src/api/impl/node/index.ts b/packages/beacon-node/src/api/impl/node/index.ts
@@ -71,7 +71,9 @@ export function getNodeApi(
  throw new ApiError(400, `Invalid syncing status code: ${syncingStatus}`);
  }
 
- if (sync.getSyncStatus().isSyncing) {
+ const {isSyncing, isOptimistic, elOffline} = sync.getSyncStatus();
+
+ if (isSyncing || isOptimistic || elOffline) {
  // 206: Node is syncing but can serve incomplete data
  return {status: syncingStatus ?? routes.node.NodeHealth.SYNCING};
  } else {