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

AWS Secrets Management Vault doesn't update existing secrets #373

Closed
rafaelmag110 opened this issue Jul 10, 2024 · 1 comment · Fixed by #379
Closed

AWS Secrets Management Vault doesn't update existing secrets #373

rafaelmag110 opened this issue Jul 10, 2024 · 1 comment · Fixed by #379
Assignees
@zub4t
Labels
bug_report Suspected bugs, awaiting triage triage all new issues awaiting classification

Comments

@rafaelmag110
Copy link
Contributor

Bug Report

Describe the Bug

The current implemention of the AWS Vault storeSecret method does not update an existing secret.
This behavior is unexpected, as other implementations Vault interface (such as the hashicorp vault) do update the secret.

This unexpected behavior also breaks the current functioning of the EDR cache, since it needs to update the underlying secret when an EDR is refreshed.
@github-actions github-actions bot added the triage all new issues awaiting classification label Jul 10, 2024
@ndr-brt ndr-brt added the bug_report Suspected bugs, awaiting triage label Jul 11, 2024
@zub4t
Copy link
Contributor

zub4t commented Jul 16, 2024

Can I have this issue ? @ndr-brt @rafaelmag110

@zub4t zub4t mentioned this issue Jul 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zub4t zub4t

Labels
bug_report Suspected bugs, awaiting triage triage all new issues awaiting classification
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Modifies the AWS vault implementation to update existing secrets zub4t/Technology-Aws
3 participants
@ndr-brt @rafaelmag110 @zub4t