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

fix(secrets): handle params with no secretstore references as plaintext #13812

Closed
wants to merge 1 commit into from

Conversation

redbaron
Copy link
Contributor

Required for all PRs

resolves #13807

If secret value has no reference to secret stores, handle it as a plaintext string and save scarce mlockable memory

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Aug 22, 2023
@telegraf-tiger
Copy link
Contributor

@powersj
Copy link
Contributor

powersj commented Aug 28, 2023

next steps: similar to #13804, want to get on the same page with Sven and go through each of these.

@srebhan
Copy link
Member

srebhan commented Sep 26, 2023

Superseeded by #13998. Let's continue there...

@srebhan srebhan closed this Sep 26, 2023
@redbaron redbaron deleted the secret-plaintext branch September 26, 2023 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

secrets with no secret store references should behave like plain strings
3 participants