Skip to content

[v15] Make tctl users add prompt for MFA just once#37198

Merged
Joerger merged 2 commits intobranch/v15from
joerger/v15/reuse-mfa-tctl-users-add
Jan 25, 2024
Merged

[v15] Make tctl users add prompt for MFA just once#37198
Joerger merged 2 commits intobranch/v15from
joerger/v15/reuse-mfa-tctl-users-add

Conversation

@Joerger
Copy link
Copy Markdown
Contributor

@Joerger Joerger commented Jan 24, 2024

Backport #36997 to branch/v15

@github-actions github-actions Bot added backport size/sm tctl tctl - Teleport admin tool labels Jan 24, 2024
@github-actions
Copy link
Copy Markdown
Contributor

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 Jan 24, 2024
* Refactor MFA prompt logic for the API client to enable various MFA
ceremony use cases.

* Address comments.

* Prompt for MFA before creating a user with tctl.

* Check if AdminActionMFA is enforced before attempting to create an MFA auth challenge.
@Joerger Joerger added this pull request to the merge queue Jan 24, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 24, 2024
@Joerger Joerger added this pull request to the merge queue Jan 25, 2024
Merged via the queue into branch/v15 with commit d6c7930 Jan 25, 2024
@Joerger Joerger deleted the joerger/v15/reuse-mfa-tctl-users-add branch January 25, 2024 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants