Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OCI join method - Add client auth ceremony #51445

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Conversation

atburke
Copy link
Contributor

@atburke atburke commented Jan 24, 2025

This change adds the client side of the auth ceremony for the Oracle join method (RFD).

Part of #41705.
Follows #51444.

@github-actions github-actions bot requested a review from rudream January 24, 2025 01:49
@atburke atburke added the no-changelog Indicates that a PR does not require a changelog entry label Jan 24, 2025
go.mod Outdated Show resolved Hide resolved
lib/auth/join/join.go Outdated Show resolved Hide resolved
lib/auth/join/oracle/oracle.go Outdated Show resolved Hide resolved
lib/auth/join/oracle/oracle.go Outdated Show resolved Hide resolved
lib/auth/join/oracle/oracle.go Show resolved Hide resolved
@atburke atburke force-pushed the atburke/oci-join-2 branch from e353656 to acaf9c2 Compare January 28, 2025 21:47
@atburke atburke force-pushed the atburke/oci-join-1 branch 4 times, most recently from 9e12491 to 3323789 Compare January 29, 2025 19:51
@atburke atburke force-pushed the atburke/oci-join-2 branch from acaf9c2 to 6ad2308 Compare January 29, 2025 20:21
Base automatically changed from atburke/oci-join-1 to master January 29, 2025 20:31
@atburke atburke force-pushed the atburke/oci-join-2 branch from 6ad2308 to 21128b0 Compare January 29, 2025 21:40
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from rudream January 30, 2025 09:09
return string(canonicalRegion), realm
}

var ociRealms = []string{
Copy link
Contributor

Choose a reason for hiding this comment

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

The realm lookup could be constant time instead of linear if this was a set

Suggested change
var ociRealms = []string{
var ociRealms = map[string]struct{}{

This change adds the client side of the auth ceremony for the
Oracle join method.
@atburke atburke force-pushed the atburke/oci-join-2 branch from d647b43 to db0511e Compare February 4, 2025 01:06
@atburke atburke enabled auto-merge February 4, 2025 01:15
@atburke atburke added this pull request to the merge queue Feb 4, 2025
Merged via the queue into master with commit e15bc74 Feb 4, 2025
44 checks passed
@atburke atburke deleted the atburke/oci-join-2 branch February 4, 2025 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants