Skip to content

feat(request-node): allow ws connection#691

Merged
alexandre-abrioux merged 4 commits intomasterfrom
node-ws
Dec 1, 2021
Merged

feat(request-node): allow ws connection#691
alexandre-abrioux merged 4 commits intomasterfrom
node-ws

Conversation

@alexandre-abrioux
Copy link
Contributor

Description of the changes

  • Allow usage of websocket connection string in WEB3_PROVIDER_URL or argv.providerUrl for the request-node
  • Handles heartbeats/keepalive

Comment on lines +36 to +38
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore
providerOrUrl: new Web3WsProvider(config.getStorageWeb3ProviderUrl(), {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 89.181% when pulling b7b3cd4 on node-ws into f224545 on master.

@alexandre-abrioux alexandre-abrioux merged commit 2c879a6 into master Dec 1, 2021
@alexandre-abrioux alexandre-abrioux deleted the node-ws branch December 1, 2021 09:31
@bertux bertux removed their request for review December 2, 2021 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants