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: cli env upload #1817

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

quinton11
Copy link
Contributor

@quinton11 quinton11 commented May 12, 2024

Closes #1612
@maidul98

@akhilmhdh wrt the commented out block in secret-service.ts, using the cli secrets set command, where in this case the secret key already exists, instead of updating the existing secret with the new value it always gave this response

error: CallUpdateSecretsV3: Unsuccessful response. Please make sure your secret path, workspace and environment name are all correct [response={"statusCode":400,"message":"Secret already exist","error":"BadRequest"}]
Unable to process secret update request

Commenting out that block resolves it. Is it by design or its a bug?

@akhilmhdh
Copy link
Member

Hey @quinton11 Its a valid block and it should not be commented out. Its a check to ensure the new key submitted doesn't exist

@quinton11
Copy link
Contributor Author

@akhilmhdh pushed an update undoing the comment

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.

.env upload via CLI
2 participants