Skip to content

Deprecate /webapi/ssh/certs + Use 2-step MFA login flow for Teleport Connect#47153

Merged
Joerger merged 7 commits intomasterfrom
joerger/deprectate-password-login-endpoint
Oct 4, 2024
Merged

Deprecate /webapi/ssh/certs + Use 2-step MFA login flow for Teleport Connect#47153
Joerger merged 7 commits intomasterfrom
joerger/deprectate-password-login-endpoint

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Oct 3, 2024

Deprecate /webapi/ssh/certs in favor of /webapi/mfa/login/begin+finish and new /webapi/headless/login endpoints.

Now that MFA cannot be disabled or made optional, we can always use the mfa login flow. This allows us to make clients stupider - remove a bunch of second_factor switches and instead just use whatever MFA challenge the server provides.

This also results in an arguably nicer 2-step login flow for OTP, matching the current Webauthn flow. See this slack thread for discussion + new Teleport Connect demo.

Changelog: Use 2-step OTP login flow for Teleport Connect.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 3, 2024

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@Joerger Joerger changed the title Deprecate /webapi/ssh/certs Deprecate /webapi/ssh/certs + Use 2-step MFA login flow for Teleport Connect Oct 3, 2024
Comment thread lib/client/api.go Outdated
Comment thread lib/client/api.go Outdated
Comment thread lib/client/api.go Outdated
@Joerger Joerger mentioned this pull request Oct 3, 2024
@Joerger Joerger force-pushed the joerger/deprectate-password-login-endpoint branch from c7bc384 to 515cd9b Compare October 4, 2024 01:00
@Joerger Joerger force-pushed the joerger/deprectate-password-login-endpoint branch from 515cd9b to 20d569d Compare October 4, 2024 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants