embarklabs
/
embark
mirror of https://github.com/embarklabs/embark.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

fix(@embark/proxy): only up event listeners on available providers 

Only the  Ganache provider has the setMaxListener
This commit is contained in:
Jonathan Rainville 2020-02-18 09:56:42 -05:00 committed by Iuri Matias
parent e5b15c3342
commit caae92281d
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/stack/proxy/src/proxy.js
View File

@ -42,8 +42,9 @@ export class Proxy {
_createWeb3RequestManager(provider) {
const manager = new Web3RequestManager.Manager(provider);
// Up max listener because the default 10 limit is too low for all the events the proxy handles
// Warning mostly appeared in tests
manager.provider.setMaxListeners(100);
// Warning mostly appeared in tests. The warning is also only with the Ganache provider
// eslint-disable-next-line no-unused-expressions
manager.provider?.setMaxListeners?.(100);
return manager;
}
@ -53,6 +54,7 @@ export class Proxy {
// Using net_version instead of eth_accounts, because eth_accounts can fail if EIP1102 is not approved first
await reqMgr.send({ method: 'net_version' });
} catch (e) {
this.logger.debug(e);
throw new Error(__(`Unable to connect to the blockchain endpoint`));
}
}