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

Google Secrets Manager Support #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

ashish-ck
Copy link

Google Secrets manager support in Kork Library.

  • SecretEngine interface implementation class.
  • Test cases for implementation class.
  • Gradle dependency changes.

link108 and others added 10 commits October 7, 2021 20:17
…ort spinnaker#888) (spinnaker#900)

Co-authored-by: j-sandy <jsandy>
Co-authored-by: Sandesh <[email protected]>
Co-authored-by: Cameron Motevasselani <[email protected]>
…ackport spinnaker#891) (spinnaker#901)

Co-authored-by: j-sandy <jsandy>
Co-authored-by: Sandesh <[email protected]>
Co-authored-by: Cameron Motevasselani <[email protected]>
…CVE (backport spinnaker#892) (spinnaker#902)

Co-authored-by: j-sandy <jsandy>
Co-authored-by: Sandesh <[email protected]>
Co-authored-by: Cameron Motevasselani <[email protected]>
Co-authored-by: Cameron Motevasselani <[email protected]>
(cherry picked from commit bd19fe4)

Co-authored-by: Cameron Motevasselani <[email protected]>
…spinnaker#925)

When we moved to Jackson 2.12 there were some constructors that weren't
updated and weren't caught until runtime. I don't know enough about
Jackson's Kotlin integration to why this wasn't an issue. I built and
ran tests on master for this project and everything was fine, even after
making the constructor changes.

I tested this change by using composite builds to bring this into rosco
and see that it starts successfully now. I'm not sure if we'll need to
do similar kork bumps in other services.

(cherry picked from commit de24322)

Co-authored-by: Fernando Freire <[email protected]>
- SecretEngine interface implementation class.
- Test cases for implementation class.
- Gradle dependency changes.
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 this pull request may close these issues.

2 participants