Allow createInfuraMiddleware to take an RPC service #116
Conversation
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring: Next stepsTake a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with |
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
mcmire
force-pushed
the
integrate-rpc-service
branch
from
dae01b6 to
b0ea84d
Compare
|
@SocketSecurity ignore npm/[email protected] New author is OK. Either it's us or known members of the NPM community. |
mcmire
force-pushed
the
integrate-rpc-service
branch
3 times, most recently
from
9128d9b to
089e660
Compare
|
@SocketSecurity ignore npm/@metamask/[email protected] These are ours. |
mcmire
force-pushed
the
integrate-rpc-service
branch
2 times, most recently
from
d12e9a4 to
77e4bfc
Compare
`@metamask/network-controller` now contains an `RpcService` class. Using this class not only allows us to remove a lot of code from this package, as it incorporates the vast majority of logic contained in `createInfuraMiddleware`, including the retry logic, but it also allows us to use the circuit breaker pattern and the exponential backoff pattern to prevent too many retries if the network is unreliable, and then automatically cut over to a failover node when the network truly goes down.
mcmire
force-pushed
the
integrate-rpc-service
branch
from
77e4bfc to
c88e9c7
Compare
|
@SocketSecurity ignore npm/@metamask/[email protected] This is our package. |
| * The interface for a service class responsible for making a request to an RPC | ||
| * endpoint. | ||
| */ | ||
| export type AbstractRpcService = { |
There was a problem hiding this comment.
We could get this from @metamask/network-controller, but that would create a circular dependency.
@metamask/network-controllernow contains anRpcServiceclass. Using this class not only allows us to remove a lot of code from this package, as it incorporates the vast majority of logic contained increateInfuraMiddleware, including the retry logic, but it also allows us to use the circuit breaker pattern and the exponential backoff pattern to prevent too many retries if the network is unreliable, and then automatically cut over to a failover node when the network truly goes down.Closes #115.
Closes #77.