-
Notifications
You must be signed in to change notification settings - Fork 82
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: added binaryFiles option #211
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work! Thank you for contributing and adding tests.
Can you add a section to the README about using this functionality? I would put it above this section
{ | ||
name: "KRM From Binary File", | ||
dir: "test/krm/binaryfile", | ||
}, | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Can you add the legacy one too?
I ran into the issue that I cannot encrypt a binary file and import that as file secret. With this patch it is possible.
Generated secrets use data instead of stringdata.
Using the base64 encoded data attributes allow the use of encrypted binary files to be used.
This is a followup PR to #210