Merge pull request paraswap#257 from paraswap/fix/826-curve-zero-divi…

…sion BACK-826: Caught zero division errors for CurveV1Factory
Native-org · Dec 9, 2022 · c6bf0d9 · c6bf0d9
2 parents 6543f0f + 08e63d3
commit c6bf0d9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@paraswap/dex-lib",
-  "version": "2.9.25",
+  "version": "2.9.26",
   "main": "build/index.js",
   "types": "build/index.d.ts",
   "repository": "https://github.com/paraswap/paraswap-dex-lib",

diff --git a/src/dex/curve-v1-factory/price-handlers/price-handler.ts b/src/dex/curve-v1-factory/price-handlers/price-handler.ts
@@ -96,6 +96,14 @@ export class PriceHandler {
             e.message.startsWith(CONVERGENCE_ERROR_PREFIX)
           ) {
             this.logger.trace(`Convergence Error: `, e);
+          } else if (
+            e instanceof Error &&
+            e.message.startsWith(`Division by zero`)
+          ) {
+            this.logger.trace(
+              `Zero division Error suppressed because mostly it is expected`,
+              e,
+            );
           } else {
             this.logger.error(`Unexpected error while calculating price: `, e);
           }