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

feat: secrets unset-all #3994

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: secrets unset-all #3994

wants to merge 1 commit into from

Conversation

samjtro
Copy link

@samjtro samjtro commented Oct 10, 2024

Change Summary

What and Why:

added functionality for 'fly secrets unset-all', which unsets all secrets with one command.

How:

TL:DR - added runUnsetAll to secrets.New(); runUnsetAll uses client.GetAppSecrets, otherwise its the same code as runUnset.

gets all secrets for the current fly app using client.GetAppSecrets. iterates thru to get the names and add to a string array, then calls UnsetSecretsAndDeploy from runUnset with the secrets. essentially the same code as runUnset, but getting secrets again using the same strategy as other components in the library.

i have tested multiple times on my deployments and it works like a charm!

Related to:

#3973


Documentation

  • Fresh Produce
  • In superfly/docs, or asked for help from docs team
  • n/a

- ref: superfly#3973
- desc: add runUnsetAll to secrets.New(); runUnsetAll uses client.GetAppSecrets, otherwise its the same code as runUnset
@samjtro samjtro changed the title enhancement: secrets unset-all feat: secrets unset-all Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant