Skip to content

Conversation

@robertvoinescu-work
Copy link
Contributor

When using an impersonated service account, scopes specified in the JSON file were previously ignored.

This change updates the CredentialFactory to read the scopes array from the impersonated credential configuration. Programmatically set scopes via CreateScoped will continue to take precedence over the scopes defined in the JSON file.

b/450323529

@robertvoinescu-work robertvoinescu-work requested a review from a team as a code owner October 15, 2025 20:27
@robertvoinescu-work robertvoinescu-work force-pushed the users/robertvoinescu/impersonated_credential_scopes branch from 4331162 to 4c8b268 Compare October 15, 2025 20:31
@robertvoinescu-work robertvoinescu-work self-assigned this Oct 15, 2025
@robertvoinescu-work robertvoinescu-work force-pushed the users/robertvoinescu/impersonated_credential_scopes branch 3 times, most recently from 93afb1e to bdd0b84 Compare October 15, 2025 21:03
Copy link
Contributor

@amanda-tarafa amanda-tarafa left a comment

Choose a reason for hiding this comment

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

A couple of minor changes, but this looks great, thanks.

@robertvoinescu-work robertvoinescu-work force-pushed the users/robertvoinescu/impersonated_credential_scopes branch from bdd0b84 to 5b5c6fe Compare October 16, 2025 18:35
@robertvoinescu-work robertvoinescu-work force-pushed the users/robertvoinescu/impersonated_credential_scopes branch from 5b5c6fe to 586d1d9 Compare October 16, 2025 18:40
When using an impersonated service account, scopes specified in the JSON file were previously ignored.

This change updates the `CredentialFactory` to read the `scopes` array from the impersonated credential
configuration. Programmatically set scopes via `CreateScoped` will continue to take precedence over the scopes
defined in the JSON file.
@robertvoinescu-work robertvoinescu-work force-pushed the users/robertvoinescu/impersonated_credential_scopes branch from 586d1d9 to 1008f57 Compare October 16, 2025 18:43
Copy link
Contributor

@amanda-tarafa amanda-tarafa left a comment

Choose a reason for hiding this comment

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

Thanks!

@robertvoinescu-work robertvoinescu-work merged commit 589197d into googleapis:main Oct 16, 2025
4 checks passed
@robertvoinescu-work robertvoinescu-work deleted the users/robertvoinescu/impersonated_credential_scopes branch October 20, 2025 21:02
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