Skip to content
This repository was archived by the owner on Mar 6, 2026. It is now read-only.

fix: missing ssj for impersonate cred#1377

Merged
arithmetic1728 merged 1 commit intomainfrom
fix_imp
Aug 31, 2023
Merged

fix: missing ssj for impersonate cred#1377
arithmetic1728 merged 1 commit intomainfrom
fix_imp

Conversation

@arithmetic1728
Copy link
Contributor

b/294308809

For the impersonated credential, when the source cred is a service account and self signed jwt is enabled for source cred, after we create a copy of the source cred with IAM scopes, the source cred's internal _jwt_credential is lost so self signed jwt can no longer be used. We need to recreate one with the IAM scopes so self signed jwt can be used.

@arithmetic1728 arithmetic1728 requested review from a team August 31, 2023 21:18
Copy link
Contributor

@BigTailWolf BigTailWolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@arithmetic1728 arithmetic1728 merged commit 7d453dc into main Aug 31, 2023
@arithmetic1728 arithmetic1728 deleted the fix_imp branch August 31, 2023 21:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants