Skip to content

[v16] Unify and simplify MFA Ceremony helpers#47157

Merged
Joerger merged 2 commits intobranch/v16from
joerger/v16/mfa-ceremony-refactor
Oct 16, 2024
Merged

[v16] Unify and simplify MFA Ceremony helpers#47157
Joerger merged 2 commits intobranch/v16from
joerger/v16/mfa-ceremony-refactor

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Oct 3, 2024

Backport #46986 to branch/v16

* Refactor MFA ceremony helpers.

* Refactor session MFA ceremony to use new MFA ceremony helpers.

* Simplify calls to NewMFACeremony.

* Remove remaining usage of tc.PromptMFA in favor of Ceremony.

* Rename prompt constructor.

* Add godoc to ceremony; update tests.

* Cleanup.

* Resolve comments; fix tests.

* Update comments.

* Fix test.

* Fix lint.
@Joerger Joerger force-pushed the joerger/v16/mfa-ceremony-refactor branch from f38a210 to 991db45 Compare October 3, 2024 19:53
@github-actions github-actions Bot added backport desktop-access size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Oct 3, 2024
@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.

1 similar comment
@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 added the no-changelog Indicates that a PR does not require a changelog entry label Oct 8, 2024
@Joerger Joerger enabled auto-merge October 16, 2024 19:06
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from codingllama October 16, 2024 19:07
@Joerger Joerger added this pull request to the merge queue Oct 16, 2024
Merged via the queue into branch/v16 with commit 4a59807 Oct 16, 2024
@Joerger Joerger deleted the joerger/v16/mfa-ceremony-refactor branch October 16, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport desktop-access no-changelog Indicates that a PR does not require a changelog entry size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants