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

Use Alchemy URL from environment #643

Open
agostbiro opened this issue Sep 3, 2024 · 6 comments
Open

Use Alchemy URL from environment #643

agostbiro opened this issue Sep 3, 2024 · 6 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@agostbiro
Copy link
Member

Problem

There still some hard-coded Alchemy credentials in the Solidity tests codebase (especially in .sol test files). This is a problem, because these publicly available keys are less reliable than per-user, potentially higher throughput keys. It's not high prio to fix it though as the RPC calls they're used for are heavily cached and we haven't seen any timeouts yet.

Definition of Done

When searching for g.alchemy.com or alchemyapi.io, there are no hard-coded credentials.

@agostbiro agostbiro added the good first issue Good for newcomers label Sep 3, 2024
@agostbiro agostbiro added this to the Solidity tests, phase 2 milestone Sep 3, 2024
@KoxyG
Copy link

KoxyG commented Sep 4, 2024

Kindly assign me to this @agostbiro

@agostbiro
Copy link
Member Author

Hi @KoxyG thanks for volunteering to take care of this issue! I've assigned you now. Please fork from the feat/solidity-tests branch and have your PR target this branch.

@agostbiro
Copy link
Member Author

Btw, out of curiosity, how did you find this issue?

@KoxyG
Copy link

KoxyG commented Sep 5, 2024

Thanks. I found it here. https://onchainsquad.com/oss/

@KoxyG
Copy link

KoxyG commented Sep 6, 2024

I am done cloning from the "feat/solidity-tests" branch. What folder am i meant to work on?, is it the "hardhat-tests/test" folder? @agostbiro

@agostbiro
Copy link
Member Author

@KoxyG the relevant files are int crates/foundry directory

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Todo
Development

No branches or pull requests

2 participants