-
-
Notifications
You must be signed in to change notification settings - Fork 734
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
Implements DecodingStrategy. #1294
Conversation
👇 Click on the image for a new way to code review
Legend |
16d62a7
to
1b0aad0
Compare
Fixes #920 Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
a69de38
to
f5bc282
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
/approve |
I will leave this one open so other people can jump in and take a look :) |
@moolen we already merged as you are on vacations, but feel free to open up issues if you spot something weird here |
This is a proposed solution to implement DecodingStrategy, with some strategies available for base64 and base64url encoding formats.
Current implementation defaults to
None
, in order to preserve current External Secrets behavior, as some secret values might get misconverted to base64 due to their nature (for instance, numbered values).Fixes #920
Signed-off-by: Gustavo Carvalho gustavo.carvalho@container-solutions.com