-
-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Infura API v3 #32
Conversation
6b3b5d0
to
26907f4
Compare
I've created #33 to note that rate limits aren't supported yet |
I've created #34 to note that currently requests cannot be cancelled. |
@@ -77,7 +82,7 @@ async function performFetch (network, req, res, source) { | |||
case 405: | |||
throw rpcErrors.rpc.methodNotFound() | |||
|
|||
case 418: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure why this was here but it is no longer correct per the docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Just one optional suggestion.
This PR updates the project to use Infura's v3 API.