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

feat: SSO MFA - Teleport Connect #48036

Merged
merged 12 commits into from
Nov 7, 2024
Merged

feat: SSO MFA - Teleport Connect #48036

merged 12 commits into from
Nov 7, 2024

Conversation

Joerger
Copy link
Contributor

@Joerger Joerger commented Oct 28, 2024

Part of the implementation of #44699

Handle SSO MFA in Teleport Connect. This PR adds a new option for SSO MFA (when applicable) in the existing MFA prompt dialog:

image

Clicking the option opens the SSO redirect URL and shows a familiar "follow steps in browser window" message:

image

Completing the SSO flow in the browser closes the dialog and successfully continues to the session as normal.

Depends on #46982 and #48035

@Joerger Joerger mentioned this pull request Oct 28, 2024
@Joerger Joerger added the no-changelog Indicates that a PR does not require a changelog entry label Oct 28, 2024
@Joerger Joerger force-pushed the joerger/sso-mfa-connect-base branch from a49742b to 72b86b5 Compare October 28, 2024 23:42
@Joerger Joerger force-pushed the joerger/sso-mfa-connect branch from f5c8854 to a9a4fec Compare October 28, 2024 23:42
lib/teleterm/daemon/mfaprompt.go Outdated Show resolved Hide resolved
lib/client/api.go Outdated Show resolved Hide resolved
@Joerger Joerger force-pushed the joerger/sso-mfa-connect branch from 8cab8f0 to 58e7152 Compare October 31, 2024 18:48
Copy link
Member

@ravicious ravicious left a comment

Choose a reason for hiding this comment

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

I submitted some minor issues, but it looks good overall.

@Joerger Joerger force-pushed the joerger/sso-mfa-connect-base branch from 9a60a6b to f19809c Compare November 4, 2024 18:59
@Joerger Joerger force-pushed the joerger/sso-mfa-connect branch from 58e7152 to 9dde5dd Compare November 4, 2024 19:01
@Joerger Joerger requested review from ravicious and gzdunek November 4, 2024 19:55
@Joerger Joerger requested a review from gzdunek November 5, 2024 19:39
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from avatus November 6, 2024 08:31
@Joerger Joerger force-pushed the joerger/sso-mfa-connect branch from 88e6c90 to b77dba2 Compare November 6, 2024 19:37
@Joerger Joerger changed the base branch from joerger/sso-mfa-connect-base to master November 6, 2024 19:37
@Joerger Joerger enabled auto-merge November 6, 2024 19:37
@Joerger Joerger added this pull request to the merge queue Nov 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 7, 2024
@Joerger Joerger added this pull request to the merge queue Nov 7, 2024
Merged via the queue into master with commit a9979ca Nov 7, 2024
43 checks passed
@Joerger Joerger deleted the joerger/sso-mfa-connect branch November 7, 2024 20:03
@public-teleport-github-review-bot

@Joerger See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/sm ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants