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.

Sign up for GitHub

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

Jump to bottom

Rotate Alchemy Keys #8219

Merged
merged 4 commits into from
Jun 25, 2024
Merged

Rotate Alchemy Keys #8219

merged 4 commits into from
Jun 25, 2024

Conversation

timolegros
Copy link
Collaborator

@timolegros timolegros commented Jun 21, 2024 •
edited
Loading

Link to Issue

Closes: #8193

Description of Changes

  • Adds a migration which rotates all Alchemy keys

Test Plan

  • Set a new ETH_ALCHEMY_API_KEY in packages/commonwealth/.env (cannot use root .env) -> DM me for the key.

Deployment Plan

  1. Set NEW_ALCHEMY_PRIVATE_KEY on all Heroku apps to the key of the new Private Alchemy app
  2. Set NEW_ALCHEMY_PUBLIC_KEY on all Heroku apps to the key of the new Public Alchemy app
  3. Create PSA to all eng to update their local ETH_ALCHEMY_API_KEY in root .env and packages/commonwealth/.env to the key of the new Dev Alchemy app.

Other Considerations

  • This completes the Alchemy app/key clean-up + update.

@timolegros timolegros changed the title Rotate Alchemy keys Rotate Alchemy Keys Jun 21, 2024
@timolegros timolegros temporarily deployed to commonwealth-frick June 24, 2024 19:52 Inactive
@timolegros timolegros temporarily deployed to commonwealth-frick June 25, 2024 15:34 Inactive
@timolegros timolegros added the deployment plan (PRs only) requires manual infrastructure changes on release label Jun 25, 2024
@timolegros timolegros marked this pull request as ready for review June 25, 2024 15:46
@timolegros timolegros merged commit 8cd12ef into master Jun 25, 2024
10 checks passed
@timolegros timolegros deleted the tim/rotate-alchemy-keys-p2 branch June 25, 2024 18:33
ilijabojanovic pushed a commit that referenced this pull request Jun 26, 2024
@timolegros @ilijabojanovic
Rotate Alchemy Keys (#8219)
2b34e51 
* rotate alchemy keys migration

* update load-db.sh script to update chain node urls

* update migration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rbennettcw rbennettcw rbennettcw approved these changes

@kurtisassad kurtisassad kurtisassad approved these changes

Assignees
No one assigned
Labels
deployment plan (PRs only) requires manual infrastructure changes on release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Alchemy
3 participants
@timolegros @kurtisassad @rbennettcw