Skip using safe tag for cronos (#193)

* Skip using safe tag for cronos * Update changelog
subquery · Oct 27, 2023 · ededfd6 · ededfd6
1 parent e53a47f
commit ededfd6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/node/CHANGELOG.md b/packages/node/CHANGELOG.md
@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- Unfinalized blocks not working with Cronos (#193)
 
 ## [3.1.1] - 2023-10-25
 ### Fixed

diff --git a/packages/node/src/ethereum/api.ethereum.ts b/packages/node/src/ethereum/api.ethereum.ts
@@ -223,7 +223,9 @@ export class EthereumApi implements ApiWrapper {
   }
 
   async getBestBlockHeight(): Promise<number> {
-    const tag = this.supportsFinalization ? 'safe' : 'latest';
+    // Cronos "safe" tag doesn't currently work as indended
+    const tag =
+      this.supportsFinalization && this.chainId !== 25 ? 'safe' : 'latest';
     return (await this.client.getBlock(tag)).number;
   }