Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

command to delete a secret from a worker #912

Closed
ashleymichal opened this issue Nov 26, 2019 · 0 comments · Fixed by #1045
Closed

command to delete a secret from a worker #912

ashleymichal opened this issue Nov 26, 2019 · 0 comments · Fixed by #1045
Labels
feature Feature requests and suggestions regression Something is broken, but works in previous releases

Comments

@ashleymichal
Copy link
Contributor

wrangler should support a command that allows the user to remove a secret environment variable from the script of their choosing. should confirm deletion before carrying out the command. should allow user to pipe yes as their input.

Syntax

$ wrangler secret:delete secret-name
=> Are you sure?
# or
$ yes | wrangler secret:create secret-name

this command should support the environment flag; when not present, the command should remove the secret in the context of the top level worker. when present, the command should remove the secret in the context of the worker specified in the env.

Depends on

  • workers config api upload change; if the upload script endpoint explicitly deletes secret bindings if they are not present in the metadata sent with the upload form, then this endpoint and command are unnecessary, and the user can simply remove the secret binding name from their wrangler.toml.
  • add endpoint to workers configuration api to delete a secret binding from a worker
  • add endpoint definition to cloudflare-rs for calling the above endpoint
@ashleymichal ashleymichal added feature Feature requests and suggestions priority - low design labels Nov 26, 2019
@ashleymichal ashleymichal added this to the Wrangler Secret support milestone Nov 26, 2019
@ashleymichal ashleymichal linked a pull request Feb 11, 2020 that will close this issue
@ashleymichal ashleymichal added status - PR attached regression Something is broken, but works in previous releases and removed cross-team - API design labels Feb 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature Feature requests and suggestions regression Something is broken, but works in previous releases
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant