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

feat: use workload identity federation to gain access to google #150

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

midu-git
Copy link
Contributor

Issue #, if available:
The code doesn't support Workload Identity Federation, which enables interaction with Google services without requiring to provide any sensitive credential data, c.f. here.

Description of changes:

  • Add an optional parameter to specify a service account to be used by from the Workload Identity Federation chain. This is the service account for which the JSON credential file needs to be downloaded without Workload Identity Federation.
  • Extended the code to allow using Google Application Default Credentials if no credential file is explicitly specified.
  • Extended the SAM configuration to include the new parameter.
  • Replace deprecated ioutil.ReadFile by os.ReadFile.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@midu-git midu-git force-pushed the feat/google_workload_identity_federation branch from fb03a9e to 1a09e20 Compare October 26, 2023 05:00
@midu-git midu-git force-pushed the feat/google_workload_identity_federation branch from 1a09e20 to c4248ad Compare November 3, 2023 06:46
@midu-git
Copy link
Contributor Author

midu-git commented Nov 3, 2023

Hi @ChrisPates. Can I provide any support here?

@ChrisPates
Copy link
Contributor

ChrisPates commented Nov 3, 2023 via email

@midu-git
Copy link
Contributor Author

Hi @ChrisPates, it have been a while since your last message. Is there a plan for when you will start working on this PR? Can I support you somehow?

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