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

Airdrop if needed initialized a new KP #151

Open
jim4067 opened this issue Sep 3, 2023 · 2 comments
Open

Airdrop if needed initialized a new KP #151

jim4067 opened this issue Sep 3, 2023 · 2 comments
Labels
client Client related issues

Comments

@jim4067
Copy link
Contributor

jim4067 commented Sep 3, 2023

When initializing a new wallet, try to airdrop devnet sol to it.

@acheroncrypto acheroncrypto added the client Client related issues label Sep 3, 2023
@acheroncrypto
Copy link
Member

This is actually implemented but was disabled in 5f3ab9b because of the extremely restrictive rate limits for devnet. What's worse is if you get rate limited simply because you've asked for an airdrop, you also get rate limited for all the other methods which makes the RPC endpoint unusable for quite a while.

Not exactly sure what's the solution going to be for this problem, do you have anything in mind?

@jim4067
Copy link
Contributor Author

jim4067 commented Sep 4, 2023

  1. We could only enable the airdrop for new wallets. e.g when Keypair.generate() is run.

  2. Since the rate limit for the airdrop is 4 sol/day, we could use local storage or indexedDB to keep track of this and not call the airdrop function if the limit is reached.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Client related issues
Projects
None yet
Development

No branches or pull requests

2 participants