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

Add support for helm secrets #50

Closed
daxmc99 opened this issue Aug 15, 2024 · 2 comments · Fixed by #54
Closed

Add support for helm secrets #50

daxmc99 opened this issue Aug 15, 2024 · 2 comments · Fixed by #54

Comments

@daxmc99
Copy link

daxmc99 commented Aug 15, 2024

Helm releases are further encoded. See https://gist.github.com/DzeryCZ/c4adf39d4a1a99ae6e594a183628eaee

It would be nice if this tool could automatically detected it was dealing with a helm secret and perform the decoding steps

elsesiy added a commit that referenced this issue Jan 2, 2025
Prior to this change, the plugin always assumed secrets to be of type
`Opaque`. This change introduces the secret type for (de)serialization
and adds a small interface that is used to keep the core logic mostly
unchanged but handle different types of secrets. Only support for
helm was added for now but expanding to more types should be
straight-forward

Fixes #50
elsesiy added a commit that referenced this issue Jan 2, 2025
Prior to this change, the plugin always assumed secrets to be of type
`Opaque`. This change introduces the secret type for (de)serialization
and adds a small interface that is used to keep the core logic mostly
unchanged but handle different types of secrets. Only support for
helm was added for now but expanding to more types should be
straight-forward

Fixes #50
elsesiy added a commit that referenced this issue Jan 3, 2025
Prior to this change, the plugin always assumed secrets to be of type
`Opaque`. This change introduces the secret type for (de)serialization
and adds a small interface that is used to keep the core logic mostly
unchanged but handle different types of secrets. Only support for
helm was added for now but expanding to more types should be
straight-forward

Fixes #50
@daxmc99
Copy link
Author

daxmc99 commented Jan 7, 2025

Thank you! ❤️

@elsesiy
Copy link
Owner

elsesiy commented Jan 10, 2025

@daxmc99 you're welcome, would you mind giving it a try before the official release and report back? Thank you!

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 a pull request may close this issue.

2 participants