Skip to content

Add GCP token validation#24492

Merged
atburke merged 1 commit intomasterfrom
atburke/gcp-validate-token
May 3, 2023
Merged

Add GCP token validation#24492
atburke merged 1 commit intomasterfrom
atburke/gcp-validate-token

Conversation

@atburke
Copy link
Copy Markdown
Contributor

@atburke atburke commented Apr 12, 2023

This PR adds validation for ID tokens generated for GCP VMs. Part of #22815.

Should merge after #23946 and before #24493.

Comment thread lib/gcp/gcp.go Outdated
@atburke atburke force-pushed the atburke/gcp-validate-token branch from 93b9ae0 to 04dfab7 Compare April 20, 2023 20:08
@atburke atburke marked this pull request as ready for review April 20, 2023 20:08
@github-actions github-actions Bot requested review from camscale and espadolini April 20, 2023 20:09
@strideynet strideynet self-requested a review April 26, 2023 06:57
Comment thread lib/gcp/gcp.go
@atburke atburke requested a review from strideynet May 1, 2023 21:52
Comment thread lib/gcp/gcp.go Outdated
@atburke atburke force-pushed the atburke/gcp-validate-token branch from f71d851 to 333c60f Compare May 2, 2023 18:27
Copy link
Copy Markdown
Contributor

@mdwn mdwn left a comment

Choose a reason for hiding this comment

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

Just a few nits, but LGTM. Approving!

Comment thread lib/gcp/token_validator.go Outdated
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.

nit: godoc.

Comment thread lib/gcp/token_validator.go Outdated
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.

nit: godoc.

Comment thread lib/gcp/token_validator.go Outdated
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.

nit: godoc.

@atburke atburke force-pushed the atburke/gcp-validate-token branch from 333c60f to ed73bc3 Compare May 3, 2023 18:32
@atburke atburke enabled auto-merge May 3, 2023 18:32
@atburke atburke added this pull request to the merge queue May 3, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 3, 2023
@atburke atburke added this pull request to the merge queue May 3, 2023
Merged via the queue into master with commit e17e2df May 3, 2023
@atburke atburke deleted the atburke/gcp-validate-token branch May 3, 2023 19:21
atburke added a commit that referenced this pull request May 12, 2023
This change adds validation for ID tokens generated for GCP VMs. (#24492)
atburke added a commit that referenced this pull request May 15, 2023
* Add GCP token validation

This change adds validation for ID tokens generated for GCP VMs. (#24492)

* Add GCP join method (#24493)

This change implements the GCP join method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants