Skip to content

fix(coderd): validate GCP instance token audience - #29765

Merged
dylanhuff-at-coder merged 1 commit into
mainfrom
dylan/plat-655-validate-gcp-audience
Sep 24, 2026
Merged

dylanhuff-at-coder merged 1 commit into
mainfrom
dylan/plat-655-validate-gcp-audience

Conversation

@dylanhuff-at-coder

Copy link
Copy Markdown
Contributor

Google instance identity authentication accepted tokens issued for any audience. A Google-signed token for a registered VM but intended for another service could therefore be exchanged for a Coder agent token.

Require aud=coder before the exchange. The standard GCP agent already requests this audience. Custom clients requesting another audience will receive 401 and need to update. Add coverage for foreign-audience rejection and the returned agent token.

Refs https://linear.app/codercom/issue/PLAT-655/fixcoderdworkspaceresourceauth-validate-audience-on-gcp-instance

@linear-code

linear-code Bot commented Sep 22, 2026

Copy link
Copy Markdown

PLAT-655

@dylanhuff-at-coder
dylanhuff-at-coder marked this pull request as ready for review September 23, 2026 18:32
@dylanhuff-at-coder
dylanhuff-at-coder requested review from a team and Emyrk September 23, 2026 18:33
@dylanhuff-at-coder
dylanhuff-at-coder merged commit b5ce79b into main Sep 24, 2026
54 of 56 checks passed
@dylanhuff-at-coder
dylanhuff-at-coder deleted the dylan/plat-655-validate-gcp-audience branch September 24, 2026 18:41
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 24, 2026
@dylanhuff-at-coder dylanhuff-at-coder added backport/v2.34 Backport PR targeting release/2.34 backport/v2.36 Backport PR targeting release/2.36 backport/v2.35 Backport PR targeting release/2.35 backport/v2.37 Backport PR targeting release/2.37 labels Sep 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/v2.34 Backport PR targeting release/2.34 backport/v2.35 Backport PR targeting release/2.35 backport/v2.36 Backport PR targeting release/2.36 backport/v2.37 Backport PR targeting release/2.37 backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants