-
Notifications
You must be signed in to change notification settings - Fork 615
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(workers): Adds support for Workers secrets #475
Conversation
SetWorkersSecret creates or updates a secret DeleteWorkersSecret deletes a secret ListWorkersSecrets lists all secrets
Support for worker plain keys and secrets was added in #465 last week. Are there any differences in this PR not in that one? |
@jacobbednarz this PR does not require uploading a Worker script to set a secret. It uses the |
🙌 thanks for the explanation |
@jacobbednarz let me know if this is a PR you'd consider merging. |
The only real prerequisites I have for changes are:
Other than that, I’m happy to review the PR. |
They're used by Wrangler but I can't find the relevant documentation on api.cloudflare.com, so I guess these are non-public endpoints. 😞 |
I couldn’t find them on developers.cloudflare.com either but if we can get the documentation added, I’m a 👍 to the PR. |
@jacobbednarz any pointers on how to update the documentation? I couldn't find a repo for the api-schema, just the repo for developers.cloudflare.com/api/. |
I don’t think there is a public repository for it. If you’re internal to Cloudflare, drop @patryk a message and he can walk you through the steps to get it into a backlog. Otherwise, a support ticket to Cloudflare Support will get routed to the right place. I’m just an interested party/contributed outside of Cloudflare as I maintain the Terraform provider 😊 |
@jacobbednarz ah! No wonder I hadn't seen your name in our internal chat 😅 I'll file a JIRA. |
Don’t worry, you’re not the first to be confused😄 Happy for you to ping me when you’re ready here as this is something that would definitely be useful upstream in Terraform too. |
Thanks! |
SetWorkersSecret creates or updates a secret DeleteWorkersSecret deletes a secret ListWorkersSecrets lists all secrets
SetWorkersSecret creates or updates a secret DeleteWorkersSecret deletes a secret ListWorkersSecrets lists all secrets
Description
TODO
Closes #474
Has your change been tested?
Types of changes
What sort of change does your code introduce/modify?
Checklist: