Skip to content

fix: strip trailing slashes from automatic GDCH audience#8692

Open
jerrylin3321 wants to merge 3 commits into
googleapis:mainfrom
jerrylin3321:fix/gdch-audience-trailing-slash
Open

fix: strip trailing slashes from automatic GDCH audience#8692
jerrylin3321 wants to merge 3 commits into
googleapis:mainfrom
jerrylin3321:fix/gdch-audience-trailing-slash

Conversation

@jerrylin3321

@jerrylin3321 jerrylin3321 commented Jun 21, 2026

Copy link
Copy Markdown

Follow up PR to #8301

Fixes #8289

See testing in go/gemini-cli-sa

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request modifies the audience formatting in GoogleAuth to remove the trailing slash from the GDCH audience URL, and updates the corresponding unit tests to reflect this change. There are no review comments, and the changes look correct.

@jerrylin3321 jerrylin3321 marked this pull request as ready for review June 21, 2026 03:43
@jerrylin3321 jerrylin3321 requested a review from a team as a code owner June 21, 2026 03:43

@macastelaz macastelaz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for catching and fixing this!

@pearigee pearigee added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2026
@pearigee pearigee added kokoro:force-run Add this label to force Kokoro to re-run the tests. automerge Merge the pull request once unit tests and other checks pass. labels Jun 22, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Merge the pull request once unit tests and other checks pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

auth: Add support for Google Distributed Cloud service identity authentication

4 participants