Skip to content

Implement MFA service's ReplicateValidatedMFAChallenge RPC.#63465

Merged
cthach merged 10 commits intomasterfrom
cthach/impl-replicate-mfa-rpc
Feb 9, 2026
Merged

Implement MFA service's ReplicateValidatedMFAChallenge RPC.#63465
cthach merged 10 commits intomasterfrom
cthach/impl-replicate-mfa-rpc

Conversation

@cthach
Copy link
Contributor

@cthach cthach commented Feb 3, 2026

Resolves #61552

This RPC is intended to be used in a Trusted Clusters deployment configuration of Teleport.

It is called by the reverse tunnel server to store the ValidatedMFAChallenge resource in the target cluster when the resource is created in the root cluster.

The logic for the reverse tunnel server to monitor the event stream and call the RPC is will added in #61542.

Signed-off-by: Chris Thach <chris.thach@goteleport.com>
@cthach cthach self-assigned this Feb 3, 2026
@cthach cthach added mfa Issues related to Multi Factor Authentication no-changelog Indicates that a PR does not require a changelog entry labels Feb 3, 2026
Signed-off-by: Chris Thach <chris.thach@goteleport.com>
Signed-off-by: Chris Thach <chris.thach@goteleport.com>
@cthach cthach marked this pull request as ready for review February 3, 2026 22:31
@github-actions github-actions bot requested review from Tener and tele-lion February 3, 2026 22:31
@cthach cthach requested a review from rosstimothy February 3, 2026 22:33
Signed-off-by: Chris Thach <chris.thach@goteleport.com>
Signed-off-by: Chris Thach <chris.thach@goteleport.com>
@cthach
Copy link
Contributor Author

cthach commented Feb 5, 2026

Friendly ping @tele-lion @Tener

Please take a look when you have a moment 🙏🏾

@cthach cthach removed the request for review from Tener February 6, 2026 21:06
@cthach cthach removed the request for review from tele-lion February 7, 2026 21:32
@cthach
Copy link
Contributor Author

cthach commented Feb 7, 2026

Requesting new reviewers


chal := &mfav1.ValidatedMFAChallenge{
Kind: types.KindValidatedMFAChallenge,
Version: "v1",
Copy link
Contributor

Choose a reason for hiding this comment

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

Replace with constant?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I didn't know we had a constant for that. Thank you!

Replaced in d2e6c8e

@cthach cthach added this pull request to the merge queue Feb 9, 2026
Merged via the queue into master with commit e24c87c Feb 9, 2026
42 of 45 checks passed
@cthach cthach deleted the cthach/impl-replicate-mfa-rpc branch February 9, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mfa Issues related to Multi Factor Authentication 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.

Implement the MFAService's ReplicateValidatedMFAChallenge RPC

3 participants