fix: add check for sendAsync method inside middleware

rarible · Jan 24, 2022 · 82bd5cb · 82bd5cb
1 parent 244b981
commit 82bd5cb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/packages/estimate-middleware/src/index.ts b/packages/estimate-middleware/src/index.ts
@@ -2,12 +2,16 @@ import { createFetchMiddleware, providerAsMiddleware, providerFromEngine } from
 import { JsonRpcEngine } from "json-rpc-engine"
 import type { SafeEventEmitterProvider } from "eth-json-rpc-middleware/dist/utils/cache"
 import { createEstimateGasMiddleware } from "./middleware"
+import { isSafeEventEmitterProvider } from "./utils"
 
 export function estimate(provider: any, estimate?: JsonRpcEngine | string, force?: boolean): SafeEventEmitterProvider {
-	const engine = new JsonRpcEngine()
-	engine.push(createEstimateGasMiddleware(getEstimateEngine(provider, estimate), force))
-	engine.push(providerAsMiddleware(provider))
-	return providerFromEngine(engine)
+	if (isSafeEventEmitterProvider(provider)) {
+		const engine = new JsonRpcEngine()
+		engine.push(createEstimateGasMiddleware(getEstimateEngine(provider, estimate), force))
+		engine.push(providerAsMiddleware(provider))
+		return providerFromEngine(engine)
+	}
+	return provider
 }
 
 function getEstimateEngine(provider: any, estimate: JsonRpcEngine | string | undefined): JsonRpcEngine {

diff --git a/packages/estimate-middleware/src/utils.ts b/packages/estimate-middleware/src/utils.ts
@@ -0,0 +1,5 @@
+import type { SafeEventEmitterProvider } from "eth-json-rpc-middleware/dist/utils/cache"
+
+export function isSafeEventEmitterProvider(x: unknown): x is SafeEventEmitterProvider {
+	return typeof x === "object" && x !== null && "send" in x && "sendAsync" in x
+}