Skip to content
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

Silence the deployer ethers provider #105

Closed
bbenligiray opened this issue Oct 15, 2020 · 4 comments
Closed

Silence the deployer ethers provider #105

bbenligiray opened this issue Oct 15, 2020 · 4 comments
Labels
deployer About the @airnode/deployer package

Comments

@bbenligiray
Copy link
Member

Deployer sometimes outputs this to the terminal

========= NOTICE =========
Request-Rate Exceeded  (this message will not be repeated)

The default API keys for each service are provided as a highly-throttled,
community resource for low-traffic projects and early prototyping.

While your application will continue to function, we highly recommended
signing up for your own API keys to improve performance, increase your
request rate/limit and enable other perks, such as metrics and advanced APIs.

For more details: https://docs.ethers.io/api-keys/
==========================

The default provider needs to be overriden to silence this message:
ethers-io/ethers.js#1036

@bbenligiray bbenligiray added the deployer About the @airnode/deployer package label Oct 15, 2020
@bbenligiray
Copy link
Member Author

Turning off all logs doesn't work
ethers-io/ethers.js#379

@ricmoo
Copy link

ricmoo commented Oct 23, 2020

There is probably no need to override anything. Simply make sure you pass in your own keys. Or if you do sub-class it, return your own default keys to the static getApiKey method. :)

@bbenligiray
Copy link
Member Author

@ricmoo Thanks for dropping by :)

This is a sort of a special case where the user doesn't know what an Ethereum provider is, doesn't have an API key to insert and will be running this only once (in a while).

We want to be more chain-agnostic anyway so I think we will move away from using the default ethers provider.

@bbenligiray
Copy link
Member Author

We're now using the provider URL from config.json
27f22f3#diff-5663c2c90283b2e61f87d95800ccea766b06ceb858857e46acdb5e48840b6ea8R21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployer About the @airnode/deployer package
Projects
None yet
Development

No branches or pull requests

2 participants